test: add failing test for problem

Author: c-git

Cargo.lock

@@ -48,6 +48,8 @@ opt-level = 2
 
 [dev-dependencies]
 insta = { version = "1.34.0", features = ["ron", "glob", "yaml"] }
+pretty_assertions = "1.4.0"
+ron = "0.8.1"
 rstest = "0.18.2"
 
 # If you want to use the bleeding edge version of egui and eframe:

Cargo.toml

@@ -14,7 +14,7 @@ const SPACE_BETWEEN_TABLES: f32 = 10.;
 #[serde(default)] // if we add new fields, give them default values when deserializing old state
 pub struct LogViewerApp {
     #[serde(skip)]
-    // TODO 1 Fix issue where if data is included in the save load fails? (Need to check if it is failing and check if we can do a round trip on deserialize then serialize then deserialize)
+    // TODO 1 Fix issue where if data is included in the load fails
     // TODO 2 after fixing issue with serializing and deserializing then change to only a map so access pattern can be consistent
     data: Option<Data>,
     details_size: f32,

src/app.rs

@@ -2,13 +2,13 @@ use std::collections::BTreeMap;
 
 use anyhow::Context;
 
-#[derive(serde::Deserialize, serde::Serialize, Default, Debug)]
+#[derive(serde::Deserialize, serde::Serialize, Default, Debug, PartialEq, Eq)]
 pub struct Data {
     pub selected_row: Option<usize>,
     rows: Vec<LogRow>,
 }
 
-#[derive(serde::Deserialize, serde::Serialize, Default, Debug)]
+#[derive(serde::Deserialize, serde::Serialize, Default, Debug, PartialEq, Eq)]
 pub struct LogRow {
     time: Option<String>,
     request_id: Option<String>,
@@ -80,6 +80,7 @@ impl TryFrom<&str> for Data {
 #[cfg(test)]
 mod tests {
     use insta::glob;
+    use pretty_assertions::assert_eq;
     use rstest::{fixture, rstest};
 
     use super::*;
@@ -104,4 +105,24 @@ mod tests {
             insta_settings.bind(|| insta::assert_ron_snapshot!(data));
         });
     }
+
+    #[rstest]
+    fn round_trip() {
+        glob!(PATH_PROJECT_ROOT, PATH_TEST_SAMPLES, |path| {
+            let input = std::fs::read_to_string(path).unwrap();
+            let rows_before = Data::try_from(&input[..]).unwrap();
+
+            // Test single row
+            for row_before in rows_before.rows() {
+                let as_ron = ron::to_string(&row_before).unwrap();
+                let row_after: LogRow = ron::from_str(&dbg!(as_ron)).unwrap();
+                assert_eq!(&row_after, row_before);
+            }
+
+            // Test all rows
+            let as_ron = ron::to_string(&rows_before).unwrap();
+            let rows_after: Data = ron::from_str(&dbg!(as_ron)).unwrap();
+            assert_eq!(rows_after, rows_before);
+        });
+    }
 }