Skip to content

PD-13323 Move the integration tests to unit tests, and speed up CI - #7675

Draft
cryptalith wants to merge 46 commits into
mainfrom
lmendoa/PD-13323-integration-to-unit-tests
Draft

cryptalith wants to merge 46 commits into
mainfrom
lmendoa/PD-13323-integration-to-unit-tests

Conversation

@cryptalith

@cryptalith cryptalith commented Sep 7, 2026 •

Copy link
Copy Markdown
Member

Implements PD-13323.

@cryptalith
cryptalith force-pushed the lmendoa/PD-13323-integration-to-unit-tests branch 2 times, most recently from 30e6d2d to e81d09e Compare September 7, 2026 13:42
@github-actions

github-actions Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Unit test coverage

Module Lines Branches Covered / total lines Tests run Lines (db stage)
orcid-activemq n/a n/a 0 / 0 — n/a
orcid-api-common 43.9% 21.8% 1,816 / 4,132 102 —
orcid-api-web 58.1% 35.5% 1,245 / 2,143 1245 5.4%
orcid-core 35.8% 29.1% 12,954 / 36,205 1514 35.4%
orcid-internal-api 66.3% 50.0% 181 / 273 21 —
orcid-message-listener 34.9% 29.1% 945 / 2,704 121 13.2%
orcid-persistence 2.1% ⚠️ 2.9% 141 / 6,689 8 36.9%
orcid-pub-web 54.4% 57.5% 792 / 1,457 405 14.9%
orcid-scheduler-web 28.7% 26.4% 788 / 2,744 52 —
orcid-utils 12.6% ⚠️ 15.8% 473 / 3,762 41 —
orcid-web 49.6% 41.8% 4,899 / 9,871 588 8.2%
Reactor 34.6% 28.5% 24,234 / 69,980 4,097

Files changed in this pull request

File Lines Branches
orcid-test/src/main/java/org/orcid/test/DBUnitTest.java no coverage report for this module —
orcid-test/src/main/java/org/orcid/test/DatabaseTest.java no coverage report for this module —
orcid-test/src/main/java/org/orcid/test/OrcidMockitoJUnitRunner.java no coverage report for this module —

JaCoCo line and branch coverage per module, from each module's own unit tests. No thresholds are enforced; ⚠️ marks under 25% line coverage for information only. The database stage is reported separately, not merged: a DAO covered only there would otherwise read as uncovered.

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.

2 participants