add the list certificates error test back

ref #725

Signed-off-by: speruri <[email protected]>

Author: speruri

aws/acm_test.go

@@ -1,6 +1,7 @@
 package aws
 
 import (
+	"fmt"
 	"testing"
 
 	"github.com/aws/aws-sdk-go-v2/aws"
@@ -37,7 +38,7 @@ func TestACM(t *testing.T) {
 						CertificateChain: aws.String(chain),
 					},
 				},
-				nil,
+				nil, nil,
 			),
 			expect: acmExpect{
 				ARN:         "foobar",
@@ -52,7 +53,7 @@ func TestACM(t *testing.T) {
 						Certificate: aws.String(cert),
 					},
 				},
-				nil,
+				nil, nil,
 			),
 			expect: acmExpect{
 				ARN:         "foobar",
@@ -78,6 +79,7 @@ func TestACM(t *testing.T) {
 						Tags: []types.Tag{{Key: aws.String("production"), Value: aws.String("false")}},
 					},
 				},
+				nil,
 			),
 			filterTag: "production=true",
 			expect: acmExpect{
@@ -98,6 +100,7 @@ func TestACM(t *testing.T) {
 						Tags: []types.Tag{{Key: aws.String("production"), Value: aws.String("false")}},
 					},
 				},
+				nil,
 			),
 			filterTag: "production=true",
 			expect: acmExpect{
@@ -106,6 +109,19 @@ func TestACM(t *testing.T) {
 				DomainNames: []string{"foobar.de"},
 			},
 		},
+		{
+			msg: "Fail on ListCertificates error",
+			api: fake.NewACMClient(
+				nil, nil,
+				map[string]error{
+					"ListCertificates": fmt.Errorf("ListCertificates error"),
+				},
+			),
+			filterTag: "production=true",
+			expect: acmExpect{
+				Error: "failed to list certificates page: ListCertificates error",
+			},
+		},
 	} {
 		t.Run(ti.msg, func(t *testing.T) {
 			provider := newACMCertProvider(ti.api, ti.filterTag)

aws/fake/acm.go

@@ -11,9 +11,15 @@ import (
 type ACMClient struct {
 	cert map[string]*acm.GetCertificateOutput
 	tags map[string]*acm.ListTagsForCertificateOutput
+	err  map[string]error
 }
 
 func (m *ACMClient) ListCertificates(ctx context.Context, input *acm.ListCertificatesInput, fn ...func(*acm.Options)) (*acm.ListCertificatesOutput, error) {
+	if m.err != nil {
+		if err, ok := m.err["ListCertificates"]; ok {
+			return nil, err
+		}
+	}
 	output := &acm.ListCertificatesOutput{
 		CertificateSummaryList: make([]types.CertificateSummary, 0),
 	}
@@ -26,6 +32,11 @@ func (m *ACMClient) ListCertificates(ctx context.Context, input *acm.ListCertifi
 }
 
 func (m *ACMClient) GetCertificate(ctx context.Context, input *acm.GetCertificateInput, fn ...func(*acm.Options)) (*acm.GetCertificateOutput, error) {
+	if m.err != nil {
+		if err, ok := m.err["GetCertificate"]; ok {
+			return nil, err
+		}
+	}
 	if input.CertificateArn == nil {
 		return nil, fmt.Errorf("expected a valid CertificateArn, got: nil")
 	}
@@ -38,6 +49,11 @@ func (m *ACMClient) GetCertificate(ctx context.Context, input *acm.GetCertificat
 }
 
 func (m *ACMClient) ListTagsForCertificate(ctx context.Context, in *acm.ListTagsForCertificateInput, fn ...func(*acm.Options)) (*acm.ListTagsForCertificateOutput, error) {
+	if m.err != nil {
+		if err, ok := m.err["ListTagsForCertificate"]; ok {
+			return nil, err
+		}
+	}
 	if in.CertificateArn == nil {
 		return nil, fmt.Errorf("expected a valid CertificateArn, got: nil")
 	}
@@ -48,10 +64,12 @@ func (m *ACMClient) ListTagsForCertificate(ctx context.Context, in *acm.ListTags
 func NewACMClient(
 	cert map[string]*acm.GetCertificateOutput,
 	tags map[string]*acm.ListTagsForCertificateOutput,
+	err map[string]error,
 ) *ACMClient {
 	c := &ACMClient{
 		cert: cert,
 		tags: tags,
+		err:  err,
 	}
 	return c
 }