riverqueue / river

Fast and reliable background jobs in Go
https://riverqueue.com
Mozilla Public License 2.0
2.86k stars 68 forks source link

fix flaky insert notify limiter test #391

Closed bgentry closed 1 month ago

bgentry commented 1 month ago

The insert notify limiter uses the FetchCooldown setting for its own debounce interval. This defaults to 100ms, which is short enough to occasionally fail in GitHub Actions when testing two trigger attempts back to back.

Failed on this run:

        client_test.go:1480: 
                Error Trace:    /home/runner/work/river/river/client_test.go:1480
                Error:          Should be false
                Test:           Test_Client_InsertMany/DoesNotTriggerInsertNotificationForNonAvailableJob