Skip to content

Commit a6bd055

Browse files
committed
Fix formatting issues (gofmt/goimports)
- Remove extra blank lines in resolver_integration_test.go - Fix whitespace in conversion_test.go - Fix trailing space in start_time_test.go comment
1 parent a3078a6 commit a6bd055

File tree

3 files changed

+2
-26
lines changed

3 files changed

+2
-26
lines changed

pkg/remoteresolution/resolver/cluster/resolver_integration_test.go

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -581,7 +581,6 @@ func TestResolveWithInvalidParams(t *testing.T) {
581581
}
582582
}
583583

584-
585584
func TestAnnotatedResourceCreation(t *testing.T) {
586585
// Create a mock resolved resource using the correct type
587586
mockResource := &clusterresolution.ResolvedClusterResource{
@@ -631,8 +630,6 @@ func TestAnnotatedResourceCreation(t *testing.T) {
631630
}
632631
}
633632

634-
635-
636633
func TestResolveWithAutoModeAndChecksum(t *testing.T) {
637634
// Test auto mode with valid checksum
638635

@@ -664,24 +661,3 @@ func TestResolveWithAutoModeAndChecksum(t *testing.T) {
664661
t.Error("Auto mode should not cache when checksum is absent (cluster resolver behavior)")
665662
}
666663
}
667-
668-
669-
670-
671-
672-
673-
674-
675-
676-
677-
678-
679-
680-
681-
682-
683-
684-
685-
686-
687-

test/conversion_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -678,7 +678,7 @@ func TestCRDConversionStrategy(t *testing.T) {
678678
v1.Kind("pipelineruns"),
679679
resolutionv1beta1.Kind("resolutionrequests"),
680680
}
681-
681+
682682
// Wait for webhooks to be ready after controller startup with cache injection overhead
683683
t.Logf("Waiting for CRD webhook conversion to be ready...")
684684
for _, kind := range kinds {

test/start_time_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ spec:
8383
t.Errorf("Got unexpected number of step states: got %d, want %d", got, want)
8484
}
8585
// Account for additional system overhead from cache injection during startup
86-
// Original test expected >= 2s, but with cache initialization overhead,
86+
// Original test expected >= 2s, but with cache initialization overhead,
8787
// allow slightly more tolerance while still validating step timing works
8888
minimumDiff := 1800 * time.Millisecond // 1.8s instead of 2.0s
8989
var lastStart metav1.Time

0 commit comments

Comments
 (0)