Merged tests to produce a single binary (#12826)

This PR should close #7147 

# Description
Merged src/tests into /tests to produce a single binary.

![image](https://github.com/nushell/nushell/assets/94604837/84726469-d447-4619-b6d1-2d1415d0f42e)

# User-Facing Changes
No user facing changes

# Tests + Formatting
Moved tests. Tollkit check pr pass.

# After Submitting

---------

Co-authored-by: Ian Manske <ian.manske@pm.me>

CONTRIBUTING.md

@@ -55,7 +55,6 @@ It is good practice to cover your changes with a test. Also, try to think about
 
 Tests can be found in different places:
 * `/tests`
-* `src/tests`
 * command examples
 * crate-specific tests
 

src/main.rs

@@ -7,8 +7,6 @@ mod signals;
 #[cfg(unix)]
 mod terminal;
 mod test_bins;
-#[cfg(test)]
-mod tests;
 
 #[cfg(feature = "mimalloc")]
 #[global_allocator]

tests/main.rs

@@ -11,5 +11,6 @@ mod path;
 mod plugin_persistence;
 #[cfg(feature = "plugin")]
 mod plugins;
+mod repl;
 mod scope;
 mod shell;

tests/repl/mod.rs

@@ -0,0 +1,27 @@
+mod test_bits;
+mod test_cell_path;
+mod test_commandline;
+mod test_conditionals;
+mod test_config;
+mod test_config_path;
+mod test_converters;
+mod test_custom_commands;
+mod test_engine;
+mod test_env;
+mod test_help;
+mod test_hiding;
+mod test_ide;
+mod test_iteration;
+mod test_known_external;
+mod test_math;
+mod test_modules;
+mod test_parser;
+mod test_ranges;
+mod test_regex;
+mod test_signatures;
+mod test_spread;
+mod test_stdlib;
+mod test_strings;
+mod test_table_operations;
+mod test_type_check;
+mod tests;

tests/repl/test_bits.rs

@@ -1,4 +1,4 @@
-use crate::tests::{run_test, TestResult};
+use crate::repl::tests::{run_test, TestResult};
 
 #[test]
 fn bits_and() -> TestResult {

tests/repl/test_cell_path.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 // Tests for null / null / Value::Nothing
 #[test]

tests/repl/test_commandline.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn commandline_test_get_empty() -> TestResult {

tests/repl/test_conditionals.rs

@@ -1,4 +1,4 @@
-use crate::tests::{run_test, TestResult};
+use crate::repl::tests::{run_test, TestResult};
 
 #[test]
 fn if_test1() -> TestResult {

tests/repl/test_config.rs

@@ -1,5 +1,4 @@
-use super::{fail_test, run_test, run_test_std};
+use crate::repl::tests::{fail_test, run_test, run_test_std, TestResult};
-use crate::tests::TestResult;
 
 #[test]
 fn mutate_nu_config() -> TestResult {

tests/repl/test_converters.rs

@@ -1,4 +1,4 @@
-use crate::tests::{run_test, TestResult};
+use crate::repl::tests::{run_test, TestResult};
 
 #[test]
 fn from_json_1() -> TestResult {

tests/repl/test_custom_commands.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, run_test_contains, TestResult};
+use crate::repl::tests::{fail_test, run_test, run_test_contains, TestResult};
 use nu_test_support::nu;
 use pretty_assertions::assert_eq;
 

tests/repl/test_engine.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 use rstest::rstest;
 
 #[test]

tests/repl/test_env.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 use nu_test_support::nu;
 
 #[test]

tests/repl/test_help.rs

@@ -1,4 +1,4 @@
-use crate::tests::{run_test, TestResult};
+use crate::repl::tests::{run_test, TestResult};
 use rstest::rstest;
 
 #[rstest]

tests/repl/test_hiding.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 // TODO: Test the use/hide tests also as separate lines in REPL (i.e., with  merging the delta in between)
 #[test]

tests/repl/test_ide.rs

@@ -1,4 +1,4 @@
-use crate::tests::{test_ide_contains, TestResult};
+use crate::repl::tests::{test_ide_contains, TestResult};
 
 #[test]
 fn parser_recovers() -> TestResult {

tests/repl/test_iteration.rs

@@ -1,4 +1,4 @@
-use crate::tests::{run_test, TestResult};
+use crate::repl::tests::{run_test, TestResult};
 
 #[test]
 fn better_block_types() -> TestResult {

tests/repl/test_known_external.rs

@@ -1,7 +1,6 @@
+use crate::repl::tests::{fail_test, run_test, run_test_contains, TestResult};
 use std::process::Command;
 
-use crate::tests::{fail_test, run_test, run_test_contains, TestResult};
-
 // cargo version prints a string of the form:
 // cargo 1.60.0 (d1fd9fe2c 2022-03-01)
 

tests/repl/test_math.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn add_simple() -> TestResult {

tests/repl/test_modules.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn module_def_imports_1() -> TestResult {

tests/repl/test_parser.rs

@@ -1,9 +1,7 @@
-use crate::tests::{fail_test, run_test, run_test_with_env, TestResult};
+use crate::repl::tests::{fail_test, run_test, run_test_contains, run_test_with_env, TestResult};
 use nu_test_support::{nu, nu_repl_code};
 use std::collections::HashMap;
 
-use super::run_test_contains;
-
 #[test]
 fn env_shorthand() -> TestResult {
     run_test("FOO=BAR if false { 3 } else { 4 }", "4")

tests/repl/test_ranges.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn int_in_inc_range() -> TestResult {

tests/repl/test_regex.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn contains() -> TestResult {

tests/repl/test_signatures.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn list_annotations() -> TestResult {

tests/repl/test_spread.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 use nu_test_support::nu;
 
 #[test]

tests/repl/test_stdlib.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test_std, TestResult};
+use crate::repl::tests::{fail_test, run_test_std, TestResult};
 
 #[test]
 fn library_loaded() -> TestResult {

tests/repl/test_strings.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn cjk_in_substrings() -> TestResult {

tests/repl/test_table_operations.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn illegal_column_duplication() -> TestResult {

tests/repl/test_type_check.rs

@@ -1,4 +1,4 @@
-use crate::tests::{fail_test, run_test, TestResult};
+use crate::repl::tests::{fail_test, run_test, TestResult};
 
 #[test]
 fn chained_operator_typecheck() -> TestResult {

tests/repl/tests.rs

@@ -1,30 +1,3 @@
-mod test_bits;
-mod test_cell_path;
-mod test_commandline;
-mod test_conditionals;
-mod test_config;
-mod test_config_path;
-mod test_converters;
-mod test_custom_commands;
-mod test_engine;
-mod test_env;
-mod test_help;
-mod test_hiding;
-mod test_ide;
-mod test_iteration;
-mod test_known_external;
-mod test_math;
-mod test_modules;
-mod test_parser;
-mod test_ranges;
-mod test_regex;
-mod test_signatures;
-mod test_spread;
-mod test_stdlib;
-mod test_strings;
-mod test_table_operations;
-mod test_type_check;
-
 use assert_cmd::prelude::*;
 use pretty_assertions::assert_eq;
 use std::collections::HashMap;