HomeSoftware Heritage

tests: only flush() the kafka journal writer once per test

This commit no longer exists in the repository. It may have been part of a branch which was deleted.

Description

tests: only flush() the kafka journal writer once per test

instead of flushing it n times per test. Since the call to kafka
Producer.flush() takes about 1s, reducing the number of calls to this
method significantly reduce the execution time of the tests.

This required a small refactoring of the JournalBackfiller class to make
the journal writer live out of the scope of the run() so the test can
access the journal writer instance and call the flush() method.

Requires swh-journal >= 1.2.0

Details

Provenance
douarddaAuthored on Oct 18 2022, 3:43 PM
douarddaPushed on Oct 21 2022, 5:48 PM
Differential Revision
D8740: tests: only flush() the kafka journal writer once per test
Build Status
Buildable 32502
Build 50899: test-and-buildJenkins console · Jenkins

Commit No Longer Exists

This commit no longer exists in the repository.