added inspect validation

Signed-off-by: munishchouhan <[email protected]>
seqeralabs · Oct 16, 2024 · cff8585 · cff8585
1 parent 42e72e2
commit cff8585
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/app/src/main/java/io/seqera/wave/cli/App.java b/app/src/main/java/io/seqera/wave/cli/App.java
@@ -348,6 +348,9 @@ protected void validateArgs() {
         if( singularity && !freeze )
             throw new IllegalCliArgumentException("Singularity build requires enabling freeze mode");
 
+        if( inspect && isEmpty(image) )
+            throw new IllegalCliArgumentException("Option --inspect requires the use of container image (--image)");
+
         if( !isEmpty(contextDir) ) {
             // check that a container file has been provided
             if( isEmpty(containerFile) )

diff --git a/app/src/test/groovy/io/seqera/wave/cli/AppTest.groovy b/app/src/test/groovy/io/seqera/wave/cli/AppTest.groovy
@@ -585,4 +585,20 @@ class AppTest extends Specification {
         '2.0.0'     | '2.1.0'       | '2.0.0 (required: 2.1.0)'
     }
 
+    def 'should fail when inspecting without container image' () {
+        given:
+        def app = new App()
+        String[] args = ["--conda-package", "bwa", "--freeze", "--singularity", "--inspect"]
+
+        when:
+        def cli = new CommandLine(app)
+        cli.parseArgs(args)
+        and:
+        app.validateArgs()
+        then:
+        def e = thrown(IllegalCliArgumentException)
+        and:
+        e.getMessage() == "Option --inspect requires the use of container image (--image)"
+    }
+
 }