fix(tests): update pytest arguments for consistency and clarity in local and remote workflows (#4322)

* fix(tests): update pytest arguments for consistency and clarity in local and remote workflows

* chore: adding changelog file 4322.fixed.md [dependabot-skip]

* ci: auto fixes from pre-commit.com hooks.

for more information, see https://pre-commit.ci

* fix(tests): update xfail condition for remote DPF upload test

* Update pyproject.toml

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: pyansys-ci-bot <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Copilot <[email protected]>

Author: 4 people

.github/workflows/test-local.yml

@@ -289,9 +289,7 @@ jobs:
           $cmd ${PYTEST_INPUT_ARGUMENTS} \
             ${PYTEST_ARGUMENTS} \
             --report-log=$file_name.jsonl \
-            --cov-report=xml:$file_name.xml \
-            --log-file=pytest.log \
-            --log-file-level="DEBUG"
+            --cov-report=xml:$file_name.xml
 
       - name: "Upload pytest reports to GitHub"
         if: always()

.github/workflows/test-remote.yml

@@ -85,7 +85,7 @@ jobs:
       DPF_START_SERVER: False
       HAS_DPF: True
       TEST_DPF_BACKEND: false
-      PYTEST_ARGUMENTS: '-vvv -ra --color=yes --durations=30 --random-order --random-order-bucket=class --maxfail=2 --reruns 2 --reruns-delay 2 --cov=ansys.mapdl.core --cov-report=html --timeout=180 --profile-svg --profile --report-log-exclude-logs-on-passed-tests --strict-markers'
+      PYTEST_ARGUMENTS: '--ignore_image_cache'
       MAPDL_PACKAGE: ghcr.io/ansys/mapdl
 
     steps:
@@ -276,11 +276,8 @@ jobs:
           echo "ON_STUDENT: $ON_STUDENT"
           xvfb-run pytest \
             ${PYTEST_ARGUMENTS} \
-            --ignore_image_cache \
             --report-log=$file_name.jsonl \
-            --cov-report=xml:$file_name.xml \
-            --log-file=pytest.log \
-            --log-file-level="DEBUG"
+            --cov-report=xml:$file_name.xml
 
       - name: "Print amount of restarts"
         if: always()

doc/changelog.d/4322.fixed.md

@@ -0,0 +1 @@
+Update pytest arguments for consistency and clarity in local and remote workflows

pyproject.toml

@@ -131,7 +131,26 @@ pymapdl = "ansys.mapdl.core.cli:main"
 pymapdl_convert_script = "ansys.mapdl.core.cli:old_pymapdl_convert_script_entry_point"
 
 [tool.pytest.ini_options]
-addopts = "-ra -vvv --maxfail=10 --strict-markers --random-order-bucket=class --random-order --durations=10  --timeout=180"
+addopts = [
+  "--cov-report=html",
+  "--cov=ansys.mapdl.core",
+  "--durations=30",
+  "--log-file-level=DEBUG",
+  "--log-file=pytest.log",
+  "--maxfail=10",
+  "--profile",
+  "--profile-svg",
+  "--random-order",
+  "--random-order-bucket=class",
+  "--report-log-exclude-logs-on-passed-tests",
+  "--reruns-delay=4",
+  "--reruns=3",
+  "--strict-markers",
+  "--timeout=180",
+  "-ra",
+  "-vvv",
+]
+
 filterwarnings = [
   "ignore::DeprecationWarning",
   "ignore::FutureWarning",

tests/test_result.py

@@ -499,7 +499,7 @@ def test_DPF_result_class(self, mapdl, _use_reader_backend, expected_cls):
         mapdl._use_reader_backend = _use_reader_backend
         assert isinstance(mapdl.result, expected_cls)
 
-    @pytest.mark.xfail(not ON_LOCAL, reason="Upload to remote using DPF is broken")
+    @pytest.mark.xfail(True, reason="Upload to remote using DPF is broken")
     def test_solve_rst_only(self, mapdl, result):
         """Test that the result object can be created with a solved RST file."""
         # Check if the result object is created successfully