Skip to content

Commit 109f732

Browse files
committed
Fix lint errors
1 parent 95a5c50 commit 109f732

File tree

2 files changed

+16
-10
lines changed

2 files changed

+16
-10
lines changed

pkg/resource-handler/controller/shard/pool_service_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,4 +309,3 @@ func TestBuildPoolHeadlessService(t *testing.T) {
309309
})
310310
}
311311
}
312-

pkg/resource-handler/controller/shard/shard_controller_internal_test.go

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ func TestBuildMultiOrchContainer_WithImage(t *testing.T) {
122122
// Test exists for coverage of defensive error handling.
123123
func TestReconcile_InvalidScheme(t *testing.T) {
124124
tests := map[string]struct {
125-
setupShard func() *multigresv1alpha1.Shard
125+
setupShard func() *multigresv1alpha1.Shard
126126
reconcileFunc func(*ShardReconciler, context.Context, *multigresv1alpha1.Shard) error
127127
}{
128128
"MultiOrchDeployment": {
@@ -287,9 +287,9 @@ func TestHandleDeletion_NoFinalizer(t *testing.T) {
287287
// TestReconcile_GetError tests error path on Get operations (not NotFound, but network errors).
288288
func TestReconcile_GetError(t *testing.T) {
289289
tests := map[string]struct {
290-
setupShard func() *multigresv1alpha1.Shard
291-
failOnKeyName string
292-
reconcileFunc func(*ShardReconciler, context.Context, *multigresv1alpha1.Shard) error
290+
setupShard func() *multigresv1alpha1.Shard
291+
failOnKeyName string
292+
reconcileFunc func(*ShardReconciler, context.Context, *multigresv1alpha1.Shard) error
293293
}{
294294
"MultiOrchDeployment": {
295295
setupShard: func() *multigresv1alpha1.Shard {
@@ -395,10 +395,10 @@ func TestReconcile_GetError(t *testing.T) {
395395
// TestUpdateStatus_MultiOrch tests updateStatus with different MultiOrch deployment scenarios.
396396
func TestUpdateStatus_MultiOrch(t *testing.T) {
397397
tests := map[string]struct {
398-
setupObjects []client.Object
399-
expectError bool
398+
setupObjects []client.Object
399+
expectError bool
400400
expectOrchReady bool
401-
setupClient func(*testing.T, *runtime.Scheme, *multigresv1alpha1.Shard) client.Client
401+
setupClient func(*testing.T, *runtime.Scheme, *multigresv1alpha1.Shard) client.Client
402402
}{
403403
"GetError": {
404404
expectError: true,
@@ -409,7 +409,10 @@ func TestUpdateStatus_MultiOrch(t *testing.T) {
409409
WithStatusSubresource(&multigresv1alpha1.Shard{}).
410410
Build()
411411
return testutil.NewFakeClientWithFailures(baseClient, &testutil.FailureConfig{
412-
OnGet: testutil.FailOnKeyName("test-shard-multiorch-zone1", testutil.ErrNetworkTimeout),
412+
OnGet: testutil.FailOnKeyName(
413+
"test-shard-multiorch-zone1",
414+
testutil.ErrNetworkTimeout,
415+
),
413416
})
414417
},
415418
},
@@ -500,7 +503,11 @@ func TestUpdateStatus_MultiOrch(t *testing.T) {
500503
}
501504

502505
if updatedShard.Status.OrchReady != tc.expectOrchReady {
503-
t.Errorf("OrchReady = %v, want %v", updatedShard.Status.OrchReady, tc.expectOrchReady)
506+
t.Errorf(
507+
"OrchReady = %v, want %v",
508+
updatedShard.Status.OrchReady,
509+
tc.expectOrchReady,
510+
)
504511
}
505512
}
506513
})

0 commit comments

Comments
 (0)