Skip to content

Commit 5c33532

Browse files
committed
refactor(test_suite): address the clippy::uninlined_format_args lint
1 parent 064e6f7 commit 5c33532

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

src/r3_test_suite/build.rs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,8 @@ fn main() {
4343
kernel_benchmark_list.push(TestMeta("kernel_benchmarks", name));
4444
} else {
4545
panic!(
46-
"Unrecognized test type: `{}`
46+
"Unrecognized test type: `{selected_test}`
4747
Test names should start with a prefix like `kernel_tests::`.",
48-
selected_test
4948
);
5049
}
5150
}
@@ -84,9 +83,8 @@ fn main() {
8483
fn expect_valid_test_name(name: &str) {
8584
if name.contains(|c: char| !c.is_alphanumeric() && c != '_') || name.is_empty() {
8685
panic!(
87-
"Invalid test name: `{}`
86+
"Invalid test name: `{name}`
8887
Test names should match /[a-zA-Z0-9_]+/",
89-
name
9088
)
9189
}
9290
}

src/r3_test_suite/src/utils.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ impl SeqTracker {
3737
// Note: Some targets don't support CAS atomics
3838
let got = self.counter.load(Ordering::Relaxed);
3939
log::debug!("{} (expected: {}) → {}", got, old, new);
40-
assert_eq!(got, old, "expected {}, got {}", old, got);
40+
assert_eq!(got, old, "expected {old}, got {got}");
4141
self.counter.store(new, Ordering::Relaxed);
4242
}
4343
}

src/r3_test_suite/src/utils/conditional.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,9 +86,7 @@ pub trait KernelTimeExt: traits::KernelBase {
8686
log::trace!("time = {:?}ms (expected = {:?}ms)", t, range);
8787
assert!(
8888
range.contains(&t),
89-
"time = {:?}ms (expected = {:?}ms)",
90-
t,
91-
range
89+
"time = {t:?}ms (expected = {range:?}ms)"
9290
);
9391
}
9492

0 commit comments

Comments
 (0)