Refactor tests.

Author: domdfcoding

repo_helper/testing/__init__.py

@@ -47,7 +47,7 @@
 from repo_helper.files.linting import lint_fix_list, lint_warn_list
 from repo_helper.templates import template_dir
 
-__all__ = ["demo_environment", "original_datadir", "temp_repo", "temp_empty_repo"]
+__all__ = ["demo_environment", "original_datadir", "temp_repo", "temp_empty_repo", "example_config"]
 
 
 @pytest.fixture()
@@ -188,8 +188,8 @@ def temp_empty_repo(tmp_pathplus, monkeypatch) -> Repo:
 	return repo
 
 
-@pytest.fixture
-def temp_repo(temp_empty_repo) -> Repo:
+@pytest.fixture()
+def temp_repo(temp_empty_repo, example_config) -> Repo:
 	"""
 	Pytest fixture to return a git repository in a temporary location.
 
@@ -199,8 +199,15 @@ def temp_repo(temp_empty_repo) -> Repo:
 	:data:`repo_helper.utils.today` is monkeypatched to return 25th July 2020.
 	"""
 
-	(temp_empty_repo.path / "repo_helper.yml").write_text(
-			(pathlib.Path(__file__).parent / "repo_helper_example.yml").read_text()
-			)
+	(temp_empty_repo.path / "repo_helper.yml").write_text(example_config)
 
 	return temp_empty_repo
+
+
+@pytest.fixture(scope="session")
+def example_config() -> str:
+	"""
+	Returns the contents of the example ``repo_helper.yml`` file.
+	"""
+
+	return (pathlib.Path(__file__).parent / "repo_helper_example.yml").read_text()

tests/test_cli/test_conda_recipe.py

@@ -13,10 +13,12 @@
 from repo_helper.cli.commands.conda_recipe import make_recipe
 
 
-def test_conda_recipe(tmp_pathplus, file_regression: FileRegressionFixture):
-	(tmp_pathplus / "repo_helper.yml").write_text(
-			(pathlib.Path(__file__).parent.parent / "repo_helper.yml_").read_text()
-			)
+def test_conda_recipe(
+		tmp_pathplus,
+		file_regression: FileRegressionFixture,
+		example_config,
+		):
+	(tmp_pathplus / "repo_helper.yml").write_text(example_config)
 	(tmp_pathplus / "requirements.txt").write_lines([
 			"apeye>=0.3.0",
 			"attrs>=20.2.0",
@@ -48,10 +50,12 @@ def test_conda_recipe(tmp_pathplus, file_regression: FileRegressionFixture):
 	check_file_output(tmp_pathplus / "conda/meta.yaml", file_regression)
 
 
-def test_conda_recipe_specifiers(tmp_pathplus, file_regression: FileRegressionFixture):
-	(tmp_pathplus / "repo_helper.yml").write_text(
-			(pathlib.Path(__file__).parent.parent / "repo_helper.yml_").read_text()
-			)
+def test_conda_recipe_specifiers(
+		tmp_pathplus,
+		file_regression: FileRegressionFixture,
+		example_config,
+		):
+	(tmp_pathplus / "repo_helper.yml").write_text(example_config)
 	(tmp_pathplus / "requirements.txt").write_lines([
 			'apeye>=0.3.0; python_version < "3.10"',
 			"attrs[extra]>=20.2.0",

tests/test_configuration.py

@@ -66,6 +66,6 @@ def test_get_gh_actions_python_versions(data_regression: DataRegressionFixture,
 			)
 
 
-def test_parse_yaml(tmp_pathplus, data_regression):
-	(tmp_pathplus / "repo_helper.yml").write_text((pathlib.Path(__file__).parent / "repo_helper.yml_").read_text())
+def test_parse_yaml(tmp_pathplus, data_regression, example_config):
+	(tmp_pathplus / "repo_helper.yml").write_text(example_config)
 	data_regression.check(parse_yaml(tmp_pathplus))

tests/test_core.py

@@ -25,11 +25,15 @@
 
 
 @pytest.mark.skipif(condition=os.sep == '\\', reason="Different test for platforms where os.sep == \\")
-def test_via_run_repo_helper_forward(temp_empty_repo, capsys, file_regression: FileRegressionFixture, monkeypatch):
+def test_via_run_repo_helper_forward(
+		temp_empty_repo,
+		capsys,
+		file_regression: FileRegressionFixture,
+		monkeypatch,
+		example_config,
+		):
 
-	(temp_empty_repo.path / "repo_helper.yml").write_text(
-			(pathlib.Path(__file__).parent / "repo_helper.yml_").read_text()
-			)
+	(temp_empty_repo.path / "repo_helper.yml").write_text(example_config)
 
 	run_repo_helper(temp_empty_repo.path, force=False, initialise=True, commit=True, message="Testing Testing")
 
@@ -49,7 +53,11 @@ def test_via_run_repo_helper_forward(temp_empty_repo, capsys, file_regression: F
 
 @pytest.mark.skipif(condition=os.sep == '/', reason="Different test for platforms where os.sep == /")
 def test_via_run_repo_helper_backward(
-		temp_empty_repo, capsys, file_regression: FileRegressionFixture, monkeypatch
+		temp_empty_repo,
+		capsys,
+		file_regression: FileRegressionFixture,
+		monkeypatch,
+		example_config,
 		):
 
 	# Monkeypatch dulwich so it doesn't try to use the global config.
@@ -61,9 +69,7 @@ def test_via_run_repo_helper_backward(
 
 	monkeypatch.setattr(repo_helper.utils, "today", FAKE_DATE)
 
-	(temp_empty_repo.path / "repo_helper.yml").write_text(
-			(pathlib.Path(__file__).parent / "repo_helper.yml_").read_text()
-			)
+	(temp_empty_repo.path / "repo_helper.yml").write_text(example_config)
 
 	run_repo_helper(temp_empty_repo.path, force=False, initialise=True, commit=True, message="Testing Testing")
 
@@ -86,15 +92,14 @@ def test_via_Repo_class(
 		capsys,
 		file_regression: FileRegressionFixture,
 		data_regression: DataRegressionFixture,
-		monkeypatch
+		monkeypatch,
+		example_config,
 		):
 
 	monkeypatch.setattr(repo_helper.utils, "today", FAKE_DATE)
 
 	with in_directory(temp_repo.path):
-		(temp_repo.path / "repo_helper.yml").write_text(
-				(pathlib.Path(__file__).parent / "repo_helper.yml_").read_text()
-				)
+		(temp_repo.path / "repo_helper.yml").write_text(example_config)
 		(temp_repo.path / "requirements.txt").touch()
 		(temp_repo.path / "README.rst").touch()
 		(temp_repo.path / "doc-source").mkdir()