fix(aws): guard GetIamPolicyDocumentContextE; remove stray rds test print

- iam.go: GetIamPolicyDocumentContextE previously dereferenced
  version.VersionId without a nil check and fell through with an empty
  defaultVersion when ListPolicyVersions returned no default, producing
  a confusing AWS validation error. Now guards the dereference and
  returns a clear error naming the policy ARN.
- rds_test.go: drop a leftover fmt.Println(err) in the error-path
  subtests; the existing assert.EqualError is sufficient.

Author: james00012

modules/aws/iam.go

@@ -120,11 +120,15 @@ func GetIamPolicyDocumentContextE(t testing.TestingT, ctx context.Context, regio
 	var defaultVersion string
 
 	for _, version := range versions.Versions {
-		if version.IsDefaultVersion {
+		if version.IsDefaultVersion && version.VersionId != nil {
 			defaultVersion = *version.VersionId
 		}
 	}
 
+	if defaultVersion == "" {
+		return "", fmt.Errorf("no default version found for IAM policy %s", policyARN)
+	}
+
 	document, err := iamClient.GetPolicyVersion(ctx, &iam.GetPolicyVersionInput{
 		PolicyArn: aws.String(policyARN),
 		VersionId: aws.String(defaultVersion),

modules/aws/rds_test.go

@@ -1,7 +1,6 @@
 package aws_test
 
 import (
-	"fmt"
 	"testing"
 
 	aws "github.com/gruntwork-io/terratest/modules/aws"
@@ -146,7 +145,6 @@ func TestGetRecommendedRdsInstanceTypeErrors(t *testing.T) {
 			t.Parallel()
 
 			_, err := aws.GetRecommendedRdsInstanceTypeE(t, scenerio.region, scenerio.databaseEngine, scenerio.databaseEngineVersion, scenerio.instanceTypes)
-			fmt.Println(err)
 			assert.EqualError(t, err, aws.NoRdsInstanceTypeError{InstanceTypeOptions: scenerio.instanceTypes, DatabaseEngine: scenerio.databaseEngine, DatabaseEngineVersion: scenerio.databaseEngineVersion}.Error())
 		})
 	}