Skip to content

test: dual DB engine coverage and flaky test fixes#9

Closed
vividctrlalt wants to merge 1 commit intodevelopfrom
test/dual-db-engine-coverage
Closed

test: dual DB engine coverage and flaky test fixes#9
vividctrlalt wants to merge 1 commit intodevelopfrom
test/dual-db-engine-coverage

Conversation

@vividctrlalt
Copy link
Owner

@vividctrlalt vividctrlalt commented Mar 14, 2026

Summary

  • Add BaseMethodTest for per-method Spring context isolation
  • Parameterized DB tests covering both LevelDB and RocksDB engines
  • Migrate 20 tests to BaseMethodTest for proper isolation
  • Fix 5 flaky test root causes: CommonParameter pollution, ZMQ port conflicts, non-deterministic zksnark assertions, invalid ECDSA signature flakiness

Test plan

  • Full framework test suite passes (2333 tests, 0 failures)
  • GitHub Actions CI passes

- Refactor config validation to support dual-engine (LevelDB/RocksDB) test override
- Add BaseMethodTest for per-method Spring context isolation
- Unify DB tests with parameterized dual-engine coverage
- Migrate 176+ BaseTest subclasses to support dual DB engine override
- Fix flaky tests (TrieTest, ConditionallyStopTest, NeedBeanCondition null safety)
- Add ARM64 RocksDB engine test support
- Add RocksDB engine test step to PR check workflow
- Centralize config file constants in TestEnv
@vividctrlalt vividctrlalt force-pushed the test/dual-db-engine-coverage branch from db104c9 to dbb4699 Compare March 15, 2026 16:33
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.

1 participant