test: Fix --reload in integration_tests (#9345)

This commit removes redundant "--reload" args because "util::deno_cmd"
recreates "DENO_DIR". 

This commit also fixes ta_reload in integration tests to actually test reload.
This commit is contained in:
Yosi Pramajaya 2021-02-24 21:18:35 +07:00 committed by GitHub
parent 9cc7e32e37
commit f6a80f34d9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 10 deletions

View File

@ -31,7 +31,6 @@ fn js_unit_tests() {
.current_dir(util::root_path())
.arg("run")
.arg("--unstable")
.arg("--reload")
.arg("-A")
.arg("cli/tests/unit/unit_test_runner.ts")
.arg("--master")
@ -1328,31 +1327,37 @@ mod integration {
fn ts_reload() {
let hello_ts = util::root_path().join("cli/tests/002_hello.ts");
assert!(hello_ts.is_file());
let mut initial = util::deno_cmd()
let deno_dir = TempDir::new().expect("tempdir fail");
let mut initial = util::deno_cmd_with_deno_dir(deno_dir.path())
.current_dir(util::root_path())
.arg("cache")
.arg("--reload")
.arg(hello_ts.clone())
.arg(&hello_ts)
.spawn()
.expect("failed to spawn script");
let status_initial =
initial.wait().expect("failed to wait for child process");
assert!(status_initial.success());
let output = util::deno_cmd()
let output = util::deno_cmd_with_deno_dir(deno_dir.path())
.current_dir(util::root_path())
.arg("cache")
.arg("--reload")
.arg("-L")
.arg("debug")
.arg(hello_ts)
.arg(&hello_ts)
.output()
.expect("failed to spawn script");
// check the output of the the bundle program.
let output_path = hello_ts.canonicalize().unwrap();
assert!(std::str::from_utf8(&output.stderr)
.unwrap()
.trim()
.contains("host.writeFile(\"deno://002_hello.js\")"));
.contains(&format!(
"host.getSourceFile(\"{}\", Latest)",
url::Url::from_file_path(&output_path).unwrap().as_str()
)));
}
#[test]
@ -1494,7 +1499,6 @@ mod integration {
let output = util::deno_cmd()
.current_dir(util::root_path())
.arg("run")
.arg("--reload")
.arg(&bundle)
.output()
.expect("failed to spawn script");

View File

@ -1151,11 +1151,15 @@ pub fn new_deno_dir() -> TempDir {
}
pub fn deno_cmd() -> Command {
let e = deno_exe_path();
let deno_dir = new_deno_dir();
deno_cmd_with_deno_dir(deno_dir.path())
}
pub fn deno_cmd_with_deno_dir(deno_dir: &std::path::Path) -> Command {
let e = deno_exe_path();
assert!(e.exists());
let mut c = Command::new(e);
c.env("DENO_DIR", deno_dir.path());
c.env("DENO_DIR", deno_dir);
c
}