Merge pull request #146 from ErichDonGubler/erichdongubler/push-ouwllslxyvot

ErichDonGubler · web-flow · commit 1e424e7fff2d · 2024-08-05T17:46:49.000-04:00
fix: actually format the fmt. str. in `Subtests::add_subtest` 😅
diff --git a/moz-webgpu-cts/src/wpt/metadata.rs b/moz-webgpu-cts/src/wpt/metadata.rs
@@ -677,7 +677,10 @@ impl<'a> metadata::Subtests<'a> for Subtests {
     ) {
         let Self(subtests) = self;
         if subtests.get(&name).is_some() {
-            emitter.emit(Rich::custom(span, "duplicate subtest {name:?}"));
+            emitter.emit(Rich::custom(
+                span,
+                format_args!("duplicate subtest {name:?}"),
+            ));
         }
         subtests.insert(name, subtest);
     }

Original file line number	Diff line number	Diff line change
`@@ -677,7 +677,10 @@ impl<'a> metadata::Subtests<'a> for Subtests {`
`677`	`677`	`) {`
`678`	`678`	`let Self(subtests) = self;`
`679`	`679`	`if subtests.get(&name).is_some() {`
`680`		`- emitter.emit(Rich::custom(span, "duplicate subtest {name:?}"));`
	`680`	`+ emitter.emit(Rich::custom(`
	`681`	`+ span,`
	`682`	`+ format_args!("duplicate subtest {name:?}"),`
	`683`	`+ ));`
`681`	`684`	`}`
`682`	`685`	`subtests.insert(name, subtest);`
`683`	`686`	`}`