X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/a709514c6bc4e171541106e908316f01ab6fdb60..093ec98e4a065acfc537ea22c08c337c115fe273:/services/keepstore/bufferpool_test.go

diff --git a/services/keepstore/bufferpool_test.go b/services/keepstore/bufferpool_test.go
index 718e2cac60..13e1cb4f33 100644
--- a/services/keepstore/bufferpool_test.go
+++ b/services/keepstore/bufferpool_test.go
@@ -1,15 +1,16 @@
-package main
+// Copyright (C) The Arvados Authors. All rights reserved.
+//
+// SPDX-License-Identifier: AGPL-3.0
+
+package keepstore
 
 import (
-	. "gopkg.in/check.v1"
-	"testing"
+	"context"
 	"time"
-)
 
-// Gocheck boilerplate
-func TestBufferPool(t *testing.T) {
-	TestingT(t)
-}
+	"git.arvados.org/arvados.git/sdk/go/ctxlog"
+	. "gopkg.in/check.v1"
+)
 
 var _ = Suite(&BufferPoolSuite{})
 
@@ -18,11 +19,16 @@ type BufferPoolSuite struct{}
 // Initialize a default-sized buffer pool for the benefit of test
 // suites that don't run main().
 func init() {
-	bufs = newBufferPool(maxBuffers, BLOCKSIZE)
+	bufs = newBufferPool(ctxlog.FromContext(context.Background()), 12, BlockSize)
+}
+
+// Restore sane default after bufferpool's own tests
+func (s *BufferPoolSuite) TearDownTest(c *C) {
+	bufs = newBufferPool(ctxlog.FromContext(context.Background()), 12, BlockSize)
 }
 
 func (s *BufferPoolSuite) TestBufferPoolBufSize(c *C) {
-	bufs := newBufferPool(2, 10)
+	bufs := newBufferPool(ctxlog.TestLogger(c), 2, 10)
 	b1 := bufs.Get(1)
 	bufs.Get(2)
 	bufs.Put(b1)
@@ -31,14 +37,14 @@ func (s *BufferPoolSuite) TestBufferPoolBufSize(c *C) {
 }
 
 func (s *BufferPoolSuite) TestBufferPoolUnderLimit(c *C) {
-	bufs := newBufferPool(3, 10)
+	bufs := newBufferPool(ctxlog.TestLogger(c), 3, 10)
 	b1 := bufs.Get(10)
 	bufs.Get(10)
 	testBufferPoolRace(c, bufs, b1, "Get")
 }
 
 func (s *BufferPoolSuite) TestBufferPoolAtLimit(c *C) {
-	bufs := newBufferPool(2, 10)
+	bufs := newBufferPool(ctxlog.TestLogger(c), 2, 10)
 	b1 := bufs.Get(10)
 	bufs.Get(10)
 	testBufferPoolRace(c, bufs, b1, "Put")
@@ -62,7 +68,7 @@ func testBufferPoolRace(c *C, bufs *bufferPool, unused []byte, expectWin string)
 }
 
 func (s *BufferPoolSuite) TestBufferPoolReuse(c *C) {
-	bufs := newBufferPool(2, 10)
+	bufs := newBufferPool(ctxlog.TestLogger(c), 2, 10)
 	bufs.Get(10)
 	last := bufs.Get(10)
 	// The buffer pool is allowed to throw away unused buffers