Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Draft] JUnit graph proxy support added #78

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GoodforGod
Copy link
Contributor

@GoodforGod GoodforGod commented Jul 4, 2024

  • GraphProxy support when replace component requires itself original as a dependency added
  override fun graph(): KoraGraphModification =
    KoraGraphModification.create()
      .proxyComponent(
        Executor::class.java,
        Function {
          TestExecutor(it)
        },
      )

@GoodforGod GoodforGod added the module: test Kora test modules label Jul 4, 2024
@GoodforGod GoodforGod marked this pull request as draft July 4, 2024 12:43
Copy link

github-actions bot commented Jul 4, 2024

Test Results

579 tests   575 ✅  7m 26s ⏱️
 81 suites    4 💤
 81 files      0 ❌

Results for commit b7c1fa3.

♻️ This comment has been updated with latest results.

@GoodforGod GoodforGod changed the title [Draft] GraphProxy support added [Draft] JUnit graph proxy support added Jul 4, 2024
@GoodforGod GoodforGod requested a review from Squiry July 4, 2024 12:49
@mitasov-ra
Copy link
Contributor

This is the alternative to @Spy, I got it right?

@GoodforGod
Copy link
Contributor Author

This is the alternative to @Spy, I got it right?

и да и нет, тут прикол такой что ты хочешь использовать как бы оригинальный компонент из графа, но обернуть его в какой то свой допустим класс который ты в тестах обьявил и таком еще какое то поведение задал дополнительное (фасад)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: test Kora test modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants