use concat, improve test

Author: chrisbutcher

lib/graphql/schema/field.rb

@@ -298,7 +298,7 @@ def extensions(new_extensions = nil)
       # Add `extension` to this field, initialized with `options` if provided.
       # @param extension [Class] subclass of {Schema::Fieldextension}
       # @param options [Object] if provided, given as `options:` when initializing `extension`.
-      def extension(extension, options = nil) # TODO: Do we want a single extra setter, like this too?
+      def extension(extension, options = nil)
         extensions([{extension => options}])
       end
 
@@ -313,9 +313,7 @@ def extras(new_extras = nil)
           @extras
         else
           # Append to the set of extras on this field
-          new_extras.each do |new_extra|
-            @extras << new_extra
-          end
+          @extras.concat(new_extras)
         end
       end
 

spec/graphql/schema/field_spec.rb

@@ -139,13 +139,13 @@
         object = Class.new(Jazz::BaseObject) do
           graphql_name "JustAName"
 
-          field :test, String, null: true
+          field :test, String, null: true, extras: [:lookahead]
         end
 
         field = object.fields['test']
 
         field.extras([:ast_node])
-        assert_equal [:ast_node], field.extras
+        assert_equal [:lookahead, :ast_node], field.extras
       end
     end