Remove test that can panic
This commit is contained in:
@@ -201,6 +201,7 @@ func RepositoryResourceContentsHandler(getClient GetClientFn) func(ctx context.C
|
||||
}
|
||||
}
|
||||
|
||||
// This should be unreachable because GetContents should return an error if neither file nor directory content is found.
|
||||
return nil, errors.New("no repository resource content found")
|
||||
}
|
||||
}
|
||||
|
||||
@@ -180,21 +180,6 @@ func Test_repositoryResourceContentsHandler(t *testing.T) {
|
||||
},
|
||||
expectedResult: expectedDirContent,
|
||||
},
|
||||
{
|
||||
name: "no data",
|
||||
mockedClient: mock.NewMockedHTTPClient(
|
||||
mock.WithRequestMatch(
|
||||
mock.GetReposContentsByOwnerByRepoByPath,
|
||||
),
|
||||
),
|
||||
requestArgs: map[string]any{
|
||||
"owner": []string{"owner"},
|
||||
"repo": []string{"repo"},
|
||||
"path": []string{"src"},
|
||||
},
|
||||
expectedResult: nil,
|
||||
expectError: "no repository resource content found",
|
||||
},
|
||||
{
|
||||
name: "empty data",
|
||||
mockedClient: mock.NewMockedHTTPClient(
|
||||
|
||||
Reference in New Issue
Block a user