Fix accepts_definition on interface parent

Author: rmosolgo

lib/graphql/schema/interface.rb

@@ -73,9 +73,10 @@ def own_interfaces
         end
       end
 
-      extend DefinitionMethods
       # Extend this _after_ `DefinitionMethods` is defined, so it will be used
       extend GraphQL::Schema::Member::AcceptsDefinition
+
+      extend DefinitionMethods
     end
   end
 end

spec/graphql/schema/member/accepts_definition_spec.rb

@@ -14,11 +14,15 @@ class BaseField < GraphQL::Schema::Argument
       accepts_definition :metadata
     end
 
+    GraphQL::Schema::Object.accepts_definition(:metadata2)
+
     class BaseObject < GraphQL::Schema::Object
       field_class BaseField
       accepts_definition :metadata
     end
 
+    GraphQL::Schema::Interface.accepts_definition(:metadata2)
+
     module BaseInterface
       include GraphQL::Schema::Interface
       field_class BaseField
@@ -39,10 +43,12 @@ class EnumValue < GraphQL::Schema::EnumValue
     module Thing
       include BaseInterface
       metadata :z, 888
+      metadata2 :a, :bc
     end
 
     class Query < BaseObject
       metadata :a, :abc
+      metadata2 :xyz, :zyx
 
       field :option, Option, null: false do
         metadata :a, :def
@@ -55,15 +61,16 @@ class Query < BaseObject
     query(Query)
   end
 
-
   it "passes along configs for types" do
     assert_equal [:a, 123], AcceptsDefinitionSchema::Option.metadata
     assert_equal 123, AcceptsDefinitionSchema::Option.graphql_definition.metadata[:a]
     assert_equal [:a, :abc], AcceptsDefinitionSchema::Query.metadata
     assert_equal :abc, AcceptsDefinitionSchema::Query.graphql_definition.metadata[:a]
+    assert_equal :zyx, AcceptsDefinitionSchema::Query.graphql_definition.metadata[:xyz]
 
     assert_equal [:z, 888], AcceptsDefinitionSchema::Thing.metadata
     assert_equal 888, AcceptsDefinitionSchema::Thing.graphql_definition.metadata[:z]
+    assert_equal :bc, AcceptsDefinitionSchema::Thing.graphql_definition.metadata[:a]
   end
 
   it "passes along configs for fields and arguments" do

spec/spec_helper.rb

@@ -40,6 +40,7 @@ def rails_should_be_installed?
 assign_metadata_flag = ->(target, flag) { target.metadata[flag] = true }
 GraphQL::Schema.accepts_definitions(set_metadata: assign_metadata_key)
 GraphQL::BaseType.accepts_definitions(metadata: assign_metadata_key)
+GraphQL::BaseType.accepts_definitions(metadata2: assign_metadata_key)
 GraphQL::Field.accepts_definitions(metadata: assign_metadata_key)
 GraphQL::Argument.accepts_definitions(metadata: assign_metadata_key)
 GraphQL::Argument.accepts_definitions(metadata_flag: assign_metadata_flag)