Skip to content

Commit 0bd8527

Browse files
committed
Update tests for exit
1 parent 62483d7 commit 0bd8527

File tree

5 files changed

+32
-6
lines changed

5 files changed

+32
-6
lines changed

tests/ui/exit.rs renamed to tests/ui/exit1.rs

-5
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,10 @@ fn not_main() {
66
}
77
}
88

9-
fn also_not_main() {
10-
std::process::exit(3);
11-
}
12-
139
fn main() {
1410
if true {
1511
std::process::exit(2);
1612
};
17-
also_not_main();
1813
not_main();
1914
std::process::exit(1);
2015
}

tests/ui/exit1.stderr

+10
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
error: usage of `process::exit`
2+
--> $DIR/exit1.rs:5:9
3+
|
4+
LL | std::process::exit(4);
5+
| ^^^^^^^^^^^^^^^^^^^^^
6+
|
7+
= note: `-D clippy::exit` implied by `-D warnings`
8+
9+
error: aborting due to previous error
10+

tests/ui/exit2.rs

+13
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
#[warn(clippy::exit)]
2+
3+
fn also_not_main() {
4+
std::process::exit(3);
5+
}
6+
7+
fn main() {
8+
if true {
9+
std::process::exit(2);
10+
};
11+
also_not_main();
12+
std::process::exit(1);
13+
}

tests/ui/exit.stderr renamed to tests/ui/exit2.stderr

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
error: usage of `process::exit`
2-
--> $DIR/exit.rs:3:5
2+
--> $DIR/exit2.rs:4:5
33
|
44
LL | std::process::exit(3);
55
| ^^^^^^^^^^^^^^^^^^^^^

tests/ui/exit3.rs

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
#[warn(clippy::exit)]
2+
3+
fn main() {
4+
if true {
5+
std::process::exit(2);
6+
};
7+
std::process::exit(1);
8+
}

0 commit comments

Comments
 (0)