fix(e2e): slightly improves execution time

Author: Jeff Peeler

pkg/controller/operators/olm/operatorgroup.go

@@ -21,6 +21,7 @@ func (a *Operator) syncOperatorGroups(obj interface{}) error {
 		log.Debugf("wrong type: %#v\n", obj)
 		return fmt.Errorf("casting OperatorGroup failed")
 	}
+	log.Infof("syncing operator group %v", op)
 
 	err, targetedNamespaces := a.updateNamespaceList(op)
 	log.Debugf("Got targetedNamespaces: '%v'", targetedNamespaces)

test/e2e/operator_groups_e2e_test.go

@@ -116,18 +116,23 @@ func TestOperatorGroup(t *testing.T) {
 			Name:      "e2e-operator-group",
 			Namespace: testNamespace,
 		},
-		Spec: v1alpha2.OperatorGroupSpec{
-			Selector: metav1.LabelSelector{
-				MatchLabels: matchingLabel,
-			},
-		},
 	}
 	_, err = crc.OperatorsV1alpha2().OperatorGroups(testNamespace).Create(&operatorGroup)
 	require.NoError(t, err)
 	expectedOperatorGroupStatus := v1alpha2.OperatorGroupStatus{
 		Namespaces: []*corev1.Namespace{createdOtherNamespace},
 	}
 
+	// instead of setting the label selector initially, do it here to force immediate reconcile
+	fetchedOpGroup, err := crc.OperatorsV1alpha2().OperatorGroups(testNamespace).Get(operatorGroup.GetName(), metav1.GetOptions{})
+	fetchedOpGroup.Spec = v1alpha2.OperatorGroupSpec{
+		Selector: metav1.LabelSelector{
+			MatchLabels: matchingLabel,
+		},
+	}
+	_, err = crc.OperatorsV1alpha2().OperatorGroups(testNamespace).Update(fetchedOpGroup)
+	require.NoError(t, err)
+
 	log.Debug("Waiting on operator group to have correct status")
 	err = wait.Poll(pollInterval, pollDuration, func() (bool, error) {
 		fetched, fetchErr := crc.OperatorsV1alpha2().OperatorGroups(testNamespace).Get(operatorGroup.Name, metav1.GetOptions{})