refs #11906

[arvados.git] / services / keepstore / s3_volume_test.go

diff --git a/services/keepstore/s3_volume_test.go b/services/keepstore/s3_volume_test.go
index 3fc56391ccf78a0daf71138a111d831d78aca152..c2084eea8d58718f98f223c8380320e9d8e80bf2 100644 (file)
--- a/services/keepstore/s3_volume_test.go
+++ b/services/keepstore/s3_volume_test.go
@@ -1,3 +1,7 @@
+// Copyright (C) The Arvados Authors. All rights reserved.
+//
+// SPDX-License-Identifier: AGPL-3.0
+
 package main
 
 import (
@@ -174,11 +178,11 @@ func (s *StubbedS3Suite) testContextCancel(c *check.C, testFunc func(context.Con
        handler.unblock = make(chan struct{})
        defer close(handler.unblock)
 
-       doneGet := make(chan struct{})
+       doneFunc := make(chan struct{})
        go func() {
                err := testFunc(ctx, v)
                c.Check(err, check.Equals, context.Canceled)
-               close(doneGet)
+               close(doneFunc)
        }()
 
        timeout := time.After(10 * time.Second)
@@ -187,9 +191,9 @@ func (s *StubbedS3Suite) testContextCancel(c *check.C, testFunc func(context.Con
        // Get() is waiting for an s3 operation.
        select {
        case <-timeout:
-               c.Fatal("timed out waiting for Get to call our handler")
-       case <-doneGet:
-               c.Fatal("Get finished without calling our handler!")
+               c.Fatal("timed out waiting for test func to call our handler")
+       case <-doneFunc:
+               c.Fatal("test func finished without even calling our handler!")
        case <-handler.requested:
        }
 
@@ -198,7 +202,7 @@ func (s *StubbedS3Suite) testContextCancel(c *check.C, testFunc func(context.Con
        select {
        case <-timeout:
                c.Fatal("timed out")
-       case <-doneGet:
+       case <-doneFunc:
        }
 }