artifactory_certificate testcase is wrongly configured - Githubissues

atlassian / terraform-provider-artifactory

Terraform provider to manage Artifactory

Apache License 2.0

89 stars 42 forks source link

artifactory_certificate testcase is wrongly configured #64

Closed volkc-basf closed 4 years ago

volkc-basf commented 4 years ago

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 Version

all

Affected Resource(s)

artifactory_certificate

Expected Behavior

CheckDestroy should execute testAccCheckCertificateDestroy

Actual Behavior

It acutally refers to testAccCheckUserDestroy

bodgit commented 4 years ago

Good catch, I must have started with the user tests as the basis for the certificate ones, and it manages to pass as it tries to find a user with the certificate ID so it assumes it was deleted.

I'll create a small PR to fix this.