Remove test that can panic

Author: williammartin

pkg/github/repository_resource.go

@@ -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")
 	}
 }

pkg/github/repository_resource_test.go

@@ -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(