Create test cases to demonstrate lossy map order

David Orchard · David Orchard · commit 62c84297e534 · 2021-08-15T10:31:10.000-07:00
diff --git a/tests/Settings.hjson b/tests/Settings.hjson
@@ -11,6 +11,8 @@
     rating: 4.5
     creator: {
       name: John Smith
+      username: jsmith
+      email: jsmith@localhost
     }
   }
 }
diff --git a/tests/Settings.json b/tests/Settings.json
@@ -11,7 +11,9 @@
     "reviews": 3866,
     "rating": 4.5,
     "creator": {
-      "name": "John Smith"
+      "name": "John Smith",
+      "username": "jsmith",
+      "email": "jsmith@localhost"
     }
   }
 }
diff --git a/tests/Settings.json5 b/tests/Settings.json5
@@ -13,6 +13,8 @@
     rating: 4.5,
     creator: {
       name: "John Smith",
+      "username": "jsmith",
+      "email": "jsmith@localhost",
     }
   }
 }
diff --git a/tests/Settings.ron b/tests/Settings.ron
@@ -12,7 +12,9 @@
     rating: Some(4.5),
     telephone: None,
     creator: {
-      "name": "John Smith"
+      "name": "John Smith",
+      "username": "jsmith",
+      "email": "jsmith@localhost"
     }
   )
 )
diff --git a/tests/Settings.toml b/tests/Settings.toml
@@ -41,6 +41,8 @@ rating = 4.5
 
 [place.creator]
 name = "John Smith"
+username = "jsmith"
+email = "jsmith@localhost"
 
 [proton]
 up = 2
diff --git a/tests/Settings.yaml b/tests/Settings.yaml
@@ -10,3 +10,5 @@ place:
   rating: 4.5
   creator:
     name: John Smith
+    username: jsmith
+    email: jsmith@localhost
diff --git a/tests/file_hjson.rs b/tests/file_hjson.rs
@@ -60,8 +60,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/file_json.rs b/tests/file_json.rs
@@ -60,8 +60,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/file_json5.rs b/tests/file_json5.rs
@@ -59,8 +59,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/file_ron.rs b/tests/file_ron.rs
@@ -62,8 +62,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/file_toml.rs b/tests/file_toml.rs
@@ -71,8 +71,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/file_yaml.rs b/tests/file_yaml.rs
@@ -60,8 +60,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/get.rs b/tests/get.rs
@@ -122,8 +122,14 @@ fn test_map_str() {
     let c = make();
     let m: HashMap<String, String> = c.get("place.creator").unwrap();
 
-    assert_eq!(m.len(), 1);
-    assert_eq!(m["name"], "John Smith".to_string());
+    assert_eq!(
+        m.into_iter().collect::<Vec<(String, String)>>(),
+        vec![
+            ("name".to_string(), "John Smith".to_string()),
+            ("username".to_string(), "jsmith".to_string()),
+            ("email".to_string(), "jsmith@localhost".to_string()),
+        ]
+    );
 }
 
 #[test]
diff --git a/tests/legacy/file_hjson.rs b/tests/legacy/file_hjson.rs
@@ -58,8 +58,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/legacy/file_json.rs b/tests/legacy/file_json.rs
@@ -58,8 +58,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/legacy/file_ron.rs b/tests/legacy/file_ron.rs
@@ -60,8 +60,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/legacy/file_toml.rs b/tests/legacy/file_toml.rs
@@ -69,8 +69,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/legacy/file_yaml.rs b/tests/legacy/file_yaml.rs
@@ -58,8 +58,15 @@ fn test_file() {
     assert_eq!(s.elements.len(), 10);
     assert_eq!(s.elements[3], "4".to_string());
     assert_eq!(
-        s.place.creator["name"].clone().into_string().unwrap(),
-        "John Smith".to_string()
+        s.place
+            .creator
+            .into_iter()
+            .collect::<Vec<(String, config::Value)>>(),
+        vec![
+            ("name".to_string(), "John Smith".into()),
+            ("username".into(), "jsmith".into()),
+            ("email".into(), "jsmith@localhost".into()),
+        ]
     );
 }
 
diff --git a/tests/legacy/get.rs b/tests/legacy/get.rs
@@ -120,8 +120,14 @@ fn test_map_str() {
     let c = make();
     let m: HashMap<String, String> = c.get("place.creator").unwrap();
 
-    assert_eq!(m.len(), 1);
-    assert_eq!(m["name"], "John Smith".to_string());
+    assert_eq!(
+        m.into_iter().collect::<Vec<(String, String)>>(),
+        vec![
+            ("name".to_string(), "John Smith".to_string()),
+            ("username".to_string(), "jsmith".to_string()),
+            ("email".to_string(), "jsmith@localhost".to_string()),
+        ]
+    );
 }
 
 #[test]
diff --git a/tests/legacy/merge.rs b/tests/legacy/merge.rs
@@ -2,6 +2,8 @@
 
 extern crate config;
 
+use std::collections::HashMap;
+
 use self::config::*;
 
 fn make() -> Config {
@@ -21,11 +23,17 @@ fn test_merge() {
 
     assert_eq!(c.get("debug").ok(), Some(false));
     assert_eq!(c.get("production").ok(), Some(true));
+    assert_eq!(c.get("place.rating").ok(), Some(4.9));
+
+    let m: HashMap<String, String> = c.get("place.creator").unwrap();
     assert_eq!(
-        c.get("place.creator.name").ok(),
-        Some("Somebody New".to_string())
+        m.into_iter().collect::<Vec<(String, String)>>(),
+        vec![
+            ("name".to_string(), "Somebody New".to_string()),
+            ("username".to_string(), "jsmith".to_string()),
+            ("email".to_string(), "jsmith@localhost".to_string()),
+        ]
     );
-    assert_eq!(c.get("place.rating").ok(), Some(4.9));
 }
 
 #[test]
diff --git a/tests/merge.rs b/tests/merge.rs
@@ -2,6 +2,8 @@
 
 extern crate config;
 
+use std::collections::HashMap;
+
 use config::*;
 
 fn make() -> Config {
@@ -18,11 +20,17 @@ fn test_merge() {
 
     assert_eq!(c.get("debug").ok(), Some(false));
     assert_eq!(c.get("production").ok(), Some(true));
+    assert_eq!(c.get("place.rating").ok(), Some(4.9));
+
+    let m: HashMap<String, String> = c.get("place.creator").unwrap();
     assert_eq!(
-        c.get("place.creator.name").ok(),
-        Some("Somebody New".to_string())
+        m.into_iter().collect::<Vec<(String, String)>>(),
+        vec![
+            ("name".to_string(), "Somebody New".to_string()),
+            ("username".to_string(), "jsmith".to_string()),
+            ("email".to_string(), "jsmith@localhost".to_string()),
+        ]
     );
-    assert_eq!(c.get("place.rating").ok(), Some(4.9));
 }
 
 #[test]

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,8 @@`
`11`	`11`	`rating: 4.5`
`12`	`12`	`creator: {`
`13`	`13`	`name: John Smith`
	`14`	`+ username: jsmith`
	`15`	`+ email: jsmith@localhost`
`14`	`16`	`}`
`15`	`17`	`}`
`16`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,9 @@`
`11`	`11`	`"reviews": 3866,`
`12`	`12`	`"rating": 4.5,`
`13`	`13`	`"creator": {`
`14`		`- "name": "John Smith"`
	`14`	`+ "name": "John Smith",`
	`15`	`+ "username": "jsmith",`
	`16`	`+ "email": "jsmith@localhost"`
`15`	`17`	`}`
`16`	`18`	`}`
`17`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,8 @@`
`13`	`13`	`rating: 4.5,`
`14`	`14`	`creator: {`
`15`	`15`	`name: "John Smith",`
	`16`	`+ "username": "jsmith",`
	`17`	`+ "email": "jsmith@localhost",`
`16`	`18`	`}`
`17`	`19`	`}`
`18`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,9 @@`
`12`	`12`	`rating: Some(4.5),`
`13`	`13`	`telephone: None,`
`14`	`14`	`creator: {`
`15`		`- "name": "John Smith"`
	`15`	`+ "name": "John Smith",`
	`16`	`+ "username": "jsmith",`
	`17`	`+ "email": "jsmith@localhost"`
`16`	`18`	`}`
`17`	`19`	`)`
`18`	`20`	`)`