From ccb50506f8876ff93fa291669f865b1f9bf8239b Mon Sep 17 00:00:00 2001
From: German <28149841+germa89@users.noreply.github.com>
Date: Mon, 20 Nov 2023 16:15:27 +0100
Subject: [PATCH] Removing skip regression check cli argument (#2487)

* Removing skip regression check cli argument

* Removing pytest-pyvista argument
---
 .github/workflows/ci.yml |  1 -
 tests/conftest.py        | 12 ------------
 2 files changed, 13 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index becb5eaceb..5af6400a7e 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -644,7 +644,6 @@ jobs:
           export ANSYSLMD_LICENSE_FILE=1055@${{ secrets.LICENSE_SERVER }}
           pytest -k "not test_dpf" \
             ${{ env.PYTEST_ARGUMENTS }} \
-            --skip-regression-check \
             --cov-report=xml:ubuntu-v22.2.0-local-minimal.xml
         
       - uses: codecov/codecov-action@v3
diff --git a/tests/conftest.py b/tests/conftest.py
index 364728c27f..70732888c8 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -245,12 +245,6 @@ def pytest_addoption(parser):
         default=False,
         help="run only GUI tests",
     )
-    parser.addoption(
-        "--skip-regression-check",
-        action="store_true",
-        default=False,
-        help="Avoid checking the image regression check in all tests",
-    )
 
 
 def pytest_collection_modifyitems(config, items):
@@ -302,13 +296,7 @@ def pytest_collection_modifyitems(config, items):
 
     @pytest.fixture(autouse=True)
     def wrapped_verify_image_cache(verify_image_cache, pytestconfig):
-        # Checking if we want to avoid the check using pytest cli.
-        skip_regression_check = pytestconfig.option.skip_regression_check
-        if skip_regression_check:
-            verify_image_cache.skip = True
-
         # Configuration
-        # default check
         verify_image_cache.error_value = 500.0
         verify_image_cache.warning_value = 200.0