Add PlanCheck to update test creation

[trodge]

This additional step will ensure that new update tests always update the resource in-place rather than delete and recreate.

Release Note Template for Downstream PRs (will be copied)

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

[NickElliot]

I'm seeing a lot of the current implementations in other tests are as follows:

ConfigPlanChecks: resource.ConfigPlanChecks{
					PreApply: []plancheck.PlanCheck{
						acctest.ExpectNoDelete(),
					},
				},

Should this be suggested or mentioned as an option for update-steps that update more than one resource at once?

Also is there plans to update most other existing tests to use this check, or will we just be recommending it going forward?

[trodge]

I'm seeing a lot of the current implementations in other tests are as follows:

ConfigPlanChecks: resource.ConfigPlanChecks{
					PreApply: []plancheck.PlanCheck{
						acctest.ExpectNoDelete(),
					},
				},

Should this be suggested or mentioned as an option for update-steps that update more than one resource at once?

Also is there plans to update most other existing tests to use this check, or will we just be recommending it going forward?

The plan is to eventually update existing tests to use ExpectResourceAction, and this is preferred over ExpectNoDelete