Skip to content

Commit

Permalink
Add context to test functions
Browse files Browse the repository at this point in the history
  • Loading branch information
StupidScience committed Mar 8, 2023
1 parent d079283 commit 1fa17da
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions internal/service/acmpca/certificate_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ func TestAccACMPCACertificate_rootCertificate(t *testing.T) {
}

func TestAccACMPCACertificate_rootCertificateWithApiPassthrough(t *testing.T) {
ctx := acctest.Context(t)
resourceName := "aws_acmpca_certificate.test"
certificateAuthorityResourceName := "aws_acmpca_certificate_authority.test"

Expand All @@ -74,12 +75,12 @@ func TestAccACMPCACertificate_rootCertificateWithApiPassthrough(t *testing.T) {
PreCheck: func() { acctest.PreCheck(t) },
ErrorCheck: acctest.ErrorCheck(t, acmpca.EndpointsID),
ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories,
CheckDestroy: testAccCheckCertificateDestroy,
CheckDestroy: testAccCheckCertificateDestroy(ctx),
Steps: []resource.TestStep{
{
Config: testAccCertificateConfig_rootWithAPIPassthrough(domain),
Check: resource.ComposeAggregateTestCheckFunc(
testAccCheckCertificateExists(resourceName),
testAccCheckCertificateExists(ctx, resourceName),
testAccCheckCertificateExtension(resourceName),
acctest.MatchResourceAttrRegionalARN(resourceName, "arn", "acm-pca", regexp.MustCompile(`certificate-authority/.+/certificate/.+$`)),
resource.TestCheckResourceAttrSet(resourceName, "certificate"),
Expand Down

0 comments on commit 1fa17da

Please sign in to comment.