Skip to content

Commit c3c646f

Browse files
committed
Refactor Crossterm mocks exports
1 parent c38fd36 commit c3c646f

File tree

7 files changed

+19
-16
lines changed

7 files changed

+19
-16
lines changed

src/application.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ mod tests {
197197
input::{Event, KeyCode, KeyEvent, KeyModifiers},
198198
module::Modules,
199199
runtime::{Installer, RuntimeError},
200-
test_helpers::{create_event_reader, mocks::crossterm::CrossTerm},
200+
test_helpers::{create_event_reader, mocks::CrossTerm},
201201
testutil::{set_git_directory, DefaultTestModule, TestModuleProvider},
202202
};
203203

src/display.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -367,7 +367,7 @@ mod tests {
367367
use rstest::rstest;
368368

369369
use super::*;
370-
use crate::test_helpers::mocks::crossterm::{CrossTerm, State};
370+
use crate::test_helpers::mocks::{CrossTerm, CrosstermMockState};
371371

372372
#[test]
373373
fn draw_str() {
@@ -585,13 +585,13 @@ mod tests {
585585
fn start() {
586586
let mut display = Display::new(CrossTerm::new(), &Theme::new());
587587
display.start().unwrap();
588-
assert_eq!(display.tui.get_state(), State::Normal);
588+
assert_eq!(display.tui.get_state(), CrosstermMockState::Normal);
589589
}
590590

591591
#[test]
592592
fn end() {
593593
let mut display = Display::new(CrossTerm::new(), &Theme::new());
594594
display.end().unwrap();
595-
assert_eq!(display.tui.get_state(), State::Ended);
595+
assert_eq!(display.tui.get_state(), CrosstermMockState::Ended);
596596
}
597597
}

src/editor.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#[cfg(not(test))]
22
use crate::display::CrossTerm;
33
#[cfg(test)]
4-
use crate::test_helpers::mocks::crossterm::CrossTerm;
4+
use crate::test_helpers::mocks::CrossTerm;
55
use crate::{
66
application::Application,
77
arguments::Args,

src/test_helpers/mocks.rs

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
pub(crate) mod crossterm;
1+
mod crossterm;
22
mod notifier;
33

4-
pub(crate) use self::notifier::Notifier;
4+
pub(crate) use self::{
5+
crossterm::{CrossTerm, CrosstermMockState, MockableTui},
6+
notifier::Notifier,
7+
};

src/test_helpers/mocks/crossterm.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ use crate::display::{ColorMode, DisplayError, Size, Tui};
88
/// The state of the `CrossTerm` instance.
99
#[derive(Copy, Clone, Debug, PartialEq, Eq)]
1010
#[allow(clippy::exhaustive_enums)]
11-
pub(crate) enum State {
11+
pub(crate) enum CrosstermMockState {
1212
/// The TUI is new and unchanged.
1313
New,
1414
/// The TUI is in the normal mode.
@@ -138,7 +138,7 @@ struct CrossTermInternalState {
138138
output: Vec<String>,
139139
position: (u16, u16),
140140
size: Size,
141-
state: State,
141+
state: CrosstermMockState,
142142
}
143143

144144
/// A mocked version of `CrossTerm`, useful for testing.
@@ -157,7 +157,7 @@ impl MockableTui for CrossTerm {
157157
state.attributes = Attributes::from(Attribute::Reset);
158158
state.colors = Colors::new(Color::Reset, Color::Reset);
159159
state.output.clear();
160-
state.state = State::Normal;
160+
state.state = CrosstermMockState::Normal;
161161
Ok(())
162162
}
163163

@@ -224,12 +224,12 @@ impl MockableTui for CrossTerm {
224224
}
225225

226226
fn start(&mut self) -> Result<(), DisplayError> {
227-
self.state.write().state = State::Normal;
227+
self.state.write().state = CrosstermMockState::Normal;
228228
Ok(())
229229
}
230230

231231
fn end(&mut self) -> Result<(), DisplayError> {
232-
self.state.write().state = State::Ended;
232+
self.state.write().state = CrosstermMockState::Ended;
233233
Ok(())
234234
}
235235
}
@@ -247,7 +247,7 @@ impl CrossTerm {
247247
output: vec![],
248248
position: (0, 0),
249249
size: Size::new(10, 10),
250-
state: State::New,
250+
state: CrosstermMockState::New,
251251
})),
252252
}
253253
}
@@ -260,7 +260,7 @@ impl CrossTerm {
260260

261261
/// Get the current state.
262262
#[must_use]
263-
pub(crate) fn get_state(&self) -> State {
263+
pub(crate) fn get_state(&self) -> CrosstermMockState {
264264
self.state.read().state
265265
}
266266

src/view/tests.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
use super::*;
2-
use crate::{config::Theme, display::Size, test_helpers::mocks::crossterm::CrossTerm};
2+
use crate::{config::Theme, display::Size, test_helpers::mocks::CrossTerm};
33

44
fn assert_render(width: usize, height: usize, view_data: &ViewData, expected: &[&str]) {
55
let theme = Theme::new();

src/view/thread.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ mod tests {
165165
display::{Display, DisplayError},
166166
runtime::Status,
167167
test_helpers::{
168-
mocks::crossterm::{CrossTerm, MockableTui},
168+
mocks::{CrossTerm, MockableTui},
169169
ThreadableTester,
170170
},
171171
view::ViewData,

0 commit comments

Comments
 (0)