From 334bc5a1fb844ec2adcdabc6c9ed24895dcdc420 Mon Sep 17 00:00:00 2001 From: subhamkrai Date: Tue, 10 Feb 2026 12:36:36 +0530 Subject: [PATCH] test: fix unit test code to remove extra comma this commit fix regression comming from another pr which was merged and since github was down ci were not running. Signed-off-by: subhamkrai (cherry picked from commit f98dd9f9f0e143431983b721e83272e65ad1437c) --- pkg/operator/ceph/nfs/controller_test.go | 6 +++--- pkg/operator/ceph/nvmeof/controller_test.go | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/operator/ceph/nfs/controller_test.go b/pkg/operator/ceph/nfs/controller_test.go index 03bdb3814..a836a4077 100644 --- a/pkg/operator/ceph/nfs/controller_test.go +++ b/pkg/operator/ceph/nfs/controller_test.go @@ -207,7 +207,7 @@ func TestCephNFSController(t *testing.T) { newReconcile := func(clusterCtx *clusterd.Context, cl client.WithWatch) *ReconcileCephNFS { // Create a ReconcileCephNFS object with the scheme and fake client. - return &ReconcileCephNFS{client: cl, scheme: testScheme, context: clusterCtx, opManagerContext: ctx, recorder: events.NewFakeRecorder(50),} + return &ReconcileCephNFS{client: cl, scheme: testScheme, context: clusterCtx, opManagerContext: ctx, recorder: events.NewFakeRecorder(50)} } // Mock request to simulate Reconcile() being called on an event for a @@ -626,7 +626,7 @@ func TestNFSKeyRotation(t *testing.T) { } cl := fake.NewClientBuilder().WithScheme(s).WithRuntimeObjects(object...).Build() - r := &ReconcileCephNFS{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50),} + r := &ReconcileCephNFS{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50)} req := reconcile.Request{ NamespacedName: types.NamespacedName{ @@ -666,7 +666,7 @@ func TestNFSKeyRotation(t *testing.T) { }) t.Run("subsequent reconcile - retain cephx status", func(t *testing.T) { - r := &ReconcileCephNFS{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50),} + r := &ReconcileCephNFS{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50)} updateDeploymentAndWait, _ = testopk8s.UpdateDeploymentAndWaitStub() _, err := r.Reconcile(ctx, req) assert.NoError(t, err) diff --git a/pkg/operator/ceph/nvmeof/controller_test.go b/pkg/operator/ceph/nvmeof/controller_test.go index fc21dcc0a..e398667bb 100644 --- a/pkg/operator/ceph/nvmeof/controller_test.go +++ b/pkg/operator/ceph/nvmeof/controller_test.go @@ -166,7 +166,7 @@ func TestCephNVMeOFGatewayController(t *testing.T) { } newReconcile := func(clusterCtx *clusterd.Context, cl client.WithWatch) *ReconcileCephNVMeOFGateway { - return &ReconcileCephNVMeOFGateway{client: cl, scheme: testScheme, context: clusterCtx, opManagerContext: ctx, recorder: events.NewFakeRecorder(50),} + return &ReconcileCephNVMeOFGateway{client: cl, scheme: testScheme, context: clusterCtx, opManagerContext: ctx, recorder: events.NewFakeRecorder(50)} } req := reconcile.Request{ @@ -514,7 +514,7 @@ func TestNVMeOFKeyRotation(t *testing.T) { } cl := fake.NewClientBuilder().WithScheme(s).WithRuntimeObjects(object...).Build() - r := &ReconcileCephNVMeOFGateway{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50),} + r := &ReconcileCephNVMeOFGateway{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50)} req := reconcile.Request{ NamespacedName: types.NamespacedName{ @@ -551,7 +551,7 @@ func TestNVMeOFKeyRotation(t *testing.T) { }) t.Run("subsequent reconcile - retain cephx status", func(t *testing.T) { - r := &ReconcileCephNVMeOFGateway{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50),} + r := &ReconcileCephNVMeOFGateway{client: cl, scheme: s, context: c, opManagerContext: ctx, recorder: events.NewFakeRecorder(50)} _, err := r.Reconcile(ctx, req) assert.NoError(t, err) nvmeofResult := cephv1.CephNVMeOFGateway{}