Skip to content

[ISSUE #9623] Isolate message store filter tests - #11237

Open
yym108 wants to merge 1 commit into
apache:developfrom
yym108:fix/issue-9623-filter-test-isolation
Open

yym108 wants to merge 1 commit into
apache:developfrom
yym108:fix/issue-9623-filter-test-isolation

Conversation

@yym108

@yym108 yym108 commented Sep 26, 2026

Copy link
Copy Markdown

Which Issue(s) This PR Fixes

Brief Description

MessageStoreWithFilterTest reuses a fixed store directory, so a previous test's files can affect subsequent tests when cleanup fails. Give each test an assured-deletion TemporaryFolder, use an ephemeral HA port, release retrieved message buffers, and close the store and statistics manager even when initialization or teardown fails. A failed store load now fails the test consistently on Windows as well.

How Did You Test This Change?

  • All 3 MessageStoreWithFilterTest cases passed on macOS ARM64 with Zulu JDK 21, including temporary-directory deletion.
  • Maven's configured Checkstyle/SpotBugs checks and git diff --check passed.
  • Local JDK 21 execution used JaCoCo 0.8.12, Javassist 3.30.2-GA, Byte Buddy experimental support and JDK module opens. No build configuration changes are included.
  • Windows execution remains unverified; the reported Windows scenario was addressed through fixture isolation and resource cleanup.

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.

[Enhancement] Fix Failing Tests In MessageStoreWithFilterTest

1 participant