Move QName parsing tests to the appropriate place

Author: Mingun

src/events/mod.rs

@@ -1050,47 +1050,6 @@ mod test {
     use super::*;
     use pretty_assertions::assert_eq;
 
-    #[test]
-    fn local_name() {
-        use std::str::from_utf8;
-        let xml = r#"
-            <foo:bus attr='bar'>foobusbar</foo:bus>
-            <foo: attr='bar'>foobusbar</foo:>
-            <:foo attr='bar'>foobusbar</:foo>
-            <foo:bus:baz attr='bar'>foobusbar</foo:bus:baz>
-            "#;
-        let mut rdr = Reader::from_str(xml);
-        let mut buf = Vec::new();
-        let mut parsed_local_names = Vec::new();
-        loop {
-            match rdr
-                .read_event_into(&mut buf)
-                .expect("unable to read xml event")
-            {
-                Event::Start(ref e) => parsed_local_names.push(
-                    from_utf8(e.local_name().as_ref())
-                        .expect("unable to build str from local_name")
-                        .to_string(),
-                ),
-                Event::End(ref e) => parsed_local_names.push(
-                    from_utf8(e.local_name().as_ref())
-                        .expect("unable to build str from local_name")
-                        .to_string(),
-                ),
-                Event::Eof => break,
-                _ => {}
-            }
-        }
-        assert_eq!(parsed_local_names[0], "bus".to_string());
-        assert_eq!(parsed_local_names[1], "bus".to_string());
-        assert_eq!(parsed_local_names[2], "".to_string());
-        assert_eq!(parsed_local_names[3], "".to_string());
-        assert_eq!(parsed_local_names[4], "foo".to_string());
-        assert_eq!(parsed_local_names[5], "foo".to_string());
-        assert_eq!(parsed_local_names[6], "bus:baz".to_string());
-        assert_eq!(parsed_local_names[7], "bus:baz".to_string());
-    }
-
     #[test]
     fn bytestart_create() {
         let b = BytesStart::owned_name("test");

src/name.rs

@@ -797,4 +797,31 @@ mod namespaces {
         );
         assert_eq!(resolver.find(name, &buffer), Unknown(b"unknown".to_vec()));
     }
+
+    /// Checks how the QName is decomposed to a prefix and a local name
+    #[test]
+    fn prefix_and_local_name() {
+        let name = QName(b"foo:bus");
+        assert_eq!(name.prefix(), Some(Prefix(b"foo")));
+        assert_eq!(name.local_name(), LocalName(b"bus"));
+        assert_eq!(name.decompose(), (LocalName(b"bus"), Some(Prefix(b"foo"))));
+
+        let name = QName(b"foo:");
+        assert_eq!(name.prefix(), Some(Prefix(b"foo")));
+        assert_eq!(name.local_name(), LocalName(b""));
+        assert_eq!(name.decompose(), (LocalName(b""), Some(Prefix(b"foo"))));
+
+        let name = QName(b":foo");
+        assert_eq!(name.prefix(), Some(Prefix(b"")));
+        assert_eq!(name.local_name(), LocalName(b"foo"));
+        assert_eq!(name.decompose(), (LocalName(b"foo"), Some(Prefix(b""))));
+
+        let name = QName(b"foo:bus:baz");
+        assert_eq!(name.prefix(), Some(Prefix(b"foo")));
+        assert_eq!(name.local_name(), LocalName(b"bus:baz"));
+        assert_eq!(
+            name.decompose(),
+            (LocalName(b"bus:baz"), Some(Prefix(b"foo")))
+        );
+    }
 }