Merge pull request #363 from ethauvin/master

file:// no longer considered valid maven coordinates

modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/MavenId.kt

@@ -17,8 +17,12 @@ class MavenId private constructor(val groupId: String, val artifactId: String, v
         val classifier: String?, val version: String?) {
 
     companion object {
-        fun isMavenId(id: String) = with(id.split(':')) {
+        fun isMavenId(id: String) = if (id.startsWith("file://")) {
-            size >= 3 && size <= 5
+            false
+        } else {
+            with(id.split(':')) {
+                size >= 3 && size <= 5
+            }
         }
 
         fun isRangedVersion(s: String): Boolean {

src/test/kotlin/com/beust/kobalt/maven/MavenIdTest.kt

@@ -20,6 +20,16 @@ class MavenIdTest {
         )
     }
 
+    @Test
+    fun isMavenId() {
+        Assert.assertFalse(MavenId.isMavenId("file://C:\\foo\\bar"))
+        Assert.assertFalse(MavenId.isMavenId("file:///home/user/foo/bar"))
+        Assert.assertFalse(MavenId.isMavenId("com.example:foo"))
+        Assert.assertTrue(MavenId.isMavenId("com.example:foo:"))
+        Assert.assertTrue(MavenId.isMavenId("com.example:foo:0.5.7"))
+        Assert.assertTrue(MavenId.isMavenId("com.example:foo:jar:0.3.0"))
+    }
+
     @Test(dataProvider = "dp")
     fun parseVersions(id: String, groupId: String, artifactId: String, version: String?,
             packaging: String?, qualifier: String?) {