Skip to content

Commit b808612

Browse files
committed
Remove skipped tests
Signed-off-by: Trevor Rosen <trevrosen@github.com>
1 parent b6013cf commit b808612

1 file changed

Lines changed: 0 additions & 16 deletions

File tree

pkg/cmd/attestation/verification/sigstore_test.go

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,6 @@ func TestChooseVerifierWithNilPublicGood(t *testing.T) {
2727
require.ErrorContains(t, err, "public good verifier is not available")
2828
}
2929

30-
// TestChooseVerifierWithGitHubIssuer tests that chooseVerifier can select
31-
// GitHub verifier even when PGI verifier is nil.
32-
func TestChooseVerifierWithGitHubIssuer(t *testing.T) {
33-
// We'll test this scenario with the actual initialization
34-
// to ensure GitHub verifier is properly created
35-
t.Skip("This requires integration test with actual TUF client - covered by integration tests")
36-
}
37-
3830
// TestChooseVerifierUnrecognizedIssuer tests that an error is returned
3931
// for unrecognized issuers.
4032
func TestChooseVerifierUnrecognizedIssuer(t *testing.T) {
@@ -49,14 +41,6 @@ func TestChooseVerifierUnrecognizedIssuer(t *testing.T) {
4941
require.ErrorContains(t, err, "leaf certificate issuer is not recognized")
5042
}
5143

52-
// TestGetBundleIssuer tests the getBundleIssuer helper function
53-
func TestGetBundleIssuer(t *testing.T) {
54-
// This test would require setting up a mock bundle
55-
// For now, we'll just verify it exists and can be called
56-
// Integration tests cover the actual functionality
57-
t.Skip("getBundleIssuer requires a valid bundle which needs integration test setup")
58-
}
59-
6044
func TestLiveSigstoreVerifier_noVerifierSet(t *testing.T) {
6145
verifier := &LiveSigstoreVerifier{
6246
Logger: io.NewTestHandler(),

0 commit comments

Comments
 (0)