Refactored to avoid allocating so many strings.

1 files changed, 15 insertions, 9 deletions

diff --git a/tests/test_deserialize.rs b/tests/test_deserialize.rs
index 776d562..b6a65a5 100644
--- a/tests/test_deserialize.rs
+++ b/tests/test_deserialize.rs
@@ -4,26 +4,32 @@ extern crate serde_urlencoded;
 
 #[test]
 fn deserialize_bytes() {
-    let result = vec![("first".to_owned(), 23), ("last".to_owned(), 42)];
+    let mut result = vec![("first".to_owned(), 23), ("last".to_owned(), 42)];
+    let mut deres: Vec<(String, u8)> = serde_urlencoded::from_bytes(b"first=23&last=42").unwrap();
 
-    assert_eq!(serde_urlencoded::from_bytes(b"first=23&last=42"),
-               Ok(result));
+    result.sort();
+    deres.sort();
+    assert_eq!(deres, result);
 }
 
 #[test]
 fn deserialize_str() {
-    let result = vec![("first".to_owned(), 23), ("last".to_owned(), 42)];
+    let mut result = vec![("first".to_owned(), 23), ("last".to_owned(), 42)];
+    let mut deres: Vec<(String, u8)> = serde_urlencoded::from_str("first=23&last=42").unwrap();
 
-    assert_eq!(serde_urlencoded::from_str("first=23&last=42"),
-               Ok(result));
+    result.sort();
+    deres.sort();
+    assert_eq!(deres, result);
 }
 
 #[test]
 fn deserialize_reader() {
-    let result = vec![("first".to_owned(), 23), ("last".to_owned(), 42)];
+    let mut result = vec![("first".to_owned(), 23), ("last".to_owned(), 42)];
+    let mut deres: Vec<(String, u8)> = serde_urlencoded::from_reader(b"first=23&last=42" as &[_]).unwrap();
 
-    assert_eq!(serde_urlencoded::from_reader(b"first=23&last=42" as &[_]),
-               Ok(result));
+    result.sort();
+    deres.sort();
+    assert_eq!(deres, result);
 }