Use actual test spec files for simplicity and as examples

Author: NonlinearFruit

nuunit.nu

@@ -1,13 +1,12 @@
 def main [
   --test-spec-module-name = "test-spec.nu"
-  --test-spec-module-script:string
   --as-json
 ] {
-  if (not ($test_spec_module_name | path exists) and ($test_spec_module_script | is-empty)) {
+  if (not ($test_spec_module_name | path exists)) {
     return $"Invalid test spec module: ($test_spec_module_name)"
   }
-  let module = $test_spec_module_name | str replace '.nu' ''
-  let importScript = $test_spec_module_script | default $"use ($test_spec_module_name) *"
+  let module = $test_spec_module_name | split row '/' | last | str replace '.nu' ''
+  let importScript = $"use ($test_spec_module_name) *"
   let testResults = discover-tests $module $importScript
   | run-tests
 

test-spec.nu

@@ -7,70 +7,66 @@ export def "test handles when spec does not exist" [] {
 }
 
 export def "test exit with error when test errors" [] {
-  let specScript = "
-    export module test {
-      export def test_that_fails [] { exit 400 }
-    }
-  "
+  use tests/test-spec-that-errs.nu "verify json results"
+  let specFile = "tests/test-spec-that-errs.nu"
+
   do {
-    ^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name "test" --test-spec-module-script $specScript
+    ^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile
   }
   | complete
   | assert not equal 0 ($in.exit_code)
 }
 
-export def "test handles when no tests" [] {
-  let specScript = "
-    export module test {
-    }
-    use test *
-  "
-  (^$nu.current-exe --no-config-file nuunit.nu
-    --test-spec-module-name "test"
-    --test-spec-module-script $specScript
-    --as-json)
+export def "test when there are no tests everything still works" [] {
+  use tests/test-spec-with-zero-tests.nu "verify json results"
+  let specFile = "tests/test-spec-with-zero-tests.nu"
+
+  (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
   | from json
-  | length
-  | assert equal 0 $in
+  | verify json results
 }
 
-export def "test runs tests from spec script" [] {
-  let specScript = "
-    export module test {
-      export def test_the_stuff [] {}
-      export def test_the_other [] {}
-    }
-    use test *
-  "
-  (^$nu.current-exe --no-config-file nuunit.nu
-    --test-spec-module-name "test"
-    --test-spec-module-script $specScript
-    --as-json)
+export def "test when there is one test everything still works" [] {
+  use tests/test-spec-with-one-test.nu "verify json results"
+  let specFile = "tests/test-spec-with-one-test.nu"
+
+  (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
+  | from json
+  | verify json results
+}
+
+export def "test when there are two tests everything still works" [] {
+  use tests/test-spec-with-two-tests.nu "verify json results"
+  let specFile = "tests/test-spec-with-two-tests.nu"
+
+  (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
   | from json
-  | length
-  | assert equal 2 $in
+  | verify json results
 }
 
 export def "test when test errors runner keeps chugging" [] {
-  let specScript = "
-    export module test {
-      export def test_the_stuff [] {}
-    }
-    use test *
-  "
-  (^$nu.current-exe --no-config-file nuunit.nu
-    --test-spec-module-name "test"
-    --test-spec-module-script $specScript
-    --as-json)
-  | print
+  use tests/test-spec-that-errs.nu "verify json results"
+  let specFile = "tests/test-spec-that-errs.nu"
+
+  (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
+  | from json
+  | verify json results
 }
 
-export def "not starting with test means the command will not run" [] {
-  "why would you run this?"
-  exit 1
+export def "test when exported command does not match pattern it is not included" [] {
+  use tests/test-spec-with-exported-commands-that-are-not-tests.nu "verify json results"
+  let specFile = "tests/test-spec-with-exported-commands-that-are-not-tests.nu"
+
+  (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
+  | from json
+  | verify json results
 }
 
-def "test private commands are not magically ran" [] {
-  "why would you run this?"
-  exit 1
+export def "test private commands that look likes tests are not included" [] {
+  use tests/test-spec-with-private-commands-that-look-like-tests.nu "verify json results"
+  let specFile = "tests/test-spec-with-private-commands-that-look-like-tests.nu"
+
+  (^$nu.current-exe --no-config-file nuunit.nu --test-spec-module-name $specFile --as-json)
+  | from json
+  | verify json results
 }

tests/test-spec-that-errs.nu

@@ -0,0 +1,12 @@
+export def "test that is broken" [] {
+  error make { msg: "It broke!" }
+}
+
+export def "verify json results" [] {
+  let results = $in
+  use std assert
+
+  assert equal 1 ($results | length)
+  assert not equal 0 ($results.exit_code)
+  assert not equal "" ($results.stderr)
+}

tests/test-spec-with-exported-commands-that-are-not-tests.nu

@@ -0,0 +1,11 @@
+export def "not starting with test means the command will not run" [] {
+  "why would you run this?"
+  exit 1
+}
+
+export def "verify json results" [] {
+  let results = $in
+  use std assert
+
+  assert equal 0 ($results | length)
+}

tests/test-spec-with-one-test.nu

@@ -0,0 +1,9 @@
+export def "test 1" [] {}
+
+export def "verify json results" [] {
+  let results = $in
+  use std assert
+
+  assert equal 1 ($results | length)
+}
+

tests/test-spec-with-private-commands-that-look-like-tests.nu

@@ -0,0 +1,11 @@
+def "test private command that looks like a test is not included" [] {
+  "why would you run this?"
+  exit 1
+}
+
+export def "verify json results" [] {
+  let results = $in
+  use std assert
+
+  assert equal 0 ($results | length)
+}

tests/test-spec-with-two-tests.nu

@@ -0,0 +1,10 @@
+export def "test 1" [] {}
+export def "test 2" [] {}
+
+export def "verify json results" [] {
+  let results = $in
+  use std assert
+
+  assert equal 2 ($results | length)
+}
+

tests/test-spec-with-zero-tests.nu

@@ -0,0 +1,6 @@
+export def "verify json results" [] {
+  let results = $in
+  use std assert
+
+  assert equal 0 ($results | length)
+}