evolution-gaming / kafka-journal / 34110512796

89%

push

github

Integration tests - 3x speed up, maintenance refactoring (#994)

Main points:
- "testFull" reduced from 12 minutes to slightly under 4 minutes on M1
- fixed this issue by using test containers on available ports and not
closing them explicitly on JVM exit (see SharedItEnv for details):
https://github.com/evolution-gaming/kafka-journal/issues/961

Details:
- removed unused code and config parts, refactored integration test init
code
- use consumer pool instead of creating a new consumer on each read in
integration tests (the biggest test time win comes from this)
- reworked shared int test services (Cassandra, Kafka, replicator) to
start on random available ports but without explicit stopping in a
custom shutdown hook - this hardens the tests against potential env
issues (occupied ports) and avoids the sbt 2 classloader close issue for
forked test modules
- moved per-int-spec configuration overrides into config files named
after the spec classes for better readability
- fixed race in JournalPerfSpec - the test case code wasn't waiting for
the test data prepare code
- simplified logging config

1121 of 1466 branches covered (76.47%)

Branch coverage included in aggregate %.

6025 of 6527 relevant lines covered (92.31%)

1.63 hits per line