Merge pull request #2281 from justinsb/dont_try_update_if_not_updateu…

…nstruct

tests: skip update portions of the test if we don't have an update configured.

pkg/controller/dynamic/dynamic_controller_integration_test.go

@@ -355,6 +355,10 @@ func testUpdate(ctx context.Context, t *testing.T, testContext testrunner.TestCo
 	if resourceContext.SkipUpdate || resourceContext.IsAutoGenerated(systemContext.SMLoader) {
 		return
 	}
+	if testContext.UpdateUnstruct == nil {
+		t.Logf("UpdateUnstruct not set; skipping update")
+		return
+	}
 	kubeClient := systemContext.Manager.GetClient()
 	initialUnstruct := testContext.CreateUnstruct.DeepCopy()
 	if err := kubeClient.Get(ctx, testContext.NamespacedName, initialUnstruct); err != nil {
@@ -367,9 +371,6 @@ func testUpdate(ctx context.Context, t *testing.T, testContext testrunner.TestCo
 
 	// Update resource from test data
 	updateUnstruct := testContext.UpdateUnstruct.DeepCopy()
-	if updateUnstruct == nil {
-		t.Fatalf("updateUnstruct is nil for '%v'. should SkipUpdate be set to true in resourcefixture/contexts?", testContext.ResourceFixture.Name)
-	}
 	updateUnstruct.SetResourceVersion(initialUnstruct.GetResourceVersion())
 	// For resources with server-generated IDs, ensure the relevant fields are in the status
 	status := initialUnstruct.Object["status"]