New Check: V2 schema.ResourceImporter.State Deprecation

[bflad]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

The github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.ResourceImporter.State field is deprecated in preference of the context-aware StateContext. The new functionality is not available in version 1 of the Terraform Plugin SDK so this check should only flag version 2 usage of State.

New or Affected Checks(s)

• S###

Flagged Provider Code

&schema.ResourceImporter{
  // ... other configuration ...

  State: func(d *schema.ResourceData, meta interface{}) ([]*schema.ResourceData, error) { /* ... */},
}

Passing Provider Code

&schema.ResourceImporter{
  // ... other configuration ...

  StateContext: func(ctx context.Context, d *schema.ResourceData, meta interface{}) ([]*schema.ResourceData, error) { /* ... */},
}

References

• https://www.terraform.io/docs/extend/guides/v2-upgrade-guide.html#more-support-for-context-context
• Context aware SDK with timeouts hashicorp/terraform-plugin-sdk#276