Skip to content

Restore some of disabled *Tests - #449

Draft
mgaffigan wants to merge 29 commits into
OpenIntegrationEngine:mainfrom
mgaffigan:maint/restore-tests
Draft

Restore some of disabled *Tests#449
mgaffigan wants to merge 29 commits into
OpenIntegrationEngine:mainfrom
mgaffigan:maint/restore-tests

Conversation

@mgaffigan

@mgaffigan mgaffigan commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

The gradle build, as with the ant build before it, only runs *Test files. Let's fix that. Presumably these were added for a reason.

@github-actions

github-actions Bot commented Sep 19, 2026 •
edited
Loading

Copy link
Copy Markdown

Test Results

 1 files   - 128   1 suites   - 128   32s ⏱️ - 2m 9s
69 tests  - 668  69 ✅  - 668  0 💤 ±0  0 ❌ ±0 
69 runs   - 758  69 ✅  - 752  0 💤  - 6  0 ❌ ±0 

Results for commit 8bfe4da. ± Comparison against base commit fbda474.

This pull request removes 722 and adds 54 tests. Note that renamed tests count towards both. 
com.mirth.commons.encryption.test.DigesterTest ‑ testArgon2d
com.mirth.commons.encryption.test.DigesterTest ‑ testArgon2i
com.mirth.commons.encryption.test.DigesterTest ‑ testArgon2id
com.mirth.commons.encryption.test.DigesterTest ‑ testFallback1
com.mirth.commons.encryption.test.DigesterTest ‑ testFallback2
com.mirth.commons.encryption.test.DigesterTest ‑ testFallback3
com.mirth.commons.encryption.test.DigesterTest ‑ testFallback4
com.mirth.commons.encryption.test.DigesterTest ‑ testPBKDF2
com.mirth.commons.encryption.test.DigesterTest ‑ testSHA256
com.mirth.commons.encryption.test.EncryptionTest ‑ testDigest
…

org.openintegrationengine.smoketest.BlockingResponseTransformerTest ‑ destinationStaysPendingUntilTheResponseTransformerReturns()
org.openintegrationengine.smoketest.CustomMetaDataColumnRedeployTest ‑ redeployAddsRemovesAndRetypesColumns()
org.openintegrationengine.smoketest.DestinationQueueRotationTest ‑ aMessageThatNeverSendsDoesNotBlockTheOnesQueuedBehindIt()
org.openintegrationengine.smoketest.DestinationQueueTest ‑ queuedMessagesSurviveAChannelRestartAndDrainInOrder()
org.openintegrationengine.smoketest.EncryptionAtRestTest ‑ decryptsContentWhenReadBack()
org.openintegrationengine.smoketest.EncryptionAtRestTest ‑ storesEveryContentStageEncrypted()
org.openintegrationengine.smoketest.MessageDeletionTest ‑ deleteConnectorMessage()
org.openintegrationengine.smoketest.MessageDeletionTest ‑ deleteOneMessage()
org.openintegrationengine.smoketest.MessageDeletionTest ‑ removeAllMessages()
org.openintegrationengine.smoketest.MessageDeletionTest ‑ removeAllMessagesClearingStatistics()
…

♻️ This comment has been updated with latest results.

@mgaffigan
mgaffigan force-pushed the maint/restore-tests branch 2 times, most recently from 6b5ce1e to 9e052d8 Compare September 21, 2026 19:17
mgaffigan and others added 27 commits October 26, 2026 13:47
@mgaffigan
Restore some of disabled *Tests
4be641b 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Re-add EDISerializerTests using new fixtures
37167db 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Re-add NCPDP Tests, expand to include D.0 response, add bijection
9cd87c0 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Re-add DICOM tests, thanks Saga
a92bb88 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Drop dead test entrypoints (junit reigns)
3ea4b26 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Restore DocumentSerailizerTests
234b9f5 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Re-add MigrationUtilTest assuming current behavior is correct
187ebf2 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate FilterTransformerTests to fixtures
c69cdcc 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate testOutboundSerializationFailure to fixture test
6a9ca04 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Fix test run outside of container
c2ad991 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate ChannelTests message storage settings to fixtures
6105e9b 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translated DestinationChainTests to fixtures
015840a 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate response tests to fixtures
b3aa941 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate testAfterSend to fixture test
8d9ecaf 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate metadata tests to fixtures
91f88f0 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate testEncryption to ci
dd8f14b 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Add atomic set configuration map entry endpoint
61d311f 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate QueueTests and others to ci and unit tests
f038e26 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate testInsertMessageAttachment to fixture test
fd708ca 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate testDeleteMessage to ci test
c8c8fc1 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Fix statistics errors caused by channel ID scoping
a252a9d 
Issue: OpenIntegrationEngine#456
Issue: OpenIntegrationEngine#457
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Translate StatisticsTests to ci test
2eec577 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Fix 500 after metadata schema changes
d126c7e 
Postgres would have intermittent 500 errors after a metadata column
changed from one type to another. Fix that.

Issue: OpenIntegrationEngine#458
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Ignore precision on metadata ci tests
5840847 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Add comment and warning for MySQL repeatable read
3e92868 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@pacmano1 @mgaffigan
Add an unrecognised message type to the strict-serialization fixture
2a4f344 
MSH-9 names a type no structure jar provides, so HAPI parses it into a
GenericMessage and serialization fails. Distinct from 02-not-hl7, where
the parse itself fails.

Signed-off-by: Finnegan's Owner <44065187+pacmano1@users.noreply.github.com>
@pacmano1 @mgaffigan
Cover strict validation in the strict-serialization fixture
2cacc75 
The same channel with useStrictValidation on, running the same messages.
Nothing currently exercises that configuration, and pinning it records
that validation changes neither the outcome nor the serialized document.

Signed-off-by: Finnegan's Owner <44065187+pacmano1@users.noreply.github.com>
@mgaffigan
Remove Guice, Fix flaky tests
1dd6d5c 
Several tests would fail if run in a shared JVM depending on what tests ran before them.  Traced to setting mocks without cleaning them up.

As it happens, those were the only uses of Guice in the system.  Removed.

Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Rebase to main requires submitOneMessage
8bfe4da 
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
mgaffigan force-pushed the maint/restore-tests branch from 7fa7e89 to 8bfe4da Compare October 26, 2026 20:19
This was referenced Oct 26, 2026
Open
Open
Open
Open
Open
@pacmano1 pacmano1 mentioned this pull request Oct 29, 2026
Open
@mgaffigan mgaffigan mentioned this pull request Oct 29, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mgaffigan @pacmano1