Skip to content

Ensure ITestResult injected to @AfterMethod is apt - #3007

Merged
krmahadevan merged 1 commit into
testng-team:masterfrom
krmahadevan:bugfix/3006
Nov 21, 2023
Merged

krmahadevan merged 1 commit into
testng-team:masterfrom
krmahadevan:bugfix/3006

Conversation

@krmahadevan

Copy link
Copy Markdown
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 requested a review from juherr as a code owner November 21, 2023 05:21
Comment thread testng-core/src/main/java/org/testng/internal/invokers/TestInvoker.java Outdated
@krmahadevan
krmahadevan merged commit 1d9dba8 into testng-team:master Nov 21, 2023
@krmahadevan
krmahadevan deleted the bugfix/3006 branch November 21, 2023 09:16
@juherr juherr added this to the 7.9.0 milestone Sep 7, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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