Skip to content

Commit a944e40

Browse files
committed
fix: move attributes_order fn into test module
this avoid dead_code
1 parent e6a8af9 commit a944e40

File tree

1 file changed

+20
-15
lines changed

1 file changed

+20
-15
lines changed

crates/oxvg_lint/src/rules/attributes.rs

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -79,21 +79,26 @@ enum Pattern {
7979
Optional(Box<Pattern>),
8080
}
8181

82-
#[test]
83-
fn attributes_order() {
84-
use xml5ever::{
85-
driver::{parse_document, XmlParseOpts},
86-
tendril::TendrilSink,
87-
};
82+
#[cfg(test)]
83+
mod tests {
84+
use super::{Order, Rules};
8885

89-
let dom: rcdom::RcDom = parse_document(rcdom::RcDom::default(), XmlParseOpts::default())
90-
.one(r#"<svg z="" a=""></svg>"#);
91-
let root = &*dom.document.children.borrow()[0];
86+
#[test]
87+
fn attributes_order() {
88+
use xml5ever::{
89+
driver::{parse_document, XmlParseOpts},
90+
tendril::TendrilSink,
91+
};
92+
93+
let dom: rcdom::RcDom = parse_document(rcdom::RcDom::default(), XmlParseOpts::default())
94+
.one(r#"<svg z="" a=""></svg>"#);
95+
let root = &*dom.document.children.borrow()[0];
9296

93-
// Expect some error, as "z" is before "a"
94-
let rule = Rules {
95-
order: Some(Order::Alphabetical),
96-
..Rules::default()
97-
};
98-
assert!(rule.order(root).is_some());
97+
// Expect some error, as "z" is before "a"
98+
let rule = Rules {
99+
order: Some(Order::Alphabetical),
100+
..Rules::default()
101+
};
102+
assert!(rule.order(root).is_some());
103+
}
99104
}

0 commit comments

Comments
 (0)