From e0058fcb86b74c575b1a82b9b440ee4af8bcf606 Mon Sep 17 00:00:00 2001
From: Thomas Jung <taj@posteo.net>
Date: Wed, 14 Feb 2024 20:04:44 +0100
Subject: [PATCH] Use sets in the tests for expected and actual files to make
 those stable.

---
 czkawka_core/src/common_dir_traversal.rs | 24 +++++++++++-------------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/czkawka_core/src/common_dir_traversal.rs b/czkawka_core/src/common_dir_traversal.rs
index 26fcdc7..c81dde5 100644
--- a/czkawka_core/src/common_dir_traversal.rs
+++ b/czkawka_core/src/common_dir_traversal.rs
@@ -60,7 +60,7 @@ pub enum CheckingMethod {
     AudioContent,
 }
 
-#[derive(Clone, Debug, Default, PartialEq, Eq, Serialize, Deserialize)]
+#[derive(Clone, Debug, Default, PartialEq, Eq, Hash, Serialize, Deserialize)]
 pub struct FileEntry {
     pub path: PathBuf,
     pub size: u64,
@@ -677,6 +677,7 @@ mod tests {
     use super::*;
     use crate::common_tool::*;
     use once_cell::sync::Lazy;
+    use std::collections::HashSet;
     use std::fs;
     use std::fs::File;
     use std::io;
@@ -725,9 +726,9 @@ mod tests {
                 warnings: _,
                 grouped_file_entries,
             } => {
-                let actual: Vec<_> = grouped_file_entries.into_values().flatten().collect();
+                let actual: HashSet<_> = grouped_file_entries.into_values().flatten().collect();
                 assert_eq!(
-                    [
+                    HashSet::from([
                         FileEntry {
                             path: src,
                             size: 1,
@@ -743,8 +744,7 @@ mod tests {
                             size: 1,
                             modified_date: secs,
                         },
-                    ]
-                    .to_vec(),
+                    ]),
                     actual
                 );
             }
@@ -771,9 +771,9 @@ mod tests {
                 warnings: _,
                 grouped_file_entries,
             } => {
-                let actual: Vec<_> = grouped_file_entries.into_iter().flat_map(take_1_per_inode).collect();
+                let actual: HashSet<_> = grouped_file_entries.into_iter().flat_map(take_1_per_inode).collect();
                 assert_eq!(
-                    [
+                    HashSet::from([
                         FileEntry {
                             path: src,
                             size: 1,
@@ -784,8 +784,7 @@ mod tests {
                             size: 1,
                             modified_date: secs,
                         },
-                    ]
-                    .to_vec(),
+                    ]),
                     actual
                 );
             }
@@ -812,9 +811,9 @@ mod tests {
                 warnings: _,
                 grouped_file_entries,
             } => {
-                let actual: Vec<_> = grouped_file_entries.into_iter().flat_map(take_1_per_inode).collect();
+                let actual: HashSet<_> = grouped_file_entries.into_iter().flat_map(take_1_per_inode).collect();
                 assert_eq!(
-                    [
+                    HashSet::from([
                         FileEntry {
                             path: src,
                             size: 1,
@@ -830,8 +829,7 @@ mod tests {
                             size: 1,
                             modified_date: secs,
                         },
-                    ]
-                    .to_vec(),
+                    ]),
                     actual
                 );
             }