Skip to content

Commit de45c6f

Browse files
lf-ehuss
authored andcommitted
Delete uses of the USER env var from tests
1 parent 96f8dbf commit de45c6f

File tree

3 files changed

+38
-98
lines changed

3 files changed

+38
-98
lines changed

tests/testsuite/init.rs

Lines changed: 23 additions & 57 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ fn mercurial_available() -> bool {
2626
#[cargo_test]
2727
fn simple_lib() {
2828
cargo_process("init --lib --vcs none --edition 2015")
29-
.env("USER", "foo")
3029
.with_stderr("[CREATED] library package")
3130
.run();
3231

@@ -42,7 +41,6 @@ fn simple_bin() {
4241
let path = paths::root().join("foo");
4342
fs::create_dir(&path).unwrap();
4443
cargo_process("init --bin --vcs none --edition 2015")
45-
.env("USER", "foo")
4644
.cwd(&path)
4745
.with_stderr("[CREATED] binary (application) package")
4846
.run();
@@ -66,9 +64,7 @@ fn simple_git_ignore_exists() {
6664
)
6765
.unwrap();
6866

69-
cargo_process("init --lib foo --edition 2015")
70-
.env("USER", "foo")
71-
.run();
67+
cargo_process("init --lib foo --edition 2015").run();
7268

7369
assert!(paths::root().is_dir());
7470
assert!(paths::root().join("foo/Cargo.toml").is_file());
@@ -99,9 +95,7 @@ fn git_ignore_exists_no_conflicting_entries() {
9995
fs::create_dir_all(paths::root().join("foo")).unwrap();
10096
fs::write(paths::root().join("foo/.gitignore"), "**/some.file").unwrap();
10197

102-
cargo_process("init --lib foo --edition 2015")
103-
.env("USER", "foo")
104-
.run();
98+
cargo_process("init --lib foo --edition 2015").run();
10599

106100
let fp = paths::root().join("foo/.gitignore");
107101
let contents = fs::read_to_string(&fp).unwrap();
@@ -118,7 +112,6 @@ fn git_ignore_exists_no_conflicting_entries() {
118112
#[cargo_test]
119113
fn both_lib_and_bin() {
120114
cargo_process("init --lib --bin")
121-
.env("USER", "foo")
122115
.with_status(101)
123116
.with_stderr("[ERROR] can't specify both lib and binary outputs")
124117
.run();
@@ -139,15 +132,9 @@ fn bin_already_exists(explicit: bool, rellocation: &str) {
139132
fs::write(&sourcefile_path, content).unwrap();
140133

141134
if explicit {
142-
cargo_process("init --bin --vcs none")
143-
.env("USER", "foo")
144-
.cwd(&path)
145-
.run();
135+
cargo_process("init --bin --vcs none").cwd(&path).run();
146136
} else {
147-
cargo_process("init --vcs none")
148-
.env("USER", "foo")
149-
.cwd(&path)
150-
.run();
137+
cargo_process("init --vcs none").cwd(&path).run();
151138
}
152139

153140
assert!(paths::root().join("foo/Cargo.toml").is_file());
@@ -200,7 +187,6 @@ fn confused_by_multiple_lib_files() {
200187
fs::write(path2, r#" fn qqq () { println!("Hello, world 3!"); }"#).unwrap();
201188

202189
cargo_process("init --vcs none")
203-
.env("USER", "foo")
204190
.cwd(&path)
205191
.with_status(101)
206192
.with_stderr(
@@ -224,7 +210,6 @@ fn multibin_project_name_clash() {
224210
fs::write(path2, r#"fn main () { println!("Hello, world 3!"); }"#).unwrap();
225211

226212
cargo_process("init --lib --vcs none")
227-
.env("USER", "foo")
228213
.cwd(&path)
229214
.with_status(101)
230215
.with_stderr(
@@ -249,10 +234,7 @@ fn lib_already_exists(rellocation: &str) {
249234
let content = "pub fn qqq() {}";
250235
fs::write(&sourcefile_path, content).unwrap();
251236

252-
cargo_process("init --vcs none")
253-
.env("USER", "foo")
254-
.cwd(&path)
255-
.run();
237+
cargo_process("init --vcs none").cwd(&path).run();
256238

257239
assert!(paths::root().join("foo/Cargo.toml").is_file());
258240
assert!(!paths::root().join("foo/src/main.rs").is_file());
@@ -274,9 +256,7 @@ fn lib_already_exists_nosrc() {
274256

275257
#[cargo_test]
276258
fn simple_git() {
277-
cargo_process("init --lib --vcs git")
278-
.env("USER", "foo")
279-
.run();
259+
cargo_process("init --lib --vcs git").run();
280260

281261
assert!(paths::root().join("Cargo.toml").is_file());
282262
assert!(paths::root().join("src/lib.rs").is_file());
@@ -286,7 +266,7 @@ fn simple_git() {
286266

287267
#[cargo_test]
288268
fn auto_git() {
289-
cargo_process("init --lib").env("USER", "foo").run();
269+
cargo_process("init --lib").run();
290270

291271
assert!(paths::root().join("Cargo.toml").is_file());
292272
assert!(paths::root().join("src/lib.rs").is_file());
@@ -300,7 +280,6 @@ fn invalid_dir_name() {
300280
fs::create_dir_all(&foo).unwrap();
301281
cargo_process("init")
302282
.cwd(foo.clone())
303-
.env("USER", "foo")
304283
.with_status(101)
305284
.with_stderr(
306285
"\
@@ -328,7 +307,6 @@ fn reserved_name() {
328307
fs::create_dir_all(&test).unwrap();
329308
cargo_process("init")
330309
.cwd(test.clone())
331-
.env("USER", "foo")
332310
.with_status(101)
333311
.with_stderr(
334312
"\
@@ -354,7 +332,7 @@ or change the name in Cargo.toml with:
354332
fn git_autodetect() {
355333
fs::create_dir(&paths::root().join(".git")).unwrap();
356334

357-
cargo_process("init --lib").env("USER", "foo").run();
335+
cargo_process("init --lib").run();
358336

359337
assert!(paths::root().join("Cargo.toml").is_file());
360338
assert!(paths::root().join("src/lib.rs").is_file());
@@ -366,7 +344,7 @@ fn git_autodetect() {
366344
fn mercurial_autodetect() {
367345
fs::create_dir(&paths::root().join(".hg")).unwrap();
368346

369-
cargo_process("init --lib").env("USER", "foo").run();
347+
cargo_process("init --lib").run();
370348

371349
assert!(paths::root().join("Cargo.toml").is_file());
372350
assert!(paths::root().join("src/lib.rs").is_file());
@@ -380,7 +358,7 @@ fn gitignore_appended_not_replaced() {
380358

381359
fs::write(&paths::root().join(".gitignore"), "qqqqqq\n").unwrap();
382360

383-
cargo_process("init --lib").env("USER", "foo").run();
361+
cargo_process("init --lib").run();
384362

385363
assert!(paths::root().join("Cargo.toml").is_file());
386364
assert!(paths::root().join("src/lib.rs").is_file());
@@ -397,7 +375,7 @@ fn gitignore_added_newline_in_existing() {
397375

398376
fs::write(&paths::root().join(".gitignore"), "first").unwrap();
399377

400-
cargo_process("init --lib").env("USER", "foo").run();
378+
cargo_process("init --lib").run();
401379

402380
assert!(paths::root().join(".gitignore").is_file());
403381

@@ -409,7 +387,7 @@ fn gitignore_added_newline_in_existing() {
409387
fn gitignore_no_newline_in_new() {
410388
fs::create_dir(&paths::root().join(".git")).unwrap();
411389

412-
cargo_process("init --lib").env("USER", "foo").run();
390+
cargo_process("init --lib").run();
413391

414392
assert!(paths::root().join(".gitignore").is_file());
415393

@@ -423,7 +401,7 @@ fn mercurial_added_newline_in_existing() {
423401

424402
fs::write(&paths::root().join(".hgignore"), "first").unwrap();
425403

426-
cargo_process("init --lib").env("USER", "foo").run();
404+
cargo_process("init --lib").run();
427405

428406
assert!(paths::root().join(".hgignore").is_file());
429407

@@ -435,7 +413,7 @@ fn mercurial_added_newline_in_existing() {
435413
fn mercurial_no_newline_in_new() {
436414
fs::create_dir(&paths::root().join(".hg")).unwrap();
437415

438-
cargo_process("init --lib").env("USER", "foo").run();
416+
cargo_process("init --lib").run();
439417

440418
assert!(paths::root().join(".hgignore").is_file());
441419

@@ -445,9 +423,7 @@ fn mercurial_no_newline_in_new() {
445423

446424
#[cargo_test]
447425
fn terminating_newline_in_new_git_ignore() {
448-
cargo_process("init --vcs git --lib")
449-
.env("USER", "foo")
450-
.run();
426+
cargo_process("init --vcs git --lib").run();
451427

452428
let content = fs::read_to_string(&paths::root().join(".gitignore")).unwrap();
453429

@@ -461,9 +437,7 @@ fn terminating_newline_in_new_mercurial_ignore() {
461437
if !mercurial_available() {
462438
return;
463439
}
464-
cargo_process("init --vcs hg --lib")
465-
.env("USER", "foo")
466-
.run();
440+
cargo_process("init --vcs hg --lib").run();
467441

468442
let content = fs::read_to_string(&paths::root().join(".hgignore")).unwrap();
469443

@@ -477,7 +451,7 @@ fn terminating_newline_in_existing_git_ignore() {
477451
fs::create_dir(&paths::root().join(".git")).unwrap();
478452
fs::write(&paths::root().join(".gitignore"), b"first").unwrap();
479453

480-
cargo_process("init --lib").env("USER", "foo").run();
454+
cargo_process("init --lib").run();
481455

482456
let content = fs::read_to_string(&paths::root().join(".gitignore")).unwrap();
483457

@@ -491,7 +465,7 @@ fn terminating_newline_in_existing_mercurial_ignore() {
491465
fs::create_dir(&paths::root().join(".hg")).unwrap();
492466
fs::write(&paths::root().join(".hgignore"), b"first").unwrap();
493467

494-
cargo_process("init --lib").env("USER", "foo").run();
468+
cargo_process("init --lib").run();
495469

496470
let content = fs::read_to_string(&paths::root().join(".hgignore")).unwrap();
497471

@@ -504,9 +478,7 @@ fn terminating_newline_in_existing_mercurial_ignore() {
504478
fn cargo_lock_gitignored_if_lib1() {
505479
fs::create_dir(&paths::root().join(".git")).unwrap();
506480

507-
cargo_process("init --lib --vcs git")
508-
.env("USER", "foo")
509-
.run();
481+
cargo_process("init --lib --vcs git").run();
510482

511483
assert!(paths::root().join(".gitignore").is_file());
512484

@@ -520,7 +492,7 @@ fn cargo_lock_gitignored_if_lib2() {
520492

521493
fs::write(&paths::root().join("lib.rs"), "").unwrap();
522494

523-
cargo_process("init --vcs git").env("USER", "foo").run();
495+
cargo_process("init --vcs git").run();
524496

525497
assert!(paths::root().join(".gitignore").is_file());
526498

@@ -532,9 +504,7 @@ fn cargo_lock_gitignored_if_lib2() {
532504
fn cargo_lock_not_gitignored_if_bin1() {
533505
fs::create_dir(&paths::root().join(".git")).unwrap();
534506

535-
cargo_process("init --vcs git --bin")
536-
.env("USER", "foo")
537-
.run();
507+
cargo_process("init --vcs git --bin").run();
538508

539509
assert!(paths::root().join(".gitignore").is_file());
540510

@@ -548,7 +518,7 @@ fn cargo_lock_not_gitignored_if_bin2() {
548518

549519
fs::write(&paths::root().join("main.rs"), "").unwrap();
550520

551-
cargo_process("init --vcs git").env("USER", "foo").run();
521+
cargo_process("init --vcs git").run();
552522

553523
assert!(paths::root().join(".gitignore").is_file());
554524

@@ -558,9 +528,7 @@ fn cargo_lock_not_gitignored_if_bin2() {
558528

559529
#[cargo_test]
560530
fn with_argument() {
561-
cargo_process("init foo --vcs none")
562-
.env("USER", "foo")
563-
.run();
531+
cargo_process("init foo --vcs none").run();
564532
assert!(paths::root().join("foo/Cargo.toml").is_file());
565533
}
566534

@@ -595,7 +563,6 @@ fn formats_source() {
595563
fs::write(&paths::root().join("rustfmt.toml"), "tab_spaces = 2").unwrap();
596564

597565
cargo_process("init --lib")
598-
.env("USER", "foo")
599566
.with_stderr("[CREATED] library package")
600567
.run();
601568

@@ -615,7 +582,6 @@ mod tests {
615582
#[cargo_test]
616583
fn ignores_failure_to_format_source() {
617584
cargo_process("init --lib")
618-
.env("USER", "foo")
619585
.env("PATH", "") // pretend that `rustfmt` is missing
620586
.with_stderr("[CREATED] library package")
621587
.run();

0 commit comments

Comments
 (0)