fixup! Add servo as browser option

Re-flow doc. commentary for `read_files_at`.
erichdongubler-mozilla · May 10, 2024 · 3fa0570 · 3fa0570
1 parent 548610f
commit 3fa0570
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/moz-webgpu-cts/src/main.rs b/moz-webgpu-cts/src/main.rs
@@ -1425,8 +1425,8 @@ fn render_metadata_parse_errors<'a>(
 }
 
 /// Returns a "naturally" sorted list of files found by searching for `glob_pattern` in `base`.
-/// `checkout` is stripped as a prefix from the absolute paths recorded into `log` entries
-/// emitted by this function.
+/// `checkout` is stripped as a prefix from the absolute paths recorded into `log` entries emitted
+/// by this function.
 ///
 /// # Returns
 ///

diff --git a/moz-webgpu-cts/src/shared.rs b/moz-webgpu-cts/src/shared.rs
@@ -615,11 +615,15 @@ impl<'a> TestPath<'a> {
             scope,
         } = self;
 
-        let scope_dir = match (scope.browser, scope.visibility) {
+        let TestScope {
+            browser,
+            visibility,
+        } = scope;
+        let scope_dir = match (browser, visibility) {
             (Browser::Firefox, TestVisibility::Public) => SCOPE_DIR_FX_PUBLIC_COMPONENTS,
             (Browser::Firefox, TestVisibility::Private) => SCOPE_DIR_FX_PRIVATE_COMPONENTS,
             (Browser::Servo, TestVisibility::Public) => SCOPE_DIR_SERVO_PUBLIC_COMPONENTS,
-            (Browser::Servo, _) => todo!(),
+            (Browser::Servo, TestVisibility::Private) => todo!(),
         }
         .iter()
         .chain(&["meta"])