Skip to content

Commit

Permalink
fix: make sure tests are not failing on cloudtop
Browse files Browse the repository at this point in the history
Signed-off-by: bwplotka <bwplotka@google.com>
  • Loading branch information
bwplotka committed Jun 12, 2024
1 parent 054f28c commit a1ed6ac
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions pkg/export/setup/setup_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ import (
func TestFromFlags_NotOnGCE(t *testing.T) {
// Asserting there is actually no GCE underneath.
if metadata.OnGCE() {
t.Fatal("This test assumes we don't run on GCP")
t.Skip("This test can't run on GCP or Cloudtop; we expect no metadata server.")
}

fake := kingpin.New("test", "test")
Expand All @@ -53,11 +53,6 @@ func TestFromFlags_NotOnGCE(t *testing.T) {
// Regression test for b/344740239. We ensure that even stuck metadata servers
// calls will timeout correctly (we propagate context properly).
func TestTryPopulateUnspecifiedFromMetadata(t *testing.T) {
// Asserting there is actually no GCE underneath.
if metadata.OnGCE() {
t.Fatal("This test assumes we don't run on GCP")
}

var wg sync.WaitGroup
wg.Add(1)

Expand Down

0 comments on commit a1ed6ac

Please sign in to comment.