Skip to content

Fix TestScanHandlesCancelledContext having a WaitGroup without a delta and a few other log processor flaky tests

Georgi N. Georgiev requested to merge fix_TestScanHandlesCancelledContext_test into master

What does this MR do?

The goroutine would finish a second later and depending on whether the tests in this file were finished it could cause a panic. Moreover, the test itself doesn't wait for the goroutine at all since Wait returns immediately without a delta.

Example job

Why was this MR needed?

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Edited by Georgi N. Georgiev

Merge request reports