junit fix (#8)

Author: Alex Remedios

python-cli/pyproject.toml

@@ -39,4 +39,7 @@ addopts = [
   "--strict-markers",
   # "--capture=tee-sys",
   "--forked"
-]
+]
+
+[tool.poetry.plugins."pytest11"]
+deeptest = "deeptest.pytest_plugin"

python-cli/src/deeptest/pytest_plugin.py

@@ -0,0 +1,20 @@
+import shutil
+from typing import Any
+
+from _pytest.config import Config
+from pytest import ExitCode
+
+out_path = ".deeptest/junit.xml"
+
+
+def pytest_configure(config):
+    if config.option.xmlpath is None:
+        config.option.xmlpath = out_path
+
+
+def pytest_terminal_summary(
+    terminalreporter: Any, exitstatus: ExitCode, config: Config
+):
+    if not config.option.xmlpath == out_path:
+        print(f"Copying {config.option.xmlpath} to {out_path}")
+        shutil.copy(config.option.xmlpath, out_path)

python-cli/tests/resources/pytest.ini

@@ -1,5 +1,4 @@
 [pytest]
 addopts =
   --cov=src
-  --cov-context=test
-  --junit-xml=.deeptest/junit.xml
+  --cov-context=test

python-cli/tests/test_cli.py

@@ -5,17 +5,15 @@
 from click.testing import CliRunner
 from deeptest.cli import File
 from snapshottest.pytest import PyTestSnapshotTest
+from tests.util import RESOURCES
 
 from deeptest import cli
 
 pytest_plugins = "pytester"
 import shutil
 import sys
-from pathlib import Path
 from subprocess import CalledProcessError, check_output
 
-RESOURCES = (Path(__file__) / ".." / "resources").resolve()
-
 
 @pytest.fixture
 def tested_dir():

python-cli/tests/test_pytest_plugin.py

@@ -0,0 +1,35 @@
+from __future__ import print_function
+
+from distutils.dir_util import copy_tree
+from pathlib import Path
+
+import pytest
+from _pytest.pytester import Testdir
+from pytest import ExitCode
+from tests.util import RESOURCES
+
+pytest_plugins = "pytester"
+NB_VERSION = 4
+import shutil
+
+
+@pytest.fixture
+def fake_repo(testdir: Testdir):
+    copy_tree(RESOURCES.as_posix(), ".")
+
+
+def test_when_no_xml_then_output_correctly(testdir: Testdir, fake_repo: object):
+    shutil.rmtree(".deeptest", ignore_errors=True)
+    hook_recorder = testdir.inline_run()
+
+    assert hook_recorder.ret == ExitCode.TESTS_FAILED
+    assert Path(".deeptest/junit.xml").exists()
+
+
+def test_when_other_xml_then_output_correctly(testdir: Testdir, fake_repo: object):
+    shutil.rmtree(".deeptest", ignore_errors=True)
+    hook_recorder = testdir.inline_run("--junit-xml=junit.xml")
+
+    assert hook_recorder.ret == ExitCode.TESTS_FAILED
+    assert Path(".deeptest/junit.xml").exists()
+    assert Path("junit.xml").exists()

python-cli/tests/util.py

@@ -0,0 +1,3 @@
+from pathlib import Path
+
+RESOURCES = (Path(__file__) / ".." / "resources").resolve()