Add import support for random id and random integer #20
Conversation
| ImportState: true, | ||
| ImportStateId: "3,1,3,12345", | ||
| ImportStateVerify: true, | ||
| }, |
There was a problem hiding this comment.
Ideally, we'd see a version of this that didn't set the Seed, just to exercise that code path, too.
There was a problem hiding this comment.
That's a difficult test to write: the import tests create a resource and then import it and check that the result matches. How would you go about guaranteeing that the resource will be created in that first step with a particular result without the seed?
There was a problem hiding this comment.
That's a fair question! Request rescinded.
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Allow importing random resources to be able to replace a config value with a value interpolated from the random provider without experiencing diffs.
See hashicorp/terraform-provider-google#1054 (which then unblocks hashicorp/terraform-provider-google#780) for more information on why this is a useful feature.
Once this is merged, I'll add import support for the remaining random resources for completeness.