SI-9750 Tweak tests for what is a number

Leaves the error string as is, but adds test to show how it looks.
Java calls it a version number. `Not a version: 1.9`.

Don't strip `1.` prefix recursively. (That was Snytt's fault.)

1 files changed, 4 insertions, 21 deletions

diff --git a/test/junit/scala/util/SpecVersionTest.scala b/test/junit/scala/util/SpecVersionTest.scala
index 2b69f288fa..4c16ff08fb 100644
--- a/test/junit/scala/util/SpecVersionTest.scala
+++ b/test/junit/scala/util/SpecVersionTest.scala
@@ -50,30 +50,13 @@ class SpecVersionTest {
   }
 
   @Test def variousBadVersionStrings(): Unit = {
-    assertThrows[NumberFormatException] { sut7 isJavaAtLeast "1.9" }
+    assertThrows[NumberFormatException](sut7.isJavaAtLeast("1.9"), _ == "Not a version: 1.9")
     assertThrows[NumberFormatException] { sut9 isJavaAtLeast "1.9" }
     assertThrows[NumberFormatException] { sut7 isJavaAtLeast "9.1" }
     assertThrows[NumberFormatException] { sut9 isJavaAtLeast "9.1" }
-  }
 
-  @Test(expected = classOf[NumberFormatException])
-  def badVersion(): Unit = {
-    sut7 isJavaAtLeast "1.a"
-  }
-  @Test(expected = classOf[NumberFormatException])
-  def noVersion(): Unit = {
-    sut7 isJavaAtLeast ""
-  }
-  @Test(expected = classOf[NumberFormatException])
-  def dotOnly(): Unit = {
-    sut7 isJavaAtLeast "."
-  }
-  @Test(expected = classOf[NumberFormatException])
-  def leadingDot(): Unit = {
-    sut7 isJavaAtLeast ".5"
-  }
-  @Test(expected = classOf[NumberFormatException])
-  def notASpec(): Unit = {
-    sut7 isJavaAtLeast "1.7.1"
+    val badvs = List("1.1.8", "1.", "1.a", "", ".", ".5", "1.7.1")
+
+    for (v <- badvs) assertThrows[NumberFormatException](sut7.isJavaAtLeast(v))
   }
 }