Gracefully handled when nuunit commands are not imported (#11)

Author: NonlinearFruit

nuunit.nu

@@ -4,7 +4,7 @@ export def main [
   --test-spec-module-name = "test-spec.nu"
   --as-json
 ] {
-  if (not ($test_spec_module_name | path exists)) {
+  if (is-not-valid-test-spec $test_spec_module_name) {
     return $"Invalid test spec module: ($test_spec_module_name)"
   }
   let module = $test_spec_module_name | split row '/' | last | str replace '.nu' ''
@@ -24,17 +24,30 @@ export def main [
   | exit $in
 }
 
+def is-not-valid-test-spec [spec] {
+  not ($spec | path exists)
+}
+
 def run-test-discoverer [module testImportScript] {
   run-nushell [
     --commands
     $"($testImportScript)
-    (view source discover-tests)
+    (inline-source-code discover-tests)
     discover-tests '($module)' '($testImportScript)'
     | to nuon"
   ]
   | from nuon
 }
 
+def inline-source-code [command] {
+  try {
+    view source $command
+  } catch {
+    view source $"nuunit ($command)"
+    | str replace 'nuunit ' ''
+  }
+}
+
 export def discover-tests [module testImportScript] {
   scope modules
   | where name == $module

test-spec.nu

@@ -114,6 +114,18 @@ export def "test that the after each passes context to a test" [] {
   | verify json results
 }
 
+export def "test that importing all nuunit commands is gracefully handled" [] {
+  use std assert
+  let specFile = "tests/generic-test-spec.nu"
+
+  let results = do {
+    ^$nu.current-exe --no-config-file -c $'use nuunit.nu; nuunit --test-spec-module-name ($specFile)'
+  }
+  | complete
+
+  assert (0 == $results.exit_code)
+}
+
 def run-test-spec [specFile] {
   (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
   | from json