introspection: expose fields/values with empty deprecation_reason

Replicates graphql/graphql-js@dd95ec2

Author: Cito

src/graphql/type/introspection.py

@@ -276,7 +276,7 @@ def fields(type_, _info, includeDeprecated=False):
         if is_object_type(type_) or is_interface_type(type_):
             items = type_.fields.items()
             if not includeDeprecated:
-                return [item for item in items if not item[1].deprecation_reason]
+                return [item for item in items if not item[1].is_deprecated]
             return list(items)
 
     @staticmethod
@@ -295,7 +295,7 @@ def enum_values(type_, _info, includeDeprecated=False):
         if is_enum_type(type_):
             items = type_.values.items()
             if not includeDeprecated:
-                return [item for item in items if not item[1].deprecation_reason]
+                return [item for item in items if not item[1].is_deprecated]
             return items
 
     @staticmethod

tests/type/test_introspection.py

@@ -975,6 +975,9 @@ def identifies_deprecated_fields():
                 "deprecated": GraphQLField(
                     GraphQLString, deprecation_reason="Removed in 1.0"
                 ),
+                "deprecatedWithEmptyReason": GraphQLField(
+                    GraphQLString, deprecation_reason=""
+                ),
             },
         )
 
@@ -1007,6 +1010,11 @@ def identifies_deprecated_fields():
                             "isDeprecated": True,
                             "deprecationReason": "Removed in 1.0",
                         },
+                        {
+                            "name": "deprecatedWithEmptyReason",
+                            "isDeprecated": True,
+                            "deprecationReason": "",
+                        },
                     ],
                 }
             },
@@ -1112,7 +1120,10 @@ def respects_the_include_deprecated_parameter_for_enum_values():
             {
                 "NON_DEPRECATED": GraphQLEnumValue(0),
                 "DEPRECATED": GraphQLEnumValue(1, deprecation_reason="Removed in 1.0"),
-                "ALSO_NON_DEPRECATED": GraphQLEnumValue(2),
+                "DEPRECATED_WITH_EMPTY_REASON": GraphQLEnumValue(
+                    2, deprecation_reason=""
+                ),
+                "ALSO_NON_DEPRECATED": GraphQLEnumValue(3),
             },
         )
 
@@ -1122,7 +1133,6 @@ def respects_the_include_deprecated_parameter_for_enum_values():
         source = """
             {
               __type(name: "TestEnum") {
-                  name
                 trueValues: enumValues(includeDeprecated: true) {
                   name
                 }
@@ -1139,10 +1149,10 @@ def respects_the_include_deprecated_parameter_for_enum_values():
         assert graphql_sync(schema=schema, source=source) == (
             {
                 "__type": {
-                    "name": "TestEnum",
                     "trueValues": [
                         {"name": "NON_DEPRECATED"},
                         {"name": "DEPRECATED"},
+                        {"name": "DEPRECATED_WITH_EMPTY_REASON"},
                         {"name": "ALSO_NON_DEPRECATED"},
                     ],
                     "falseValues": [