Merge pull request #252 from peret/fix-multiple-names-test

Fix multiple names test

Author: KSXGitHub

tests/_utils.rs

@@ -74,16 +74,16 @@ impl Default for SampleWorkspace {
     fn default() -> Self {
         let temp = Temp::new_dir().expect("create working directory for sample workspace");
 
-        MergeableFileSystemTree::<&str, &str>::from(dir! {
+        MergeableFileSystemTree::<&str, String>::from(dir! {
             "flat" => dir! {
                 "0" => file!("")
-                "1" => file!("a")
-                "2" => file!("ab")
-                "3" => file!("abc")
+                "1" => file!("a".repeat(1000))
+                "2" => file!("a".repeat(2000))
+                "3" => file!("a".repeat(3000))
             }
             "nested" => dir! {
                 "0" => dir! {
-                    "1" => file!("abcdef")
+                    "1" => file!("a".repeat(5000))
                 }
             }
             "empty-dir" => dir! {}

tests/usual_cli.rs

@@ -617,7 +617,6 @@ fn multiple_names() {
                 reporter: ErrorOnlyReporter::new(ErrorReport::SILENT),
             };
             let mut data_tree: DataTree<OsStringDisplay, _> = builder.into();
-            data_tree.par_sort_by(|left, right| left.data().cmp(&right.data()).reverse());
             *data_tree.name_mut() = OsStringDisplay::os_string_from(name);
             data_tree
         })
@@ -627,7 +626,8 @@ fn multiple_names() {
                 0.into(),
                 children.collect(),
             )
-        });
+        })
+        .into_par_sorted(|left, right| left.data().cmp(&right.data()).reverse());
     data_tree.par_cull_insignificant_data(0.01);
     let visualizer = Visualizer::<OsStringDisplay, _> {
         data_tree: &data_tree,