Remove the 'env' command, as we have the variable

Author: sophiajt

crates/nu-command/src/default_context.rs

@@ -343,7 +343,6 @@ pub fn create_default_context() -> EngineState {
 
         // Env
         bind_command! {
-            Env,
             ExportEnv,
             LetEnv,
             LoadEnv,

crates/nu-command/src/env/env_command.rs

@@ -1,5 +1,4 @@
 mod config;
-mod env_command;
 mod export_env;
 mod let_env;
 mod load_env;
@@ -10,7 +9,6 @@ pub use config::ConfigEnv;
 pub use config::ConfigMeta;
 pub use config::ConfigNu;
 pub use config::ConfigReset;
-pub use env_command::Env;
 pub use export_env::ExportEnv;
 pub use let_env::LetEnv;
 pub use load_env::LoadEnv;

crates/nu-command/src/env/mod.rs

@@ -14,11 +14,3 @@ fn shorthand_env_2() -> TestResult {
 fn shorthand_env_3() -> TestResult {
     run_test(r#"FOO=BAZ BAR=MOO $env.FOO"#, "BAZ")
 }
-
-#[test]
-fn convert_non_string_env_var_to_nothing() -> TestResult {
-    run_test(
-        r#"let-env FOO = true; env | where name == FOO | get raw.0 | describe"#,
-        "nothing",
-    )
-}

src/tests/test_env.rs

@@ -234,7 +234,7 @@ fn equals_separates_long_flag() -> TestResult {
 fn let_env_expressions() -> TestResult {
     let env = HashMap::from([("VENV_OLD_PATH", "Foobar"), ("Path", "Quux")]);
     run_test_with_env(
-        r#"let-env Path = if (env | any {|x| $x.name == VENV_OLD_PATH}) { $env.VENV_OLD_PATH } else { $env.Path }; echo $env.Path"#,
+        r#"let-env Path = if ($env | columns | "VENV_OLD_PATH" in $in) { $env.VENV_OLD_PATH } else { $env.Path }; echo $env.Path"#,
         "Foobar",
         &env,
     )