Move view test utils into test_helpers

Author: MitMaro

src/components/choice/tests.rs

@@ -4,8 +4,7 @@ use super::*;
 use crate::{
 	assert_rendered_output,
 	input::StandardEvent,
-	test_helpers::assertions::assert_rendered_output::AssertRenderOptions,
-	view::testutil::with_view_state,
+	test_helpers::{assertions::assert_rendered_output::AssertRenderOptions, with_view_state},
 };
 
 #[derive(Clone, Debug, PartialEq)]

src/components/help/tests.rs

@@ -4,7 +4,7 @@ use super::*;
 use crate::{
 	assert_rendered_output,
 	input::{KeyModifiers, MouseEvent, MouseEventKind, StandardEvent},
-	view::testutil::with_view_state,
+	test_helpers::with_view_state,
 };
 
 fn handle_event(help: &mut Help, event: Event) {

src/test_helpers.rs

@@ -12,6 +12,7 @@ mod with_git_config;
 mod with_temp_bare_repository;
 mod with_temp_repository;
 mod with_todo_file;
+mod with_view_state;
 
 pub(crate) static JAN_2021_EPOCH: i64 = 1_609_459_200;
 
@@ -26,4 +27,5 @@ pub(crate) use self::{
 	with_temp_bare_repository::with_temp_bare_repository,
 	with_temp_repository::with_temp_repository,
 	with_todo_file::{with_todo_file, TodoFileTestContext},
+	with_view_state::{with_view_state, ViewStateTestContext},
 };

src/view/testutil.rs renamed to src/test_helpers/with_view_state.rs

@@ -1,5 +1,3 @@
-//! Utilities for writing tests that interact with input events.
-
 use std::time::Duration;
 
 use crate::view::{RenderAction, State, ViewAction};

src/testutil/module_test.rs

@@ -4,19 +4,21 @@ use crate::{
 	input::Event,
 	module::{Module, State},
 	process::Results,
-	test_helpers::{with_event_handler, with_todo_file, EventHandlerTestContext},
-	todo_file::TodoFile,
-	view::{
-		testutil::{with_view_state, ViewStateTestContext as ViewContext},
-		RenderContext,
-		ViewData,
+	test_helpers::{
+		with_event_handler,
+		with_todo_file,
+		with_view_state,
+		EventHandlerTestContext,
+		ViewStateTestContext,
 	},
+	todo_file::TodoFile,
+	view::{RenderContext, ViewData},
 };
 
 pub(crate) struct TestContext {
 	pub(crate) event_handler_context: EventHandlerTestContext,
 	pub(crate) render_context: RenderContext,
-	pub(crate) view_context: ViewContext,
+	pub(crate) view_context: ViewStateTestContext,
 	todo_file: Option<TodoFile>,
 }
 

src/testutil/process_test.rs

@@ -8,17 +8,22 @@ use crate::{
 	module::{self, ModuleHandler},
 	process::Process,
 	runtime::ThreadStatuses,
-	test_helpers::{with_event_handler, with_todo_file, EventHandlerTestContext},
+	test_helpers::{
+		with_event_handler,
+		with_todo_file,
+		with_view_state,
+		EventHandlerTestContext,
+		ViewStateTestContext,
+	},
 	testutil::{with_search, SearchTestContext},
-	view::testutil::{with_view_state, ViewStateTestContext as ViewContext},
 };
 
 pub(crate) struct TestContext<ModuleProvider: module::ModuleProvider + Send + 'static> {
 	pub(crate) event_handler_context: EventHandlerTestContext,
 	pub(crate) process: Process<ModuleProvider>,
 	pub(crate) search_context: SearchTestContext,
 	pub(crate) todo_file_path: PathBuf,
-	pub(crate) view_context: ViewContext,
+	pub(crate) view_context: ViewStateTestContext,
 }
 
 pub(crate) fn process_test<C, ModuleProvider: module::ModuleProvider + Send + 'static>(

src/util.rs

@@ -56,7 +56,7 @@ mod tests {
 	use rstest::rstest;
 
 	use super::*;
-	use crate::view::testutil::with_view_state;
+	use crate::test_helpers::with_view_state;
 
 	#[rstest]
 	#[case::scroll_left(StandardEvent::ScrollLeft, "ScrollLeft")]

src/view.rs

@@ -21,8 +21,6 @@ mod view_line;
 
 #[cfg(test)]
 mod tests;
-#[cfg(test)]
-pub(crate) mod testutil;
 
 use anyhow::{Error, Result};
 

src/view/render_slice.rs

@@ -195,7 +195,7 @@ impl RenderSlice {
 	}
 
 	#[cfg(test)]
-	pub(super) const fn get_actions(&self) -> &VecDeque<RenderAction> {
+	pub(crate) const fn get_actions(&self) -> &VecDeque<RenderAction> {
 		&self.actions
 	}
 

src/view/thread/state.rs

@@ -153,7 +153,10 @@ impl State {
 
 #[cfg(test)]
 mod tests {
-	use crate::view::{testutil::with_view_state, ViewData, ViewLine};
+	use crate::{
+		test_helpers::with_view_state,
+		view::{ViewData, ViewLine},
+	};
 
 	#[test]
 	fn start() {