Skip to content

Commit

Permalink
using use instead of overlay use in some tests
Browse files Browse the repository at this point in the history
  • Loading branch information
WindSoilder committed Sep 19, 2022
1 parent ccd871d commit 2ae24b2
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions src/tests/test_hiding.rs
Original file line number Diff line number Diff line change
Expand Up @@ -385,7 +385,7 @@ fn use_def_import_after_hide() -> TestResult {
#[test]
fn use_env_import_after_hide() -> TestResult {
run_test(
r#"module spam { export-env { let-env foo = "foo" } }; overlay use spam; hide-env foo; overlay use spam; $env.foo"#,
r#"module spam { export-env { let-env foo = "foo" } }; use spam; hide-env foo; use spam; $env.foo"#,
"foo",
)
}
Expand Down Expand Up @@ -417,7 +417,7 @@ fn hides_all_decls_within_scope() -> TestResult {
#[test]
fn hides_all_envs_within_scope() -> TestResult {
fail_test(
r#"module spam { export-env { let-env foo = "bar" } }; let-env foo = "foo"; overlay use spam; hide-env foo; $env.foo"#,
r#"module spam { export-env { let-env foo = "bar" } }; let-env foo = "foo"; use spam; hide-env foo; $env.foo"#,
"did you mean",
)
}
10 changes: 5 additions & 5 deletions src/tests/test_modules.rs
Original file line number Diff line number Diff line change
Expand Up @@ -43,31 +43,31 @@ fn module_def_imports_5() -> TestResult {
#[test]
fn module_env_imports_1() -> TestResult {
run_test(
r#"module foo { export-env { let-env a = '1' } }; overlay use foo; $env.a"#,
r#"module foo { export-env { let-env a = '1' } }; use foo; $env.a"#,
"1",
)
}

#[test]
fn module_env_imports_2() -> TestResult {
run_test(
r#"module foo { export-env { let-env a = '1'; let-env b = '2' } }; overlay use foo; $env.b"#,
r#"module foo { export-env { let-env a = '1'; let-env b = '2' } }; use foo; $env.b"#,
"2",
)
}

#[test]
fn module_env_imports_3() -> TestResult {
run_test(
r#"module foo { export-env { let-env a = '1' }; export-env { let-env b = '2' }; export-env {let-env c = '3'} }; overlay use foo; $env.c"#,
r#"module foo { export-env { let-env a = '1' }; export-env { let-env b = '2' }; export-env {let-env c = '3'} }; use foo; $env.c"#,
"3",
)
}

#[test]
fn module_def_and_env_imports_1() -> TestResult {
run_test(
r#"module spam { export-env { let-env foo = "foo" }; export def foo [] { "bar" } }; overlay use spam; $env.foo"#,
r#"module spam { export-env { let-env foo = "foo" }; export def foo [] { "bar" } }; use spam; $env.foo"#,
"foo",
)
}
Expand All @@ -91,7 +91,7 @@ fn module_def_import_uses_internal_command() -> TestResult {
#[test]
fn module_env_import_uses_internal_command() -> TestResult {
run_test(
r#"module foo { def b [] { "2" }; export-env { let-env a = b } }; use foo; overlay use foo; $env.a"#,
r#"module foo { def b [] { "2" }; export-env { let-env a = b } }; use foo; use foo; $env.a"#,
"2",
)
}
Expand Down
2 changes: 1 addition & 1 deletion tests/overlays/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ fn update_overlay_from_module_env() {
#[test]
fn overlay_use_do_not_eval_twice() {
let inp = &[
r#"module spam { export env FOO { "foo" } }"#,
r#"module spam { export-env { let-env FOO = "foo" } }"#,
r#"overlay use spam"#,
r#"let-env FOO = "bar""#,
r#"overlay hide spam"#,
Expand Down

0 comments on commit 2ae24b2

Please sign in to comment.