Commit 8fb06ad7 authored by Matthew Slipper's avatar Matthew Slipper Committed by GitHub

op-e2e: remove flaky assertion (#12533)

Removes an assertion that causes this test to fail often when the CI executor is under load. You can verify this failure for yourself by running the test with `GOMAXPROCS=1`. The assertion didn't seem to be providing much value, so I just removed it.
parent baa11be7
......@@ -128,7 +128,6 @@ func TestBasicRPCReceiptsFetcher_Concurrency(t *testing.T) {
mrpc.AssertExpectations(t)
finalNumCalls := int(numCalls.Load())
require.NotZero(finalNumCalls, "BatchCallContext should have been called.")
require.Less(finalNumCalls, numFetchers, "Some IterativeBatchCalls should have been shared.")
}
func runConcurrentFetchingTest(t *testing.T, rp ReceiptsProvider, numFetchers int, receipts types.Receipts, block *RPCBlock) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment