Skip to content

Fix two sources of flakiness in server_test

Scott French requested to merge work/swfrench/test-flakiness into main
  1. Tests that exercise release-version file updates were previously flaky due to racing read-update (i.e., the watcher may observe file contents that is not valid JSON, and exit). This switches from simple write to staged-write/rename, which should be atomic in all contexts we care about.

  2. TestRunDelay used a maxDuration that did not take randomization into account. This bumps it to the theoretical max, plus a small buffer.

Merge request reports