Merge pull request #340 from espressif/fix/app_path-for-python-case

fix: app_path haven't recorded in python cases

Author: hfudev

.github/workflows/test-python.yml

@@ -21,7 +21,7 @@ jobs:
       matrix:
         include:
           - python-version: "3.7"
-            arch: "ARM"
+            arch: "ARM64"
           - python-version: "3.13"
             arch: "X64"
       fail-fast: false

conftest.py

@@ -1,6 +1,7 @@
 import os
 import shutil
 import sys
+import textwrap
 from typing import List, Pattern
 
 import pytest
@@ -25,6 +26,14 @@ def copy_fixtures(testdir: Testdir):
         else:
             shutil.copytree(os.path.join(fixture_dir, item), os.path.join(str(testdir.tmpdir), item))
 
+    testdir.makepyprojecttoml(
+        textwrap.dedent("""
+        [tool.pytest.ini_options]
+        filterwarnings = "ignore:FutureWarning"
+        junit_family = "xunit1"
+    """)
+    )
+
     yield
 
 

pytest-embedded/pytest_embedded/plugin.py

@@ -749,9 +749,11 @@ def embedded_services(request: FixtureRequest) -> t.Optional[str]:
 
 @pytest.fixture
 @multi_dut_argument
-def app_path(request: FixtureRequest, test_file_path: str) -> t.Optional[str]:
+def app_path(request: FixtureRequest, test_file_path: str, record_xml_attribute) -> t.Optional[str]:
     """Enable parametrization for the same cli option"""
-    return _request_param_or_config_option_or_default(request, 'app_path', os.path.dirname(test_file_path))
+    res = _request_param_or_config_option_or_default(request, 'app_path', os.path.dirname(test_file_path))
+    record_xml_attribute('app_path', res)
+    return res
 
 
 @pytest.fixture

pytest-embedded/tests/test_base.py

@@ -683,6 +683,37 @@ def test_unclosed_file_handler(test_input, dut):
     result.assert_outcomes(passed=1024)
 
 
+def test_check_app_path_for_python_case(testdir):
+    testdir.makepyfile(r"""
+    import pytest
+
+
+    def test_foo(dut):
+        dut.write('foo')
+        dut.expect_exact('foo')
+    """)
+
+    result = testdir.runpytest(
+        '--junitxml',
+        'report.xml',
+        '--app-path',
+        '/tmp/foo',
+    )
+    result.assert_outcomes(passed=1)
+
+    junit_report = ET.parse('report.xml').getroot()[0]
+    assert junit_report[0].get('app_path') == '/tmp/foo'
+
+    result = testdir.runpytest(
+        '--junitxml',
+        'report.xml',
+    )
+    result.assert_outcomes(passed=1)
+
+    junit_report = ET.parse('report.xml').getroot()[0]
+    assert junit_report[0].get('app_path') == str(testdir)
+
+
 class TestTargetMarkers:
     def test_add_target_as_marker_simple(self, pytester):
         pytester.makepyfile("""