Fix Hibernate 7.4 dropping primary key on order_type_class_map join table

[dkayiwa]

Summary

Prepares the codebase for the Hibernate 7.4 upgrade (#6130). On Hibernate 7.4, the integration-test job fails with org.dbunit.dataset.NoPrimaryKeyException: ORDER_TYPE_CLASS_MAP (97 errors across many ITs).

Root cause

The test harness builds the in-memory H2 schema from Hibernate's hbm2ddl, and dbUnit's REFRESH operation needs a primary key to update existing rows. In Hibernate 7.4, when a @JoinTable's explicit @UniqueConstraint covers exactly the columns that would form the join table's composite primary key, Hibernate emits the unique constraint instead of the primary key.

OrderType.conceptClasses was the only @ManyToMany Set declaring such a redundant @UniqueConstraint, so its order_type_class_map join table lost its primary key (user_role and location_tag_map, which have no explicit unique constraint, kept theirs).

Fix

Remove the redundant @UniqueConstraint. The two FK columns are the natural key of a Set join table, so hbm2ddl generates the composite primary key automatically again — matching the production schema, which already defines that PK via Liquibase (liquibase-update-to-latest-2.0.x.xml). The annotation's uniqueConstraints only ever affected hbm2ddl-generated DDL, so production is unaffected and the change is harmless on the current Hibernate 7.3.6 too.

Testing

Locally bumped Hibernate to 7.4.0 and reran the affected tests:

• Before: ConceptIT, ModuleUtilIT, SchedulerServiceIT, S3StorageServiceIT, Database1_9_7UpgradeIT → 97 errors.
• After: all green, including ValidateHibernateMappingsDatabaseIT (mapping-vs-schema validation) and the 238 OrderServiceTest / OrderTypeTest cases.

Once this merges, Dependabot PR #6130 will rebase and pass.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.06%. Comparing base (9282315) to head (340ebc8).

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #6139      +/-   ##
============================================
+ Coverage     59.04%   59.06%   +0.01%     
- Complexity     9239     9242       +3     
============================================
  Files           693      693              
  Lines         37257    37257              
  Branches       5485     5485              
============================================
+ Hits          21998    22004       +6     
+ Misses        13287    13285       -2     
+ Partials       1972     1968       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.