Skip to content

DRYD-2089: Updates to Integration Tests#534

Merged
mikejritter merged 6 commits into
collectionspace:upgrade/dryd-1633-core-updatesfrom
mikejritter:upgrade/dryd-2089-tests
May 12, 2026
Merged

DRYD-2089: Updates to Integration Tests#534
mikejritter merged 6 commits into
collectionspace:upgrade/dryd-1633-core-updatesfrom
mikejritter:upgrade/dryd-2089-tests

Conversation

@mikejritter
Copy link
Copy Markdown
Contributor

What does this do?

  • Add missing dependencies needed for testing
  • Encode '%' in XML Replay tests
  • Update persistence.xml for tests which require it
  • Remove unused hibernate xml config

Why are we doing this? (with JIRA link)
Jira: https://collectionspace.atlassian.net/browse/DRYD-2089

This is the last step of getting the upgrade branch ready for merging back into develop. It makes updates so that all of the tests in the repository will pass. Mostly it's adding dependencies which were dropped likely due to shuffling of things, as well as some more hibernate configuration changes which were expected. Any configuration which isn't being used has been removed as well as having any build.xml/pom.xml references removed.

How should this be tested? Do these changes have associated tests?

  • Build and start CollectionSpace with the following tenants enabled:
    • anthro, botgarden, core, materials, testsci
  • Run mvn test and see that all tests pass
    • Some flaky tests still exist. It might be worth trying to document them when we notice but I haven't looked into what causes it.

Dependencies for merging? Releasing to production?
This will need full regression testing once we merge into develop.

Has the application documentation been updated for these changes?
No. Build documentation needs to be updated.

Did someone actually run this code to verify it works?
@mikejritter tested locally

Have any new security vulnerabilities been handled?
n/a

@mikejritter mikejritter merged commit 9c0c759 into collectionspace:upgrade/dryd-1633-core-updates May 12, 2026
@mikejritter mikejritter deleted the upgrade/dryd-2089-tests branch May 12, 2026 20:00
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 this pull request may close these issues.

1 participant