nats-io / nats-server

High-Performance server for NATS.io, the cloud and edge native messaging system.
https://nats.io
Apache License 2.0
16k stars 1.41k forks source link

De-flake TestJetStreamClusterParallelStreamCreation & improve logging when `len(errCh) > 0 ` #6155

Closed MauriceVanVeen closed 1 day ago

MauriceVanVeen commented 1 day ago

TestJetStreamClusterParallelStreamCreation called c.waitOnServerCurrent(nl) whereas it should've called c.waitOnStreamCurrent(nl, globalAccountName, "TEST"). So it waits for the stream to be current, and not just the meta layer.

Also slightly extended logging where we'd only print "Expected no errors, got 1". Now also logging the first received error to give more context what went wrong.

Signed-off-by: Maurice van Veen github@mauricevanveen.com