Merge pull request #15 from Shopify/directive-arg-helpers

Move arg helpers to WithArgs module.

lib/graphql_schema.rb

@@ -38,6 +38,14 @@ module WithArgs
     def args
       @args ||= @hash.fetch('args').map{ |arg_hash| InputValue.new(arg_hash) }
     end
+
+    def required_args
+      @required_args ||= args.select{ |arg| arg.type.non_null? }
+    end
+
+    def optional_args
+      @optional_args ||= args.reject{ |arg| arg.type.non_null? }
+    end
   end
 
   module NamedHash
@@ -111,14 +119,6 @@ def initialize(field_hash)
       @hash = field_hash
     end
 
-    def required_args
-      @required_args ||= args.select{ |arg| arg.type.non_null? }
-    end
-
-    def optional_args
-      @optional_args ||= args.reject{ |arg| arg.type.non_null? }
-    end
-
     def type
       @type ||= TypeDeclaration.new(@hash.fetch('type'))
     end

test/graphql_schema_test.rb

@@ -164,7 +164,9 @@ def test_description
 
   def test_directives
     example_directive = directive("directiveExample")
-    assert_equal %w(input), example_directive.args.map(&:name)
+    assert_equal %w(input enabled), example_directive.args.map(&:name)
+    assert_equal %w(input), example_directive.required_args.map(&:name)
+    assert_equal %w(enabled), example_directive.optional_args.map(&:name)
     assert_equal "A nice runtime customization", example_directive.description
     assert_equal ["FIELD"], example_directive.locations
     refute example_directive.builtin?

test/support/schema.rb

@@ -102,6 +102,7 @@ class DirectiveExample < GraphQL::Schema::Directive
       description "A nice runtime customization"
       locations FIELD
       argument :input, !SetIntegerInput, required: true
+      argument :enabled, Boolean, required: false
     end
 
     ExampleSchema = GraphQL::Schema.define do