introduce ExecutionPath.cacheKeyReversedIterator

same API as in sangria 2.x: #781 It will allow to optimize sangria-slow-logs
sangria-graphql · Nov 3, 2021 · 40ef50a · 40ef50a
1 parent 21d83e0
commit 40ef50a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/modules/core/src/main/scala/sangria/execution/ExecutionPath.scala b/modules/core/src/main/scala/sangria/execution/ExecutionPath.scala
@@ -39,6 +39,7 @@ class ExecutionPath private (
 
   def cacheKey: ExecutionPath.PathCacheKey = cacheKeyPath.reverseIterator.toVector
   def cacheKeyReversed: ExecutionPath.PathCacheKeyReversed = cacheKeyPath
+  def cacheKeyReversedIterator: Iterator[String] = cacheKeyPath.iterator
 
   override def toString: String = _path.reverseIterator
     .foldLeft(new StringBuilder) {

diff --git a/modules/core/src/test/scala/sangria/execution/DeprecationTrackerSpec.scala b/modules/core/src/test/scala/sangria/execution/DeprecationTrackerSpec.scala
@@ -107,6 +107,10 @@ class DeprecationTrackerSpec
       deprecationTracker.ctx.get.path.path should be(Vector("nested", "aa", "bb"))
       deprecationTracker.ctx.get.path.cacheKey should be(
         Vector("nested", "TestType", "aa", "TestType", "bb", "TestType"))
+      deprecationTracker.ctx.get.path.cacheKeyReversed should be(
+        List("TestType", "bb", "TestType", "aa", "TestType", "nested"))
+      deprecationTracker.ctx.get.path.cacheKeyReversedIterator.toList should be(
+        List("TestType", "bb", "TestType", "aa", "TestType", "nested"))
       deprecationTracker.ctx.get.field.name should be("deprecated")
       deprecationTracker.ctx.get.parentType.name should be("TestType")
     }