tests & use _name.toString

Author: ankitson

compiler/src/dotty/tools/dotc/interactive/Interactive.scala

@@ -219,7 +219,7 @@ object Interactive {
       case _  => getScopeCompletions(ctx)
     }
     interactiv.println(i"completion with pos = $pos, prefix = $prefix, termOnly = $termOnly, typeOnly = $typeOnly = ${completions.toList}%, %")
-    (completionPos, completions.toList.sortBy(_.name.show))
+    (completionPos, completions.toList.sortBy(_.name.toString))
   }
 
   /** Possible completions of members of `prefix` which are accessible when called inside `boundary` */

compiler/test/dotty/tools/repl/TabcompleteTests.scala

@@ -65,4 +65,14 @@ class TabcompleteTests extends ReplTest {
       List("\"", ")", "'", "¨", "£", ":", ",", ";", "@", "}", "[", "]")
         .foreach(src => assertTrue(tabComplete(src).suggestions.isEmpty))
     }
+
+  @Test def sortedCompletions: Unit =
+    fromInitialState { implicit state  =>
+      val src = """class Foo { def comp1 = 1; def compa = 3; def compA = 4 }"""
+      compiler.compile(src).stateOrFail
+    }
+    .andThen { implicit state =>
+      val expected = List("comp1", "compA", "compa")
+      assertEquals(expected, tabComplete("(new Foo).comp").suggestions)
+    }
 }