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

Ensure ITestResult injected to @AfterMethod is apt #3007

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

krmahadevan
Copy link
Member

Closes #3006

Fixes #3006 .

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 merged commit 1d9dba8 into testng-team:master Nov 21, 2023
6 of 9 checks passed
@krmahadevan krmahadevan deleted the bugfix/3006 branch November 21, 2023 09:16
# 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.

ITestResult injected at @AfterMethod incorrect when a configuration method failed
2 participants