feat: added additional error detail to the graphql schema creation. S…

…ee [16814] Signed-off-by: Alexander Chernov <[email protected]>
DrFaust92 · Dec 30, 2020 · 416cd08 · 416cd08
1 parent bebe732
commit 416cd08
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/aws/resource_aws_appsync_graphql_api.go b/aws/resource_aws_appsync_graphql_api.go
@@ -598,6 +598,9 @@ func resourceAwsAppsyncSchemaPut(d *schema.ResourceData, meta interface{}) error
 				if err != nil {
 					return 0, "", err
 				}
+				if *result.Status == "FAILED" {
+					return result, *result.Status, fmt.Errorf("%s", *result.Details)
+				}
 				return result, *result.Status, nil
 			},
 			Timeout: d.Timeout(schema.TimeoutCreate),

diff --git a/aws/resource_aws_appsync_graphql_api_test.go b/aws/resource_aws_appsync_graphql_api_test.go
@@ -166,6 +166,22 @@ func TestAccAWSAppsyncGraphqlApi_Schema(t *testing.T) {
 	})
 }
 
+func TestAccAWSAppsyncGraphqlApi_Schema_Error(t *testing.T) {
+	rName := acctest.RandomWithPrefix("tf-acc-test")
+
+	resource.ParallelTest(t, resource.TestCase{
+		PreCheck:     func() { testAccPreCheck(t); testAccPartitionHasServicePreCheck(appsync.EndpointsID, t) },
+		Providers:    testAccProviders,
+		CheckDestroy: testAccCheckAwsAppsyncGraphqlApiDestroy,
+		Steps: []resource.TestStep{
+			{
+				Config:      testAccAppsyncGraphqlApiConfig_Schema_Invalid(rName),
+				ExpectError: regexp.MustCompile("The field type 'PostV2' is not present when resolving type 'Query'"),
+			},
+		},
+	})
+}
+
 func TestAccAWSAppsyncGraphqlApi_AuthenticationType(t *testing.T) {
 	var api1, api2 appsync.GraphqlApi
 	rName := acctest.RandomWithPrefix("tf-acc-test")
@@ -1205,6 +1221,16 @@ resource "aws_appsync_graphql_api" "test" {
 `, rName)
 }
 
+func testAccAppsyncGraphqlApiConfig_Schema_Invalid(rName string) string {
+	return fmt.Sprintf(`
+resource "aws_appsync_graphql_api" "test" {
+  authentication_type = "API_KEY"
+  name                = %q
+  schema              = "type Mutation {\n\tputPost(id: ID!, title: String!): Post\n}\n\ntype Post {\n\tid: ID!\n\ttitle: String!\n}\n\ntype Query {\n\tsinglePost(id: ID!): PostV2\n}\n\nschema {\n\tquery: Query\n\tmutation: Mutation\n\n}\n"
+}
+`, rName)
+}
+
 func testAccAppsyncGraphqlApiConfig_SchemaUpdate(rName string) string {
 	return fmt.Sprintf(`
 resource "aws_appsync_graphql_api" "test" {