Move todo_file testutils to test_helpers

Author: MitMaro

src/module/modules.rs

@@ -69,7 +69,7 @@ impl ModuleProvider for Modules {
 #[cfg(test)]
 mod tests {
 	use super::*;
-	use crate::{test_helpers::with_temp_repository, todo_file::testutil::with_todo_file};
+	use crate::test_helpers::{with_temp_repository, with_todo_file};
 
 	pub(crate) fn modules_test<C>(callback: C)
 	where C: FnOnce(Modules) {

src/test_helpers.rs

@@ -10,6 +10,7 @@ mod with_event_handler;
 mod with_git_config;
 mod with_temp_bare_repository;
 mod with_temp_repository;
+mod with_todo_file;
 
 pub(crate) static JAN_2021_EPOCH: i64 = 1_609_459_200;
 
@@ -23,4 +24,5 @@ pub(crate) use self::{
 	with_git_config::with_git_config,
 	with_temp_bare_repository::with_temp_bare_repository,
 	with_temp_repository::with_temp_repository,
+	with_todo_file::{with_todo_file, TodoFileTestContext},
 };

src/todo_file/testutil.rs renamed to src/test_helpers/with_todo_file.rs

@@ -19,7 +19,7 @@ impl Debug for TodoFileTestContext {
 	fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {
 		f.debug_struct("TodoFileTestContext")
 			.field("todo_file", &self.todo_file)
-			.field("filepath", &self.todo_file.filepath)
+			.field("filepath", &self.todo_file.get_filepath())
 			.finish_non_exhaustive()
 	}
 }

src/testutil/module_test.rs

@@ -4,8 +4,8 @@ use crate::{
 	input::Event,
 	module::{Module, State},
 	process::Results,
-	test_helpers::{with_event_handler, EventHandlerTestContext},
-	todo_file::{testutil::with_todo_file, TodoFile},
+	test_helpers::{with_event_handler, with_todo_file, EventHandlerTestContext},
+	todo_file::TodoFile,
 	view::{
 		testutil::{with_view_state, TestContext as ViewContext},
 		RenderContext,

src/testutil/process_test.rs

@@ -8,9 +8,8 @@ use crate::{
 	module::{self, ModuleHandler},
 	process::Process,
 	runtime::ThreadStatuses,
-	test_helpers::{with_event_handler, EventHandlerTestContext},
+	test_helpers::{with_event_handler, with_todo_file, EventHandlerTestContext},
 	testutil::{with_search, SearchTestContext},
-	todo_file::testutil::with_todo_file,
 	view::testutil::{with_view_state, TestContext as ViewContext},
 };
 

src/todo_file.rs

@@ -10,9 +10,6 @@ mod history;
 mod line;
 mod line_parser;
 mod search;
-#[cfg(test)]
-#[cfg(not(tarpaulin_include))]
-pub(crate) mod testutil;
 mod todo_file_options;
 mod utils;
 

src/todo_file/search.rs

@@ -160,7 +160,7 @@ mod tests {
 	use claims::{assert_none, assert_some_eq};
 
 	use super::*;
-	use crate::todo_file::testutil::with_todo_file;
+	use crate::test_helpers::with_todo_file;
 
 	#[test]
 	fn search_empty_rebase_file() {