Update getDomain method in ProviderConfig to support local repositori…

…es - #1492 Signed-off-by: Emilio Palumbo <emiliopalumbo@gmail.com>
nextflow-io · Jun 9, 2020 · bcec488 · bcec488
1 parent 28e8c5a
commit bcec488
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 3 deletions.
diff --git a/modules/nextflow/src/main/groovy/nextflow/scm/ProviderConfig.groovy b/modules/nextflow/src/main/groovy/nextflow/scm/ProviderConfig.groovy
@@ -111,8 +111,8 @@ class ProviderConfig {
         if( p != -1 )
             result = result.substring(p+3)
         p = result.indexOf('/')
-        if( p != -1 )
-            result = result.substring(0,p)
+        if( p > 0 )
+            result = result.substring(0, p)
         return result
     }
 

diff --git a/modules/nextflow/src/test/groovy/nextflow/scm/ProviderConfigTest.groovy b/modules/nextflow/src/test/groovy/nextflow/scm/ProviderConfigTest.groovy
@@ -102,7 +102,25 @@ class ProviderConfigTest extends Specification {
     def 'should return provider attributes' () {
 
         when:
-        def config = new ProviderConfig('custom',[platform: 'github', server:'http://local.host'])
+        def config = new ProviderConfig('local',[platform: 'file', path:'/local/repo'])
+        then:
+        config.name == 'local'
+        config.platform == 'file'
+        config.domain == '/local/repo'
+        config.server == null
+        config.endpoint == null
+
+        when:
+        config = new ProviderConfig('local',[platform: 'file', path:'file:///local/repo'])
+        then:
+        config.name == 'local'
+        config.platform == 'file'
+        config.domain == '/local/repo'
+        config.server == null
+        config.endpoint == null
+
+        when:
+        config = new ProviderConfig('custom',[platform: 'github', server:'http://local.host'])
         then:
         config.name == 'custom'
         config.platform == 'github'