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

[14.0.x] Testsuite backports #11310

Merged
merged 5 commits into from Sep 29, 2023

Conversation

ryanemerson
Copy link
Contributor

@ryanemerson ryanemerson commented Sep 19, 2023

https://issues.redhat.com/browse/ISPN-14749
https://issues.redhat.com/browse/ISPN-14817
https://issues.redhat.com/browse/ISPN-14968
https://issues.redhat.com/browse/ISPN-15143

I included ISPN-14817 as it made the cherry-picking conflict resolution much simpler and will result in less conflicts between 14.0.x and main.

@ryanemerson ryanemerson marked this pull request as ready for review September 20, 2023 13:37
@ryanemerson ryanemerson force-pushed the ISPN-14968_14.0 branch 3 times, most recently from 257424b to 03fd686 Compare September 27, 2023 10:53
ryanemerson and others added 5 commits September 28, 2023 17:08
* ClusteredIT: incorporate ExtensionsIT suite run in CONTAINER mode
* HotRodCacheOperations: incorporate APITest and RemoteAsyncAPITest. Parameterized for protocol versions and types
* HotRodCacheQueries: incorporate ProtobufHotRodRestEntityIT
* HotRodCacheEvents: incorporate JsonEventsTest, ClientEventsTest, ClientFilterEventsTest, ClientCustomEventsTest,
    ClientClusterEventsTest
* SslTest, SslAuthenticationTest, SecureExecTest: removed as more in-depth tests are already present in the server testsuite
- ISPN-15002 TestClient resource shutdown can block indefinitely resulting in testsuite aborting
- Set rerunFailingTestsCount=2 on Jenkins builds

- Refactor PolarionJUnitXMLReporter to handle parallel test execution
  with the Junit5 TestNG engine and output per class .xml files

- Add flaky-test-extractor-maven-plugin to highlight flaky tests in
  Jenkins results

- Convert interface AbstractRemoteIteratorTest to concrete Util class. Avoids TestNG NPE
where interfaces with TestNG assertions are not handled correctly.

- Escape test parameter strings to prevent TestNG regex failures.

- Still utilise TestNG for CDI integration tests. It maybe possible to
  utilise Junit5 here, but we will need to convert the modules to
  utilise org.jboss.arquillian.junit5.
@tristantarrant tristantarrant merged commit d05a6d3 into infinispan:14.0.x Sep 29, 2023
1 check failed
@tristantarrant
Copy link
Member

Merged, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants