Skip to content
New issue

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

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

Already on GitHub? # to your account

Disable Native DI for BeforeSuite methods #2933

Conversation

krmahadevan
Copy link
Member

Closes #2925

Fixes #2925 .

Did you remember to?

  • Add test case(s)
  • Update CHANGES.txt
  • Auto applied styling via ./gradlew autostyleApply

We encourage pull requests that:

  • Add new features to TestNG (or)
  • Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.

Note: For more information on contribution guidelines please make sure you refer our Contributing section for detailed set of steps.

@krmahadevan krmahadevan requested a review from juherr as a code owner July 14, 2023 10:52
@krmahadevan krmahadevan force-pushed the prevent_itestcontext_injection_beforesuite branch from 488adfb to 72b0b94 Compare July 14, 2023 12:35
@krmahadevan krmahadevan force-pushed the prevent_itestcontext_injection_beforesuite branch from 72b0b94 to 9a69e52 Compare July 14, 2023 12:37
@krmahadevan krmahadevan merged commit a3c87ef into testng-team:master Jul 15, 2023
9 checks passed
@krmahadevan krmahadevan deleted the prevent_itestcontext_injection_beforesuite branch July 15, 2023 02:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue in ITestcontext.getAllTestMethods() with annotation @BeforeSuite
2 participants