-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
upgrade testng version to 7.8.0 #11462
Conversation
5394b20
to
8e84aa4
Compare
c0ffa31
to
1eee7ed
Compare
28d2918
to
b26d22a
Compare
863eccc
to
5a7c60c
Compare
fa19c34
to
f86d614
Compare
finally got this upgraded after fixing this: #11720 |
f86d614
to
5c2a304
Compare
5c2a304
to
a0d6aae
Compare
a0d6aae
to
a886af1
Compare
Codecov Report
@@ Coverage Diff @@
## master #11462 +/- ##
============================================
- Coverage 67.22% 63.07% -4.15%
+ Complexity 915 207 -708
============================================
Files 1747 2342 +595
Lines 91052 125883 +34831
Branches 14613 19357 +4744
============================================
+ Hits 61206 79400 +18194
- Misses 25726 40825 +15099
- Partials 4120 5658 +1538
Flags with carried forward coverage won't be shown. Click here to find out more. see 666 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
a886af1
to
a3747c3
Compare
a3747c3
to
cece797
Compare
hi @xiangfu0 I observed some command are not working after test verison bump. For example
and I got
do you have an equivalent testing command that has similar function with |
True, I can also reproduce this, will look into it more.
|
you can use |
for simplicity, this is the most effective way:
|
i see. thanks @xiangfu0, i tried the last command and it worked perfectly. |
maven-surefire-plugin
usage in other moduleOfflineClusterIntegrationTest