refactor(tests): use builder API for FsTreeBuilder, TreeBuilder, and Visualizer in test files

Co-authored-by: KSXGitHub <11488886+KSXGitHub@users.noreply.github.com>

Author: Copilot

tests/_utils.rs

@@ -366,18 +366,18 @@ where
     }
 
     let measure = |suffix: &str| {
-        FsTreeBuilder {
-            size_getter,
-            hardlinks_recorder: &HardlinkIgnorant,
-            reporter: &ErrorOnlyReporter::new(|error| {
+        FsTreeBuilder::builder()
+            .size_getter(size_getter)
+            .hardlinks_recorder(&HardlinkIgnorant)
+            .reporter(&ErrorOnlyReporter::new(|error| {
                 panic!("Unexpected call to report_error: {error:?}")
-            }),
-            root: root.join(suffix),
-            max_depth: 10,
-        }
-        .pipe(DataTree::<OsStringDisplay, Size>::from)
-        .into_par_sorted(|left, right| left.name().cmp(right.name()))
-        .into_reflection()
+            }))
+            .root(root.join(suffix))
+            .max_depth(10)
+            .build()
+            .pipe(DataTree::<OsStringDisplay, Size>::from)
+            .into_par_sorted(|left, right| left.name().cmp(right.name()))
+            .into_reflection()
     };
 
     let sub = |suffix: &str| root.join(suffix).pipe(OsStringDisplay::os_string_from);

tests/cli_errors.rs

@@ -130,23 +130,24 @@ fn fs_errors() {
     dbg!(&status);
     eprintln!("STDERR+STDOUT:\n{stderr}{stdout}\n");
 
-    let builder = FsTreeBuilder {
-        root: workspace.to_path_buf(),
-        size_getter: GetApparentSize,
-        hardlinks_recorder: &HardlinkIgnorant,
-        reporter: &ErrorOnlyReporter::new(ErrorReport::SILENT),
-        max_depth: 10,
-    };
+    let reporter = ErrorOnlyReporter::new(ErrorReport::SILENT);
+    let builder = FsTreeBuilder::builder()
+        .root(workspace.to_path_buf())
+        .size_getter(GetApparentSize)
+        .hardlinks_recorder(&HardlinkIgnorant)
+        .reporter(&reporter)
+        .max_depth(10)
+        .build();
     let mut data_tree: DataTree<OsStringDisplay, _> = builder.into();
     data_tree.par_sort_by(|left, right| left.size().cmp(&right.size()).reverse());
     *data_tree.name_mut() = OsStringDisplay::os_string_from(".");
-    let visualizer = Visualizer {
-        data_tree: &data_tree,
-        bytes_format: BytesFormat::MetricUnits,
-        direction: Direction::BottomUp,
-        bar_alignment: BarAlignment::Left,
-        column_width_distribution: ColumnWidthDistribution::total(100),
-    };
+    let visualizer = Visualizer::builder()
+        .data_tree(&data_tree)
+        .bytes_format(BytesFormat::MetricUnits)
+        .direction(Direction::BottomUp)
+        .bar_alignment(BarAlignment::Left)
+        .column_width_distribution(ColumnWidthDistribution::total(100))
+        .build();
     let expected_stdout = format!("{visualizer}");
     eprintln!("EXPECTED STDOUT:\n{}\n", &expected_stdout);
 

tests/json.rs

@@ -80,13 +80,14 @@ fn json_output() {
         .tree
         .pipe(sanitize_tree_reflection);
     dbg!(&actual);
-    let builder = FsTreeBuilder {
-        root: workspace.to_path_buf(),
-        size_getter: GetApparentSize,
-        hardlinks_recorder: &HardlinkIgnorant,
-        reporter: &ErrorOnlyReporter::new(ErrorReport::SILENT),
-        max_depth: 10,
-    };
+    let reporter = ErrorOnlyReporter::new(ErrorReport::SILENT);
+    let builder = FsTreeBuilder::builder()
+        .root(workspace.to_path_buf())
+        .size_getter(GetApparentSize)
+        .hardlinks_recorder(&HardlinkIgnorant)
+        .reporter(&reporter)
+        .max_depth(10)
+        .build();
     let expected = builder
         .pipe(DataTree::<_, Bytes>::from)
         .into_reflection()
@@ -135,13 +136,14 @@ fn json_input() {
     let actual = actual.trim_end();
     eprintln!("ACTUAL:\n{actual}\n");
 
-    let visualizer = Visualizer {
-        data_tree: &sample_tree(),
-        bytes_format: BytesFormat::MetricUnits,
-        direction: Direction::BottomUp,
-        bar_alignment: BarAlignment::Left,
-        column_width_distribution: ColumnWidthDistribution::total(100),
-    };
+    let tree = sample_tree();
+    let visualizer = Visualizer::builder()
+        .data_tree(&tree)
+        .bytes_format(BytesFormat::MetricUnits)
+        .direction(Direction::BottomUp)
+        .bar_alignment(BarAlignment::Left)
+        .column_width_distribution(ColumnWidthDistribution::total(100))
+        .build();
     let expected = format!("{visualizer}");
     let expected = expected.trim_end();
     eprintln!("EXPECTED:\n{expected}\n");

tests/tree_builder.rs

@@ -39,10 +39,10 @@ impl SampleTree {
     }
 
     fn tree(&self, root: &'static str) -> DataTree<SampleName, SampleData> {
-        TreeBuilder {
-            path: root.to_string(),
-            name: root.to_string(),
-            get_info: |path| {
+        TreeBuilder::builder()
+            .path(root.to_string())
+            .name(root.to_string())
+            .get_info(|path| {
                 let path: Vec<_> = path
                     .split(SAMPLE_SEPARATOR)
                     .map(ToString::to_string)
@@ -56,12 +56,12 @@ impl SampleTree {
                     )),
                     None => panic!("Path does not exist"),
                 }
-            },
-            join_path: |prefix, name| format!("{prefix}{SAMPLE_SEPARATOR}{name}"),
-            max_depth: 10,
-        }
-        .pipe(DataTree::from)
-        .into_par_sorted(|left, right| left.name().as_str().cmp(right.name().as_str()))
+            })
+            .join_path(|prefix, name| format!("{prefix}{SAMPLE_SEPARATOR}{name}"))
+            .max_depth(10)
+            .build()
+            .pipe(DataTree::from)
+            .into_par_sorted(|left, right| left.name().as_str().cmp(right.name().as_str()))
     }
 }