Merge pull request #9 from GuillaumeGomez/fix-attrs

Fix attrs

Author: GuillaumeGomez

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "html-diff"
-version = "0.0.3"
+version = "0.0.4"
 authors = ["Guillaume Gomez <[email protected]>"]
 
 description = "Library detect HTML diffs"

src/lib.rs

@@ -188,7 +188,7 @@ fn check_elements(elem1: &NodeDataRef<ElementData>,
             elem: ElementInformation::new(elem1, path),
             opposite_elem: ElementInformation::new(elem2, path),
         })
-    } else if (*e1.attributes.borrow()).map.len() != (*e2.attributes.borrow()).map.len() {
+    } else {
         let all_attributes_match =
             (*e1.attributes.borrow()).map.iter().any(|(k, v)| {
                 (*e2.attributes.borrow()).map.get(k) != Some(v)
@@ -217,8 +217,6 @@ fn check_elements(elem1: &NodeDataRef<ElementData>,
         } else {
             None
         }
-    } else {
-        None
     }
 }
 
@@ -429,3 +427,16 @@ fn test_issue_6() {
     let b = "<mark></mark><div class=\"paragraph\"><p>chunky bacon</p></div><mark></mark>";
     let _ = get_differences(a, b);
 }
+
+#[test]
+fn test_attributes() {
+    let a = r#"<span class="toto"></span>"#;
+    let b = r#"<span class="tata"></span>"#;
+    let differences = get_differences(a, b);
+    assert_eq!(differences.len(), 1);
+
+    let a = r#"<span foo="a" class="toto"></span>"#;
+    let b = r#"<span class="toto" foo="a"></span>"#;
+    let differences = get_differences(a, b);
+    assert_eq!(differences.len(), 0);
+}