Skip to content

Commit 3822401

Browse files
committed
timeout: remove FIXME in test
This FIXME comment was added in 2021 ( 5431e94 ). `timeout` is already in feat_require_unix_core, so having `true` and `false` on the machine running the test is quite reasonable and does not warrant a FIXME. Signed-off-by: Etienne Cordonnier <ecordonnier@snap.com>
1 parent 5b261bc commit 3822401

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

tests/by-util/test_timeout.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,6 @@ fn test_invalid_arg() {
1515
new_ucmd!().arg("--definitely-invalid").fails_with_code(125);
1616
}
1717

18-
// FIXME: this depends on the system having true and false in PATH
19-
// the best solution is probably to generate some test binaries that we can call for any
20-
// utility that requires executing another program (kill, for instance)
2118
#[test]
2219
fn test_subcommand_return_code() {
2320
new_ucmd!().arg("1").arg("true").succeeds();

0 commit comments

Comments
 (0)