Skip to content

Commit 35dba0c

Browse files
committed
Minor refactor of TestFile
1 parent c564340 commit 35dba0c

File tree

1 file changed

+20
-18
lines changed

1 file changed

+20
-18
lines changed

src/test_utils/test_file.rs

Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -56,15 +56,6 @@ impl TestFile {
5656
self.file = File::create(&self.path).expect("Error creating test file");
5757
}
5858

59-
pub fn write(&mut self, message: &str) {
60-
let bytes_to_add = message.as_bytes();
61-
let bytes_written = self
62-
.file
63-
.write(bytes_to_add)
64-
.expect("Error writing to file");
65-
assert_eq!(bytes_written, bytes_to_add.len());
66-
}
67-
6859
pub fn write_auth_failed_messages(&mut self, count: usize) {
6960
for i in 0usize..count {
7061
self.write_auth_failed_message(i);
@@ -73,8 +64,24 @@ impl TestFile {
7364

7465
pub fn write_auth_failed_message(&mut self, index: usize) {
7566
let message_index = index % AUTH_FAILED_TEST_MESSAGES.len();
76-
let message = create_log_line(AUTH_FAILED_TEST_MESSAGES[message_index]);
77-
self.write(&message);
67+
let message = AUTH_FAILED_TEST_MESSAGES[message_index];
68+
self.write_log_message(message);
69+
}
70+
71+
fn write_log_message(&mut self, message: &str) {
72+
let date_time = Local::now().format("%+");
73+
let line = format!("{} {}\n", date_time, message);
74+
self.write(&line);
75+
}
76+
77+
fn write(&mut self, message: &str) {
78+
print!("Writing line: {}", message);
79+
let bytes_to_add = message.as_bytes();
80+
let bytes_written = self
81+
.file
82+
.write(bytes_to_add)
83+
.expect("Error writing to file");
84+
assert_eq!(bytes_written, bytes_to_add.len());
7885
}
7986

8087
pub fn write_other_messages(&mut self, count: usize) {
@@ -85,8 +92,8 @@ impl TestFile {
8592

8693
pub fn write_other_message(&mut self, index: usize) {
8794
let message_index = index % OTHER_TEST_MESSAGES.len();
88-
let message = create_log_line(OTHER_TEST_MESSAGES[message_index]);
89-
self.write(&message);
95+
let message = OTHER_TEST_MESSAGES[message_index];
96+
self.write_log_message(message);
9097
}
9198

9299
pub fn truncate(&mut self) {
@@ -111,8 +118,3 @@ impl Drop for TestFile {
111118
self.remove();
112119
}
113120
}
114-
115-
pub fn create_log_line(message: &str) -> String {
116-
let date_time = Local::now().format("%+");
117-
return format!("{} {}\n", date_time, message);
118-
}

0 commit comments

Comments
 (0)