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

tests: add template string to assert! statements

Browse source 
Add missing "{}" template strings to `assert!()` statements.
This commit is contained in:
Jeffrey Finkelstein 2021-10-11 16:51:14 -04:00
parent 11ca4be1aa
commit 9e21d26b7c
1 changed files with 10 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

16
tests/common/util.rs
View file

@ -864,7 +864,7 @@ impl UCommand {
/// Add a parameter to the invocation. Path arguments are treated relative /// Add a parameter to the invocation. Path arguments are treated relative
/// to the test environment directory. /// to the test environment directory.
pub fn arg<S: AsRef<OsStr>>(&mut self, arg: S) -> &mut UCommand { pub fn arg<S: AsRef<OsStr>>(&mut self, arg: S) -> &mut UCommand {
assert!(!self.has_run, ALREADY_RUN); assert!(!self.has_run, "{}", ALREADY_RUN);
self.comm_string.push(' '); self.comm_string.push(' ');
self.comm_string self.comm_string
.push_str(arg.as_ref().to_str().unwrap_or_default()); .push_str(arg.as_ref().to_str().unwrap_or_default());
@ -875,7 +875,7 @@ impl UCommand {
/// Add multiple parameters to the invocation. Path arguments are treated relative /// Add multiple parameters to the invocation. Path arguments are treated relative
/// to the test environment directory. /// to the test environment directory.
pub fn args<S: AsRef<OsStr>>(&mut self, args: &[S]) -> &mut UCommand { pub fn args<S: AsRef<OsStr>>(&mut self, args: &[S]) -> &mut UCommand {
assert!(!self.has_run, MULTIPLE_STDIN_MEANINGLESS); assert!(!self.has_run, "{}", MULTIPLE_STDIN_MEANINGLESS);
let strings = args let strings = args
.iter() .iter()
.map(|s| s.as_ref().to_os_string()) .map(|s| s.as_ref().to_os_string())
@ -893,7 +893,11 @@ impl UCommand {
/// provides standard input to feed in to the command when spawned /// provides standard input to feed in to the command when spawned
pub fn pipe_in<T: Into<Vec<u8>>>(&mut self, input: T) -> &mut UCommand { pub fn pipe_in<T: Into<Vec<u8>>>(&mut self, input: T) -> &mut UCommand {
assert!(!self.bytes_into_stdin.is_some(), MULTIPLE_STDIN_MEANINGLESS); assert!(
!self.bytes_into_stdin.is_some(),
"{}",
MULTIPLE_STDIN_MEANINGLESS
);
self.bytes_into_stdin = Some(input.into()); self.bytes_into_stdin = Some(input.into());
self self
} }
@ -908,7 +912,7 @@ impl UCommand {
/// This is typically useful to test non-standard workflows /// This is typically useful to test non-standard workflows
/// like feeding something to a command that does not read it /// like feeding something to a command that does not read it
pub fn ignore_stdin_write_error(&mut self) -> &mut UCommand { pub fn ignore_stdin_write_error(&mut self) -> &mut UCommand {
assert!(!self.bytes_into_stdin.is_none(), NO_STDIN_MEANINGLESS); assert!(!self.bytes_into_stdin.is_none(), "{}", NO_STDIN_MEANINGLESS);
self.ignore_stdin_write_error = true; self.ignore_stdin_write_error = true;
self self
} }
@ -918,7 +922,7 @@ impl UCommand {
K: AsRef<OsStr>, K: AsRef<OsStr>,
V: AsRef<OsStr>, V: AsRef<OsStr>,
{ {
assert!(!self.has_run, ALREADY_RUN); assert!(!self.has_run, "{}", ALREADY_RUN);
self.raw.env(key, val); self.raw.env(key, val);
self self
} }
@ -937,7 +941,7 @@ impl UCommand {
/// Spawns the command, feeds the stdin if any, and returns the /// Spawns the command, feeds the stdin if any, and returns the
/// child process immediately. /// child process immediately.
pub fn run_no_wait(&mut self) -> Child { pub fn run_no_wait(&mut self) -> Child {
assert!(!self.has_run, ALREADY_RUN); assert!(!self.has_run, "{}", ALREADY_RUN);
self.has_run = true; self.has_run = true;
log_info("run", &self.comm_string); log_info("run", &self.comm_string);
let mut child = self let mut child = self