Merge pull request #4304 from adriaanm/rebase-4219

SI-8818 FreshName extractor forgives suffix

2 files changed, 16 insertions, 17 deletions

diff --git a/test/junit/scala/tools/nsc/symtab/FreshNameExtractorTest.scala b/test/junit/scala/tools/nsc/symtab/FreshNameExtractorTest.scala
index effbfb2f7c..7796345351 100644
--- a/test/junit/scala/tools/nsc/symtab/FreshNameExtractorTest.scala
+++ b/test/junit/scala/tools/nsc/symtab/FreshNameExtractorTest.scala
@@ -32,16 +32,16 @@ class FreshNameExtractorTest {
     val Creator = new FreshNameCreator(prefixes.head)
     val Extractor = new FreshNameExtractor(prefixes.tail.head)
     assertThrows[MatchError] {
-      val Extractor(_) = TermName(Creator.newName("foo"))
+      TermName(Creator.newName("foo")) match { case Extractor(_) => }
     }
   }
 
-  @Test @org.junit.Ignore // SI-8818
-  def extractionsFailsIfNameDoesntEndWithNumber = {
-    val Creator = new FreshNameCreator(prefixes.head)
+  @Test
+  def `no numeric suffix? no problem!` = {
+    val Creator   = new FreshNameCreator(prefixes.head)
     val Extractor = new FreshNameExtractor(prefixes.head)
-    assertThrows[MatchError] {
-      val Extractor(_) = TermName(Creator.newName("foo") + "bar")
+    TermName(Creator.newName("foo") + "bar") match {
+      case Extractor(_) =>
     }
   }
 }
diff --git a/test/junit/scala/tools/testing/AssertUtil.scala b/test/junit/scala/tools/testing/AssertUtil.scala
index d29f9a473f..d798f2e53e 100644
--- a/test/junit/scala/tools/testing/AssertUtil.scala
+++ b/test/junit/scala/tools/testing/AssertUtil.scala
@@ -36,21 +36,20 @@ object AssertUtil {
     }
   }
 
-  /** Check if throwable T (or a subclass) was thrown during evaluation of f, and that its message
-   *  satisfies the `checkMessage` predicate. If any other exception will be re-thrown.
+  /** Check that throwable T (or a subclass) was thrown during evaluation of `body`,
+   *  and that its message satisfies the `checkMessage` predicate.
+   *  Any other exception is propagated.
    */
-  def assertThrows[T <: Throwable](f: => Any,
+  def assertThrows[T <: Throwable](body: => Any,
                                    checkMessage: String => Boolean = s => true)
                                   (implicit manifest: Manifest[T]): Unit = {
-    try f
-    catch {
-      case e: Throwable if checkMessage(e.getMessage) =>
-        val clazz = manifest.runtimeClass
-        if (!clazz.isAssignableFrom(e.getClass))
-          throw e
-        else return
+    try {
+      body
+      fail("Expression did not throw!")
+    } catch {
+      case e: Throwable if (manifest.runtimeClass isAssignableFrom e.getClass) &&
+                            checkMessage(e.getMessage) =>
     }
-    fail("Expression did not throw!")
   }
 
   /** JUnit-style assertion for `IterableLike.sameElements`.