diff options
Diffstat (limited to 'internal/gitaly/service/objectpool/create_test.go')
-rw-r--r-- | internal/gitaly/service/objectpool/create_test.go | 23 |
1 files changed, 21 insertions, 2 deletions
diff --git a/internal/gitaly/service/objectpool/create_test.go b/internal/gitaly/service/objectpool/create_test.go index 09ebca2a9..35afaaf0a 100644 --- a/internal/gitaly/service/objectpool/create_test.go +++ b/internal/gitaly/service/objectpool/create_test.go @@ -156,10 +156,21 @@ func TestDelete(t *testing.T) { for _, tc := range []struct { desc string + noPool bool relativePath string error error }{ { + desc: "no pool in request fails", + noPool: true, + error: errMissingPool, + }, + { + desc: "deleting outside pools directory fails", + relativePath: ".", + error: errInvalidPoolDir, + }, + { desc: "deleting outside pools directory fails", relativePath: ".", error: errInvalidPoolDir, @@ -199,20 +210,28 @@ func TestDelete(t *testing.T) { }, } { t.Run(tc.desc, func(t *testing.T) { - _, err := client.DeleteObjectPool(ctx, &gitalypb.DeleteObjectPoolRequest{ObjectPool: &gitalypb.ObjectPool{ + request := &gitalypb.DeleteObjectPoolRequest{ObjectPool: &gitalypb.ObjectPool{ Repository: &gitalypb.Repository{ StorageName: repo.GetStorageName(), RelativePath: tc.relativePath, }, - }}) + }} + + if tc.noPool { + request.ObjectPool = nil + } + expectedErr := tc.error if tc.error == errInvalidPoolDir && testhelper.IsPraefectEnabled() { expectedErr = helper.ErrNotFound(fmt.Errorf( "mutator call: route repository mutator: get repository id: %w", commonerr.NewRepositoryNotFoundError(repo.GetStorageName(), tc.relativePath), )) + } else if tc.error == errMissingPool && testhelper.IsPraefectEnabled() { + expectedErr = helper.ErrInvalidArgumentf("repo scoped: unable to descend OID [1 1] into message gitaly.DeleteObjectPoolRequest: proto field is empty") } + _, err := client.DeleteObjectPool(ctx, request) testhelper.RequireGrpcError(t, expectedErr, err) response, err := repositoryClient.RepositoryExists(ctx, &gitalypb.RepositoryExistsRequest{ |