Merge pull request #107 from scalameta/fix-check-symbol-index

Fix checkSymbolIndex to be independent of the order files are listed
scalameta · Oct 5, 2018 · 3175406 · 3175406
2 parents d38932c + 063f152
commit 3175406
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/metadoc-tests/src/test/scala/metadoc/tests/BaseMetadocCliSuite.scala b/metadoc-tests/src/test/scala/metadoc/tests/BaseMetadocCliSuite.scala
@@ -39,7 +39,13 @@ class BaseMetadocCliSuite
         out.resolve("symbol").resolve(id.symbolIndexPath)
       )
       val index = indexes.indexes.find(_.symbol == id).get
-      val obtained = index.toProtoString
+      // Sort ranges to ensure we assert against deterministic input.
+      val indexNormalized = index.copy(
+        references = index.references.mapValues { ranges =>
+          ranges.copy(ranges = ranges.ranges.sortBy(_.startLine))
+        }
+      )
+      val obtained = indexNormalized.toProtoString
       assertNoDiffOrPrintExpected(obtained, expected)
     }
   }