fix tests

Signed-off-by: Remington Breeze <remington@breeze.software>

internal/controller/promotion/argocd.go

@@ -351,14 +351,17 @@ func (a *argoCDMechanism) updateApplicationSources(
 	app.Spec.Source = desiredSource.DeepCopy()
 	app.Spec.Sources = desiredSources.DeepCopy()
 
-	// retrieve AppProject object
-	project, err := argocd.GetAppProject(ctx, a.argocdClient, app.Spec.Project)
-	if err != nil {
-		return fmt.Errorf("error finding Argo CD AppProject %q in namespace %q: %w", app.Spec.Project, app.Namespace, err)
-	}
-
-	if !project.Spec.SyncWindows.Matches(app).CanSync(true) {
-		return fmt.Errorf("cannot sync: blocked by sync window")
+	if app.Spec.Project != "" {
+		// retrieve AppProject object
+		project, err := argocd.GetAppProject(ctx, a.argocdClient, app.Spec.Project)
+		if err != nil || project == nil {
+			return fmt.Errorf("error finding Argo CD AppProject %s: %w", app.Spec.Project, err)
+		}
+		if project.Spec.SyncWindows != nil {
+			if !project.Spec.SyncWindows.Matches(app).CanSync(true) {
+				return fmt.Errorf("cannot sync: blocked by sync window")
+			}
+		}
 	}
 
 	// Initiate a new operation.

internal/controller/promotion/argocd_test.go

@@ -1179,6 +1179,17 @@ func TestArgoCDMustPerformUpdate(t *testing.T) {
 }
 
 func TestArgoCDUpdateApplicationSources(t *testing.T) {
+	scheme := runtime.NewScheme()
+	c := fake.NewClientBuilder().WithScheme(scheme)
+	err := argocd.AddToScheme(scheme)
+	require.NoError(t, err)
+	proj := &argocd.AppProject{
+		ObjectMeta: metav1.ObjectMeta{
+			Name: "fake-project",
+		},
+	}
+	c.WithObjects(proj)
+
 	testCases := []struct {
 		name           string
 		promoMech      *argoCDMechanism
@@ -1241,6 +1252,7 @@ func TestArgoCDUpdateApplicationSources(t *testing.T) {
 		},
 	}
 	for _, testCase := range testCases {
+		testCase.promoMech.argocdClient = c.Build()
 		t.Run(testCase.name, func(t *testing.T) {
 			testCase.assertions(
 				t,