make rabbit happy

Author: MatousJobanek

pkg/owners/fetcher.go

@@ -87,7 +87,7 @@ func (o *OwnerFetcher) GetOwners(ctx context.Context, obj metav1.Object) ([]*Obj
 	}
 	// Recursively try to find the top owner
 	ownerOwners, err := o.GetOwners(ctx, ownerObject)
-	if err != nil || owners == nil {
+	if err != nil || ownerOwners == nil {
 		return append(ownerOwners, owner), err
 	}
 	return append(ownerOwners, owner), nil

pkg/owners/fetcher_test.go

@@ -69,7 +69,7 @@ func TestGetOwners(t *testing.T) {
 				noiseObjects = append(noiseObjects, noise)
 
 				// switch the type of the ownerReference (controller owner, non-controller owner) every second object to test both options properly
-				if i/2 == 0 {
+				if i%2 == 0 {
 					err := controllerruntime.SetControllerReference(owner, initObjects[len(initObjects)-1].(client.Object), scheme.Scheme)
 					require.NoError(t, err)
 				} else {