graphql-analyzer 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c4b7a368bf9b13b15f3deb07a30e5852388ee533
-  data.tar.gz: 41b2ac8e242bf559702490e2115d69b805297eff
+  metadata.gz: b0214e0468ba2d0b41740b1ba6eb83e545c9778c
+  data.tar.gz: 637e201e8b6c9461dcdf07df0c278dcb0ea117ae
 SHA512:
-  metadata.gz: cd93aab10263c3289d24baeedf477a0422e22c8f376f3043c4e0fbd9444667f7cd52ec6e3126e903eed410e0dcf7455fe6dc8be178e581e2d6a4b7ec4b040c78
-  data.tar.gz: ff215a5a7a08818b77dc0efcca5fc6dcb619a86f874c3eb2bf5ce69a1e6ee7319a5546a987e8b991c6ba3130d5617c430e35b0a8352de3bf8910dcf9d0b68590
+  metadata.gz: 776bfaa067aa81dcb4ce8eb2ffe627d20c54cae1fa1932244290d9235ff7613654f1aa72c26b0436f759da8c44d713a2a7b6abdaef0c4f2eee3fbe47613f13d7
+  data.tar.gz: 144c77105b00b0e0ebfda11964a4cc92432950e92a603736079a57cf5a8b3a85c28c2f65ce9081d327c5cc2ccc24659fb03e35984e8a3406cce57e4a1e5e1097

data/lib/graphql/analyzer.rb

@@ -1,6 +1,7 @@
 require "graphql"
 require "graphql/analyzer/version"
 require "graphql/analyzer/instrumentation/base"
+require "graphql/analyzer/instrumentation/active_record"
 require "graphql/analyzer/instrumentation/elastic_search"
 require "graphql/analyzer/instrumentation/mysql"
 require "graphql/analyzer/instrumentation/postgresql"
@@ -43,7 +44,10 @@ module GraphQL
 
     def instrument(type, field)
       instruments.reduce(field) do |field, instrumentation|
-        field.redefine { resolve(instrumentation.instrument(type, field)) }
+        field.redefine do
+          resolve(instrumentation.instrument(type, field))
+          lazy_resolve(instrumentation.instrument_lazy(type, field))
+        end
       end
     end
   end

data/lib/graphql/analyzer/instrumentation/active_record.rb

@@ -0,0 +1,43 @@
+module GraphQL
+  class Analyzer
+    module Instrumentation
+      class ActiveRecord < Base
+        private
+
+        def resolve_proc(type, field, method)
+          ->(obj, args, ctx) do
+            result = nil
+            queries = ::ActiveRecord::Base.collecting_queries_for_explain do
+              result = field.public_send(method).call(obj, args, ctx)
+              if result.respond_to?(:to_a)
+                result.to_a
+              end
+            end
+
+            if queries.any?
+              explain_output = ::ActiveRecord::Base.exec_explain(queries)
+              parsed_output = parser.parse(explain_output)
+
+              # TODO: Merge results when a field makes two types of queries
+              # e.g. path: ['user', 'name'] makes a SQL and ES Query.
+              ctx['graphql-analyzer']['resolvers'] << {
+                'path' => ctx.path,
+                'adapter' => adapter,
+                'parentType' => type.name,
+                'fieldName' => field.name,
+                'returnType' => field.type.to_s,
+                'details' => parsed_output
+              }
+            end
+
+            result
+          end
+        end
+
+        def parser
+          raise NotImplementedError, "Please override in #{self.class.name}"
+        end
+      end
+    end
+  end
+end

data/lib/graphql/analyzer/instrumentation/base.rb

@@ -3,40 +3,22 @@ module GraphQL
     module Instrumentation
       class Base
         def instrument(type, field)
-          ->(obj, args, ctx) do
-            result = nil
-            queries = ::ActiveRecord::Base.collecting_queries_for_explain do
-              result = field.resolve_proc.call(obj, args, ctx)
-              result.to_a if result.respond_to?(:to_a)
-            end
+          resolve_proc(type, field, :resolve_proc)
+        end
 
-            if queries.any?
-              explain_output = ::ActiveRecord::Base.exec_explain(queries)
-              parsed_output = parser.parse(explain_output)
+        def instrument_lazy(type, field)
+          resolve_proc(type, field, :lazy_resolve_proc)
+        end
 
-              # TODO: Merge results when a field makes two types of queries
-              # e.g. path: ['user', 'name'] makes a SQL and ES Query.
-              ctx['graphql-analyzer']['resolvers'] << {
-                'path' => ctx.path,
-                'adapter' => adapter,
-                'parentType' => type.name,
-                'fieldName' => field.name,
-                'returnType' => field.type.to_s,
-                'details' => parsed_output
-              }
-            end
+        private
 
-            result
-          end
+        def resolve_proc(type, field, method)
+          raise NotImplementedError, "Please override in #{self.class.name}"
         end
-      end
 
-      def parser
-        raise NotImplementedError, "Please override in #{self.class.name}"
-      end
-
-      def adapter
-        raise NotImplementedError, "Please override in #{self.class.name}"
+        def adapter
+          raise NotImplementedError, "Please override in #{self.class.name}"
+        end
       end
     end
   end

data/lib/graphql/analyzer/instrumentation/elastic_search.rb

@@ -1,7 +1,7 @@
 module GraphQL
   class Analyzer
     module Instrumentation
-      class ElasticSearch
+      class ElasticSearch < Base
         def initialize
           @notifications = []
           ActiveSupport::Notifications.subscribe('search.elasticsearch') do |name, start, finish, id, payload|
@@ -15,16 +15,18 @@ module GraphQL
           end
         end
 
-        def instrument(type, field)
+        private
+
+        def resolve_proc(type, field, method)
           ->(obj, args, ctx) do
-            result = field.resolve_proc.call(obj, args, ctx)
+            result = field.public_send(method).call(obj, args, ctx)
 
             if @notifications.any?
               # TODO: Merge results when a field makes two types of queries
               # e.g. path: ['user', 'name'] makes a SQL and ES Query.
               ctx['graphql-analyzer']['resolvers'] << {
                 'path' => ctx.path,
-                'adapter' => 'elasticsearch',
+                'adapter' => adapter,
                 'parentType' => type.name,
                 'fieldName' => field.name,
                 'returnType' => field.type.to_s,
@@ -37,6 +39,10 @@ module GraphQL
             result
           end
         end
+
+        def adapter
+          @adapter ||= 'elasticsearch'
+        end
       end
     end
   end

data/lib/graphql/analyzer/instrumentation/mysql.rb

@@ -1,7 +1,7 @@
 module GraphQL
   class Analyzer
     module Instrumentation
-      class Mysql < Base
+      class Mysql < ActiveRecord
         private
 
         def parser

data/lib/graphql/analyzer/instrumentation/postgresql.rb

@@ -1,7 +1,7 @@
 module GraphQL
   class Analyzer
     module Instrumentation
-      class Postgresql < Base
+      class Postgresql < ActiveRecord
         private
 
         def parser

data/lib/graphql/analyzer/instrumentation/sqlite3.rb

@@ -1,7 +1,7 @@
 module GraphQL
   class Analyzer
     module Instrumentation
-      class Sqlite3 < Base
+      class Sqlite3 < ActiveRecord
         private
 
         def parser

data/lib/graphql/analyzer/version.rb

@@ -1,5 +1,5 @@
 module GraphQL
   class Analyzer
-    VERSION = "0.1.3"
+    VERSION = "0.1.4"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: graphql-analyzer
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - DerekStride
@@ -199,6 +199,7 @@ files:
 - Rakefile
 - graphql-analyzer.gemspec
 - lib/graphql/analyzer.rb
+- lib/graphql/analyzer/instrumentation/active_record.rb
 - lib/graphql/analyzer/instrumentation/base.rb
 - lib/graphql/analyzer/instrumentation/elastic_search.rb
 - lib/graphql/analyzer/instrumentation/mysql.rb