Skip to content

Streamline TestResult due to expectedExceptions#2815

Merged
juherr merged 1 commit into
testng-team:masterfrom
krmahadevan:fix_2788
Nov 1, 2022
Merged

Streamline TestResult due to expectedExceptions#2815
juherr merged 1 commit into
testng-team:masterfrom
krmahadevan:fix_2788

Conversation

@krmahadevan

Copy link
Copy Markdown
Member

Closes #2788

Fixes #2788 .

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 October 31, 2022 16:26
Comment thread testng-core/src/main/java/org/testng/internal/invokers/TestInvoker.java Outdated
Comment thread testng-core/src/main/java/org/testng/internal/ExitCode.java
@krmahadevan

Copy link
Copy Markdown
Member Author

@juherr - I amended the commit and added the suggested changes. Please take a look

@juherr juherr merged commit 728f480 into testng-team:master Nov 1, 2022
@krmahadevan krmahadevan deleted the fix_2788 branch November 1, 2022 06:50
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.

TestResult.isSuccess() is TRUE when test fails due to expectedExceptions

2 participants