schema: Deprecated directive is also valid on :argument-definition

In the current Working Draft of the GraphQL specification, @deprecated directives are now also valid for ARGUMENT_DEFINITION clauses.
walmartlabs · Apr 20, 2023 · 295d3c4 · 295d3c4
1 parent afa9c04
commit 295d3c4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/src/com/walmartlabs/lacinia/schema.clj b/src/com/walmartlabs/lacinia/schema.clj
@@ -1878,7 +1878,7 @@
                   (map-kvs compile-directive-args
                     (assoc directive-defs
                       :deprecated {:args {:reason {:type 'String}}
-                                   :locations #{:enum-value :field-definition :input-field-definition}})))))
+                                   :locations #{:argument-definition :enum-value :field-definition :input-field-definition}})))))
 
 (defn ^:private validate-directives-by-category
   [schema category]

diff --git a/test/com/walmartlabs/lacinia/directives_test.clj b/test/com/walmartlabs/lacinia/directives_test.clj
@@ -348,7 +348,8 @@
 (deftest union-directive-inapplicable
   (directive-test
     "Directive @deprecated on union `Ebb' is not applicable."
-    {:allowed-locations #{:enum-value
+    {:allowed-locations #{:argument-definition
+                          :enum-value
                           :field-definition
                           :input-field-definition}
      :directive-type :deprecated
@@ -375,7 +376,8 @@
 (deftest scalar-directive-inapplicable
   (directive-test
     "Directive @deprecated on scalar `Ebb' is not applicable."
-    {:allowed-locations #{:enum-value
+    {:allowed-locations #{:argument-definition
+                          :enum-value
                           :field-definition
                           :input-field-definition}
      :directive-type :deprecated