Fix bug involving cascading suppression error (#992)

* Fix bug involving cascading suppression error

* Add test

Author: LWprogramming

graphql_compiler/schema_transformation/rename_schema.py

@@ -1116,6 +1116,9 @@ def enter_field_definition(
             # Therefore, we don't need to explicitly suppress the field as well and this should not
             # raise errors.
             return IDLE
+        if self.field_renamings.get(self.current_type, {}).get(field_name, {field_name}) == set():
+            # Field was also suppressed so this should not raise errors.
+            return IDLE
         if self.current_type not in self.fields_to_suppress:
             self.fields_to_suppress[self.current_type] = {}
         self.fields_to_suppress[self.current_type][field_name] = field_type

graphql_compiler/tests/schema_transformation_tests/test_rename_schema.py

@@ -1140,6 +1140,45 @@ def test_field_still_depends_on_suppressed_type(self) -> None:
                 parse(ISS.multiple_fields_schema), {"Dog": None}, {}
             )  # The type named Human contains a field of type Dog.
 
+    def test_suppress_field_prevent_cascading_suppression_error(self) -> None:
+        renamed_schema = rename_schema(
+            parse(ISS.multiple_fields_schema), {"Dog": None}, {"Human": {"pet": set()}}
+        )
+        renamed_schema_string = dedent(
+            """\
+            schema {
+              query: SchemaQuery
+            }
+
+            directive @output(
+                \"\"\"What to designate the output field generated from this property field.\"\"\"
+                out_name: String!
+            ) on FIELD
+
+            type Human  {
+              id: String
+              name: String
+              age: Int
+              droid: Droid
+            }
+
+            type Droid {
+              id: String
+              model: String
+            }
+
+            type SchemaQuery {
+              Human: Human
+              Droid: Droid
+            }
+        """
+        )
+        compare_schema_texts_order_independently(
+            self, renamed_schema_string, print_ast(renamed_schema.schema_ast)
+        )
+        self.assertEqual({}, renamed_schema.reverse_name_map)
+        self.assertEqual({}, renamed_schema.reverse_field_name_map)
+
     def test_field_of_suppressed_type_in_suppressed_type(self) -> None:
         # The schema contains an object type that contains a field of the type Human. Normally,
         # suppressing the type named Human would cause a CascadingSuppressionError because the