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

TestNG 7.4.0 Assert.assertNotSame has a bug which causes ModelConstructionTest TCK to fail #494

Closed
tevans78 opened this issue Oct 4, 2021 · 0 comments · Fixed by #495
Closed

Comments

@tevans78
Copy link
Contributor

tevans78 commented Oct 4, 2021

assertNotSame ought to assert that two objects are not the same (==) but in version 7.4.0 of TestNG it tests if they are equal. This can cause a TCK failure where two object are expected to be different instances even though they are equal. The tests that fail for me are

The bug was reported multiple times in TestNG and has been fixed but is not yet in a released version.
testng-team/testng#2486
testng-team/testng#2616

Although this issue has been found in the MP OpenAPI TCK, it may be a problem across the whole of MicroProfile 5.0, where the TestNG version is 7.4.0. I suggest that for now the MicroProfile TCKs should avoid use of Assert.assertNotSame.

@Emily-Jiang Emily-Jiang added this to the MP Open API 3.0 milestone Oct 6, 2021
Emily-Jiang added a commit to Emily-Jiang/microprofile-open-api that referenced this issue Oct 6, 2021
Azquelt pushed a commit to Azquelt/microprofile-open-api that referenced this issue Mar 17, 2022
…kus-quarkus-vertx-web-1.8.2.Final

Bump quarkus-vertx-web from 1.8.1.Final to 1.8.2.Final
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 a pull request may close this issue.

2 participants