bugfix: Fix off by one error when doing reveal

scalameta · Nov 3, 2023 · a4558e8 · a4558e8
1 parent c758243
commit a4558e8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/metals/src/main/scala/scala/meta/internal/tvp/MetalsTreeViewProvider.scala b/metals/src/main/scala/scala/meta/internal/tvp/MetalsTreeViewProvider.scala
@@ -159,7 +159,7 @@ class MetalsTreeViewProvider(
       val closestSymbol = occurrences.minBy { occ =>
         val startLine = occ.range.fold(Int.MaxValue)(_.startLine)
         val distance = math.abs(pos.getLine - startLine)
-        val isLeading = pos.getLine() > startLine
+        val isLeading = pos.getLine() >= startLine
         (!isLeading, distance)
       }
       val result =

diff --git a/tests/unit/src/test/scala/tests/ScalaToplevelSuite.scala b/tests/unit/src/test/scala/tests/ScalaToplevelSuite.scala
@@ -620,10 +620,12 @@ class ScalaToplevelSuite extends BaseSuite {
               .map(_.symbol)
               .toList
           case Toplevel =>
-            val input = AbsolutePath(Files.createTempFile("mtags", ".java"))
+            val dir = AbsolutePath(Files.createTempDirectory("mtags"))
+            val input = dir.resolve("Test.scala")
             input.writeText(code)
             val obtained = Mtags.topLevelSymbols(input, dialect)
             input.delete()
+            dir.delete()
             obtained
         }
       assertNoDiff(

diff --git a/tests/unit/src/test/scala/tests/TreeViewLspSuite.scala b/tests/unit/src/test/scala/tests/TreeViewLspSuite.scala
@@ -337,8 +337,7 @@ class TreeViewLspSuite extends BaseLspSuite("tree-view") {
               |        Text symbol-object
               |        TextMacros symbol-interface
               |        Util symbol-object
-              |        File symbol-class
-              |          value symbol-field
+              |        File symbol-object
               |""".stripMargin,
         )
         assertNoDiff(