X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/dfe0ec7bfec3fd72cd40d3962e5c8af08d2413d2..0561bd0c3c07257fd58ded6c7cfa5feeae97af57:/services/keepstore/handler_test.go diff --git a/services/keepstore/handler_test.go b/services/keepstore/handler_test.go index dc9bcb117f..751a4a77e3 100644 --- a/services/keepstore/handler_test.go +++ b/services/keepstore/handler_test.go @@ -1,3 +1,7 @@ +// Copyright (C) The Arvados Authors. All rights reserved. +// +// SPDX-License-Identifier: AGPL-3.0 + // Tests for Keep HTTP handlers: // // GetBlockHandler @@ -11,6 +15,7 @@ package main import ( "bytes" + "context" "encoding/json" "fmt" "net/http" @@ -48,7 +53,7 @@ func TestGetHandler(t *testing.T) { defer KeepVM.Close() vols := KeepVM.AllWritable() - if err := vols[0].Put(TestHash, TestBlock); err != nil { + if err := vols[0].Put(context.Background(), TestHash, TestBlock); err != nil { t.Error(err) } @@ -288,10 +293,10 @@ func TestIndexHandler(t *testing.T) { defer KeepVM.Close() vols := KeepVM.AllWritable() - vols[0].Put(TestHash, TestBlock) - vols[1].Put(TestHash2, TestBlock2) - vols[0].Put(TestHash+".meta", []byte("metadata")) - vols[1].Put(TestHash2+".meta", []byte("metadata")) + vols[0].Put(context.Background(), TestHash, TestBlock) + vols[1].Put(context.Background(), TestHash2, TestBlock2) + vols[0].Put(context.Background(), TestHash+".meta", []byte("metadata")) + vols[1].Put(context.Background(), TestHash2+".meta", []byte("metadata")) theConfig.systemAuthToken = "DATA MANAGER TOKEN" @@ -477,7 +482,7 @@ func TestDeleteHandler(t *testing.T) { defer KeepVM.Close() vols := KeepVM.AllWritable() - vols[0].Put(TestHash, TestBlock) + vols[0].Put(context.Background(), TestHash, TestBlock) // Explicitly set the BlobSignatureTTL to 0 for these // tests, to ensure the MockVolume deletes the blocks @@ -564,7 +569,7 @@ func TestDeleteHandler(t *testing.T) { } // Confirm the block has been deleted buf := make([]byte, BlockSize) - _, err := vols[0].Get(TestHash, buf) + _, err := vols[0].Get(context.Background(), TestHash, buf) var blockDeleted = os.IsNotExist(err) if !blockDeleted { t.Error("superuserExistingBlockReq: block not deleted") @@ -572,7 +577,7 @@ func TestDeleteHandler(t *testing.T) { // A DELETE request on a block newer than BlobSignatureTTL // should return success but leave the block on the volume. - vols[0].Put(TestHash, TestBlock) + vols[0].Put(context.Background(), TestHash, TestBlock) theConfig.BlobSignatureTTL = arvados.Duration(time.Hour) response = IssueRequest(superuserExistingBlockReq) @@ -588,7 +593,7 @@ func TestDeleteHandler(t *testing.T) { expectedDc, responseDc) } // Confirm the block has NOT been deleted. - _, err = vols[0].Get(TestHash, buf) + _, err = vols[0].Get(context.Background(), TestHash, buf) if err != nil { t.Errorf("testing delete on new block: %s\n", err) } @@ -940,7 +945,7 @@ func TestGetHandlerClientDisconnect(t *testing.T) { KeepVM = MakeTestVolumeManager(2) defer KeepVM.Close() - if err := KeepVM.AllWritable()[0].Put(TestHash, TestBlock); err != nil { + if err := KeepVM.AllWritable()[0].Put(context.Background(), TestHash, TestBlock); err != nil { t.Error(err) } @@ -957,7 +962,7 @@ func TestGetHandlerClientDisconnect(t *testing.T) { ok := make(chan struct{}) go func() { req, _ := http.NewRequest("GET", fmt.Sprintf("/%s+%d", TestHash, len(TestBlock)), nil) - (&LoggingRESTRouter{MakeRESTRouter()}).ServeHTTP(resp, req) + (&LoggingRESTRouter{router: MakeRESTRouter()}).ServeHTTP(resp, req) ok <- struct{}{} }() @@ -985,7 +990,7 @@ func TestGetHandlerNoBufferLeak(t *testing.T) { defer KeepVM.Close() vols := KeepVM.AllWritable() - if err := vols[0].Put(TestHash, TestBlock); err != nil { + if err := vols[0].Put(context.Background(), TestHash, TestBlock); err != nil { t.Error(err) } @@ -1040,7 +1045,7 @@ func TestUntrashHandler(t *testing.T) { KeepVM = MakeTestVolumeManager(2) defer KeepVM.Close() vols := KeepVM.AllWritable() - vols[0].Put(TestHash, TestBlock) + vols[0].Put(context.Background(), TestHash, TestBlock) theConfig.systemAuthToken = "DATA MANAGER TOKEN"