Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The acceptance test for devtest policy did not flag the issue fixed in #9077 #9120

Open
mattcowen opened this issue Nov 2, 2020 · 0 comments

Comments

@mattcowen
Copy link

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 "me too" comments, 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

Terraform (and AzureRM Provider) Version

v0.13.5 and v2.34

Affected Resource(s)

azurerm_dev_test_policy

Expected Behavior

The acceptance test should have flagged the delete problem fixed in #9077 (for v2.33). I expected the testCheckDevTestPolicyDestroy in /terraform-provider-azurerm/azurerm/internal/services/resource/tests/resource_arm_resource_group_test.go to have discovered the policy still existed after delete but it doesn't seem possible.

I also think that if the policy isn't found when trying to delete then it should error since if this scenario happens it causes subsequent problems with synchronicity of the state with Azure i.e. Terraform thinks there is no policy when in fact there is.

Actual Behavior

The acceptance test passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants