digest tests.

Author: gmac

lib/graphql/stitching/request.rb

@@ -31,7 +31,7 @@ class Request
       # @param context [Hash, nil] a contextual object passed through resolver flows.
       def initialize(supergraph, source, operation_name: nil, variables: nil, context: nil, claims: nil)
         @supergraph = supergraph
-        @claims = claims&.to_set&.freeze
+        @claims = claims&.sort&.to_set&.freeze
         @prepared_document = nil
         @string = nil
         @digest = nil

test/graphql/stitching/request/request_test.rb

@@ -145,6 +145,29 @@ def test_provides_digest_and_normalized_digest
     end
   end
 
+  def test_digests_include_claims
+    query = "{ widget { id } }"
+    request1 = GraphQL::Stitching::Request.new(@supergraph, query)
+    request2 = GraphQL::Stitching::Request.new(@supergraph, query)
+    assert_equal request1.digest, request2.digest
+    assert_equal request1.normalized_digest, request2.normalized_digest
+
+    request3 = GraphQL::Stitching::Request.new(@supergraph, query, claims: ["a"])
+    request4 = GraphQL::Stitching::Request.new(@supergraph, query, claims: ["a"])
+    assert_equal request3.digest, request4.digest
+    assert_equal request3.normalized_digest, request4.normalized_digest
+
+    request5 = GraphQL::Stitching::Request.new(@supergraph, query, claims: ["a", "b"])
+    request6 = GraphQL::Stitching::Request.new(@supergraph, query, claims: ["b", "a"])
+    assert_equal request5.digest, request6.digest
+    assert_equal request5.normalized_digest, request6.normalized_digest
+
+    assert request1.digest != request3.digest
+    assert request3.digest != request5.digest
+    assert request1.normalized_digest != request3.normalized_digest
+    assert request3.normalized_digest != request5.normalized_digest
+  end
+
   def test_prepare_variables_collects_variable_defaults
     query = %|
       query($a: String! = "defaultA", $b: String = "defaultB") {