Skip to content

Commit 19b83f6

Browse files
committed
fix(tests): creating and removing file at test time only
1 parent 8de1d1a commit 19b83f6

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

src/read.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,15 @@ pub fn read_string(xml_string: String, root_tag: String) -> Node {
7676
mod tests {
7777
use crate::f_str;
7878
use crate::read::read_file;
79+
use std::fs::{remove_file, File};
80+
use std::io::prelude::*;
7981
#[test]
8082
fn test_read_file() {
83+
let mut file = File::create("tests/test.xml").unwrap();
84+
file.write_all(b"<?xml version=\"1.0\" encoding=\"utf-8\"?>\n<root test=\"test\">\n <child test=\"test\">\n <child>test</child>\n </child>\n</root>")
85+
.unwrap();
8186
let node = read_file(f_str!("tests/test.xml"), f_str!("root"));
87+
remove_file("tests/test.xml").unwrap();
8288
assert_eq!(node.name, f_str!("root"));
8389
assert_eq!(node.attrs.len(), 1);
8490
assert_eq!(node.attrs.get("test").unwrap(), "test");

src/write.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@ mod tests {
6666
use crate::entities::Node;
6767
use crate::write::{write_file, write_node_to_string, write_string};
6868
use std::collections::HashMap;
69+
use std::fs::{read_to_string, remove_file};
6970
fn root_node() -> Node {
7071
let mut attrs = HashMap::new();
7172
attrs.insert("test".to_string(), "test".to_string());
@@ -117,8 +118,8 @@ mod tests {
117118
Some(4),
118119
Some(true),
119120
);
120-
let file_str = std::fs::read_to_string("tests/test_write.xml").unwrap();
121+
let file_str = read_to_string("tests/test_write.xml").unwrap();
122+
remove_file("tests/test_write.xml").unwrap();
121123
assert_eq!(file_str, expected);
122-
std::fs::remove_file("tests/test_write.xml").unwrap();
123124
}
124125
}

0 commit comments

Comments
 (0)