Skip to content

Commit 6ae2ced

Browse files
committed
Fix: Fix Lint issues
1 parent 06bc1b7 commit 6ae2ced

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

internal/controller/configmap_changes_test.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,6 +202,7 @@ func TestConfigMapChangesWithNonMatchingSelector(t *testing.T) {
202202
if err := env.Get(ctx, client.ObjectKeyFromObject(coreProvider), coreProvider); err != nil {
203203
return false
204204
}
205+
205206
return conditions.IsTrue(coreProvider, operatorv1.ProviderInstalledCondition)
206207
}, timeout).Should(BeTrue())
207208

@@ -286,6 +287,7 @@ func TestConfigMapChangesWithNonMatchingSelector(t *testing.T) {
286287
return false
287288
}
288289
hash := provider.GetAnnotations()[appliedSpecHashAnnotation]
290+
289291
return hash != ""
290292
}, timeout).Should(BeTrue(), "Provider should have a hash annotation after reconciliation")
291293

@@ -308,6 +310,7 @@ func TestConfigMapChangesWithNonMatchingSelector(t *testing.T) {
308310
}
309311

310312
currentHash := provider.GetAnnotations()[appliedSpecHashAnnotation]
313+
311314
return currentHash == initialHash
312315
}, 10*time.Second, 2*time.Second).Should(BeTrue())
313316

@@ -327,6 +330,7 @@ func TestConfigMapChangesWithNonMatchingSelector(t *testing.T) {
327330
}
328331

329332
currentHash := provider.GetAnnotations()[appliedSpecHashAnnotation]
333+
330334
return currentHash != "" && currentHash != initialHash
331335
}, 30*time.Second).Should(BeTrue())
332336

@@ -367,6 +371,7 @@ func TestMultipleConfigMapsError(t *testing.T) {
367371
if err := env.Get(ctx, client.ObjectKeyFromObject(coreProvider), coreProvider); err != nil {
368372
return false
369373
}
374+
370375
return conditions.IsTrue(coreProvider, operatorv1.ProviderInstalledCondition)
371376
}, timeout).Should(BeTrue())
372377

@@ -442,6 +447,7 @@ func TestMultipleConfigMapsError(t *testing.T) {
442447
if err := env.Get(ctx, client.ObjectKeyFromObject(provider), provider); err != nil {
443448
return false
444449
}
450+
445451
return conditions.IsFalse(provider, operatorv1.ProviderInstalledCondition) &&
446452
conditions.GetReason(provider, operatorv1.ProviderInstalledCondition) != ""
447453
}, timeout).Should(BeTrue())

internal/controller/genericprovider_controller.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -327,6 +327,7 @@ func processProviderConfigMaps(ctx context.Context, k8sClient client.Client, has
327327
if err := addObjectToHash(hash, cm.Data); err != nil {
328328
return err
329329
}
330+
330331
if err := addObjectToHash(hash, cm.BinaryData); err != nil {
331332
return err
332333
}

0 commit comments

Comments
 (0)