fix: exclude invalid fields in analyzer (#44)

* fix: exclude invalid fields in analyzer

* fix: lint

* fix: update test

* fix: lint

* fix: log warning on unsuccesful response

* fix: only log client-level errors

* fix: remove log

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    graphql-hive (0.5.1)
+    graphql-hive (0.5.2)
       graphql (>= 2.3, < 3)
 
 GEM

lib/graphql-hive/analyzer.rb

@@ -10,8 +10,11 @@ def initialize(query_or_multiplex)
       end
 
       def on_enter_field(node, _parent, visitor)
-        @used_fields.add(visitor.parent_type_definition.graphql_name)
-        @used_fields.add(make_id(visitor.parent_type_definition.graphql_name, node.name))
+        parent_type = visitor.parent_type_definition
+        if parent_type&.respond_to?(:graphql_name) && parent_type.fields[node.name]
+          @used_fields.add(parent_type.graphql_name)
+          @used_fields.add(make_id(parent_type.graphql_name, node.name))
+        end
       end
 
       # Visitor also calls 'on_enter_argument' when visiting input object fields in arguments

lib/graphql-hive/version.rb

@@ -2,6 +2,6 @@
 
 module Graphql
   module Hive
-    VERSION = "0.5.1"
+    VERSION = "0.5.2"
   end
 end

spec/graphql/graphql-hive/analyzer_spec.rb

@@ -349,4 +349,27 @@
       )
     end
   end
+
+  context "with an invalid field" do
+    let(:query_string) do
+      <<~GQL
+        query getGatewayProjects {
+          projectsByManyTypes(type: [STITCHING]),
+          nonExistentField {
+            subField
+          }
+        }
+      GQL
+    end
+
+    it "collects all valid fields and excludes invalid fields" do
+      expect(used_fields).to contain_exactly(
+        "Query",
+        "Query.projectsByManyTypes",
+        "ProjectType",
+        "Query.projectsByManyTypes.type",
+        "ProjectType.STITCHING"
+      )
+    end
+  end
 end