From faaef83954e9564a9ec054c464a59e041c8c2b3d Mon Sep 17 00:00:00 2001 From: Alexander Morozov Date: Fri, 11 Sep 2015 10:22:41 -0700 Subject: [PATCH] Use less strict condition to avoid problems with concurrency In latest go versions TestWatcher fails pretty often, because it is "more concurrent" now. Reproducible with go master: while go test github.com/mholt/caddy/middleware/markdown; do :; done Signed-off-by: Alexander Morozov --- middleware/markdown/watcher_test.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/middleware/markdown/watcher_test.go b/middleware/markdown/watcher_test.go index c04f8e4f..d09d07c6 100644 --- a/middleware/markdown/watcher_test.go +++ b/middleware/markdown/watcher_test.go @@ -17,10 +17,11 @@ func TestWatcher(t *testing.T) { i++ out += fmt.Sprint(i) }) - time.Sleep(interval * 8) + // wait little more because of concurrency + time.Sleep(interval * 9) stopChan <- struct{}{} - if expected != out { - t.Fatalf("Expected %v, found %v", expected, out) + if !strings.HasPrefix(out, expected) { + t.Fatalf("Expected to have prefix %v, found %v", expected, out) } out = "" i = 0