Fix DataRetrieverApiIT.testRetrieveMyDataCollections() for github mig…

[stevenwinship]

What this PR does / why we need it: DataRetrieverApiIT.testRetrieveMyDataCollections() fails in Github

Which issue(s) this PR closes: #12371

• Closes Fix DataRetrieverApiIT.testRetrieveMyDataCollections() on Github actions #12371

Special notes for your reviewer:

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

[stevenwinship]

related to issue #12368

[coveralls]

coverage: 24.96%. remained the same — 12371-fix-dataretriever-api-test-for-github into develop

[pdurbin]

A quick look.

[srmanda-cs]

After a quick check a few days back when I changed User 1 to greater than or equal to, the assertions seemed to be failing for User 2 and User 3 as well. There was always one extra dataset than expected. It was a different ID from user1's extra collection but it remained consistent from test to test. This is the primary reason why I changed it to disabled instead.

[stevenwinship]

@srmanda-cs Did the new test show that the root count changed from 1 to 2? If not then there is another issue. The root count should be added to each user's count and therefore work. I can't get it to fail locally so it's hard to test

[github-actions]

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:12371-fix-dataretriever-api-test-for-github

ghcr.io/gdcc/configbaker:12371-fix-dataretriever-api-test-for-github

🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name.

[srmanda-cs]

@srmanda-cs Did the new test show that the root count changed from 1 to 2? If not then there is another issue. The root count should be added to each user's count and therefore work. I can't get it to fail locally so it's hard to test

I realize that it is incredibly difficult to test, because it is so environment specific :(

I have little to no idea why that specific test is failing. All I realize is that when I changed it to greater than or equal to for user1, user2 returned 18 count instead of the expected 17. The root dataverse count remains constant, it is the number of other dataverses that the users have access to that have increased. Its very similar to how the Mail test was the only one failing along with this when I just ran a full Maven verify.

And as I have mentioned to Phil previously, I really don't want to create more work for y'all than you already have. I can provide you with more testing, more logs, and more specific details but it will take me a lot of time to dig into this as its a novel codebase to me. Which is why I sought your help.

[pdurbin]

I'm thinking we should merge the new GitHub Action first:

• 12367 GitHub actions workflow for jenkins redundancy #12368

Then we should rebase this PR (#12372) after the merge and see if it's passing or not.