UFAL/Failing BE tests

[Kasinhou]

Problem description

Issue: https://github.com/dataquest-dev/dspace-customers/issues/524

Manual Testing (if applicable)

• Added to testing scenarios

Copilot review

• Requested review from Copilot

[Copilot]

Pull request overview

This PR addresses failing backend tests (Issue: dataquest-dev/dspace-customers#524) by making integration tests less brittle and aligning Spring Boot cache configuration with DSpace kernel-managed caching.

Changes:

• Update MetadataBitstreamControllerIT to validate the ZIP response by parsing/extracting it instead of byte-for-byte ZIP generation.
• Make BitstreamFormatRestRepositoryIT compute the baseline number of bitstream formats dynamically rather than relying on a hardcoded constant.
• Adjust Spring Boot configuration to avoid creating a competing CacheManager (cache is managed by the DSpace kernel).

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
dspace-server-webapp/src/test/java/org/dspace/app/rest/MetadataBitstreamControllerIT.java	Switches ZIP verification to extract-and-compare approach.
dspace-server-webapp/src/test/java/org/dspace/app/rest/BitstreamFormatRestRepositoryIT.java	Removes hardcoded baseline count; initializes it from the DB per test.
dspace-server-webapp/src/main/resources/application.properties	Disables Boot cache autoconfig to prevent competing CacheManager vs kernel cache.
dspace-api/src/test/java/org/dspace/ctask/general/CreateMissingIdentifiersIT.java	Restores modified config after test to avoid cross-test pollution.
dspace-api/src/test/java/org/dspace/builder/ItemBuilder.java	Fixes indentation in test builder cleanup.

[Copilot]

After calling getNextZipEntry(), the test assumes an entry exists. If the response is not a valid ZIP or is empty, entry will be null and the test will NPE on entry.getName(), which makes failures harder to diagnose. Add an assertion that the entry is non-null (and optionally assert there are no unexpected additional entries).

[Copilot]

bitstreamContent is hardcoded again in this test method, duplicating the value used when creating the Bitstream in setUp(). To avoid future divergence (e.g., setUp changes but the assertion doesn’t), store the content in a shared constant/field or derive expected bytes by retrieving the bitstream content from bts.

[Copilot]

DEFAULT_AMOUNT_FORMATS is no longer a constant (it’s assigned in @before), but it’s still named like a constant (ALL_CAPS). Rename it to a regular field name (e.g., defaultAmountFormats/initialAmountFormats) to match Java conventions and reduce confusion.

[Kasinhou]

Current state:

Error:  Failures: 
Error:    IIIFControllerIT.findOneWithCacheEvictionAfterBitstreamUpdate:1278 JSON path "$.sequences[0].canvases[1].label"
Expected: is "Test label"
     but: was "Original label"
Error:    IIIFControllerIT.findOneWithCacheEvictionAfterItemUpdate:1400 JSON path "$.metadata[0].value"
Expected: is "Public item (revised)"
     but: was "Public item 1"
[INFO] 
Error:  Tests run: 3278, Failures: 2, Errors: 0, Skipped: 55

[kuchtiak-ufal]

See the fixes made in ufal/clarin-dspace repository, particularly:
ufal#1353