Overview of the Feature Request
Four integration test classes pass against the CI stack but never run, because they aren't listed in tests/integration-tests.txt:
| Class | Tests | Time |
|---|---|---|
DataCiteIT |
2 | ~6β10s |
SignedUrlAuthMechanismIT |
1 | <1s |
SiteMapIT |
1 | <1s |
StorageSitesIT |
5 | <1s |
Adding them to the list gives 9 more tests in CI for about 10 seconds.
What kind of user is the feature intended for?
Developers.
What inspired the request?
While working on the CI pipeline in #12782 / #12783, we ran some experiments in our fork (uncch-rdmc/dataverse):
- Run every
*ITclass without the list (uncch-rdmc/dataverse#104, run). Failsafe found the classes by name instead of readingtests/integration-tests.txt. 27 of the 80*ITclasses aren't in the list. Most were excluded for a reason: they need Testcontainers or outside services, they're@Disabled, or they fail against the CI stack. Five unlisted classes passed in all three jobs. - Add those five to the list (uncch-rdmc/dataverse#105, run). All five passed, but
DataRetrieverApiITthen failed in every job. The cause isIpGroupsIT: it grants roles to an IP group covering0.0.0.0β255.255.255.255, and its own source warns that it inflates the collection counts other tests assert. So it's correctly left out. - Add only the other four (same PR, run). Every job was green: 416 integration tests run, 0 failures, 15 skipped (was 407 / 0 / 15), and
DataRetrieverApiITpassed.
What existing behavior do you want changed?
Add DataCiteIT, SignedUrlAuthMechanismIT, SiteMapIT and StorageSitesIT to tests/integration-tests.txt.
Any brand new behavior do you want to add to Dataverse?
No.
Any open or closed issues related to this feature request?
12782
Are you thinking about creating a pull request for this feature?
Yes, a pull request will follow.