RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-27 19:17:43 +00:00
Code Issues Activity

test_test: Simplify test_file_N

Browse source 
I found the logic a little difficult to understand, and the
comment probably doesn't match what `-N` is supposed to do?

Intead, let's just manually set mtime and atime.

Hopefully this helps clear up Android flakiness in #7570. Or at
least understand better what is going on.
This commit is contained in:
Nicolas Boichat 2025-03-25 15:01:23 +01:00
parent 2488e17aef
commit 6b8135119c
1 changed files with 19 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

23
tests/by-util/test_test.rs
View file

@ -918,16 +918,31 @@ fn test_bracket_syntax_version() {
#[allow(non_snake_case)]
#[cfg(unix)]
fn test_file_N() {
use std::{fs::FileTimes, time::Duration};
let scene = TestScenario::new(util_name!());
let at = &scene.fixtures;
let f = at.make_file("file");
f.set_modified(std::time::UNIX_EPOCH).unwrap();
// Set the times so that the file is accessed _after_ being modified
// => test -N return false.
let times = FileTimes::new()
.set_accessed(std::time::UNIX_EPOCH + Duration::from_secs(123))
.set_modified(std::time::UNIX_EPOCH);
f.set_times(times).unwrap();
// TODO: stat call for debugging #7570, remove?
println!("{}", scene.cmd_shell("stat file").succeeds().stdout_str());
scene.ucmd().args(&["-N", "file"]).fails();
// The file will have different create/modified data
// so, test -N will return 0
at.touch("file");
// Set the times so that the file is modified _after_ being accessed
// => test -N return true.
let times = FileTimes::new()
.set_accessed(std::time::UNIX_EPOCH)
.set_modified(std::time::UNIX_EPOCH + Duration::from_secs(123));
f.set_times(times).unwrap();
// TODO: stat call for debugging #7570, remove?
println!("{}", scene.cmd_shell("stat file").succeeds().stdout_str());
scene.ucmd().args(&["-N", "file"]).succeeds();
}