actual fix

austinvalle · austinvalle · commit 00b76f4d309c · 2025-05-07T17:37:53.000-04:00
diff --git a/helper/resource/testing_new_import_state.go b/helper/resource/testing_new_import_state.go
@@ -458,8 +458,12 @@ func appendImportBlockWithIdentity(config string, resourceName string, identityV
 
 		case string:
 			configBuilder += fmt.Sprintf(`		%q = %q`+"\n", k, v)
-
 		default:
+			// It's valid for identity attributes to be null, we can just omit it from config
+			if v == nil {
+				break
+			}
+
 			panic(fmt.Sprintf("unexpected type %T for identity value %q", v, k))
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -458,8 +458,12 @@ func appendImportBlockWithIdentity(config string, resourceName string, identityV`
`458`	`458`
`459`	`459`	`case string:`
`460`	`460`	configBuilder += fmt.Sprintf(` %q = %q`+"\n", k, v)
`461`		`-`
`462`	`461`	`default:`
	`462`	`+ // It's valid for identity attributes to be null, we can just omit it from config`
	`463`	`+ if v == nil {`
	`464`	`+ break`
	`465`	`+ }`
	`466`	`+`
`463`	`467`	`panic(fmt.Sprintf("unexpected type %T for identity value %q", v, k))`
`464`	`468`	`}`
`465`	`469`	`}`