More granular reporting

SatelliteQE · Oct 23, 2024 · 52400d4 · 52400d4
1 parent 8031b77
commit 52400d4
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 16 deletions.
diff --git a/pytest_plugins/requirements/req_updater.py b/pytest_plugins/requirements/req_updater.py
@@ -104,7 +104,7 @@ def install_req_deviations(self):
                 ).returncode
                 == 0
             ):
-                print('Required requirements are updated.')
+                print('Mandatory requirements are updated.')
             else:
                 print('ERROR: Some issue occurred with updating the required requirements')
 

diff --git a/pytest_plugins/requirements/update_requirements.py b/pytest_plugins/requirements/update_requirements.py
@@ -35,28 +35,26 @@ def pytest_report_header(config):
     # pytest tests/foreman --collect-only --update-all-reqs
     """
     req_deviations = updater.req_deviation
-    if req_deviations:
-        # req_deviations = ignore_git_deviations(req_deviations)
-        req_deviations = list(filter(git_deviation_filter, req_deviations))
-        if req_deviations:
-            print(f"Mandatory Requirements Mismatch: {' '.join(req_deviations)}")
-        if config.getoption('update_required_reqs') or config.getoption('update_all_reqs'):
-            updater.install_req_deviations()
+    filtered_req_deviations = list(filter(git_deviation_filter, req_deviations))
+    if filtered_req_deviations:
+        print(f"Mandatory Requirements Mismatch: {' '.join(filtered_req_deviations)}")
     else:
         print('Mandatory Requirements are up to date.')
+    if req_deviations and (
+        config.getoption('update_required_reqs') or config.getoption('update_all_reqs')
+    ):
+        updater.install_req_deviations()
 
     opt_deviations = updater.opt_deviation
-    if opt_deviations:
-        # opt_deviations = ignore_git_deviations(opt_deviations)
-        opt_deviations = list(filter(git_deviation_filter, opt_deviations))
-        if opt_deviations:
-            print(f"Optional Requirements Mismatch: {' '.join(updater.opt_deviation)}")
-        if config.getoption('update_all_reqs'):
-            updater.install_opt_deviations()
+    filtered_opt_deviations = list(filter(git_deviation_filter, opt_deviations))
+    if filtered_opt_deviations:
+        print(f"Optional Requirements Mismatch: {' '.join(filtered_opt_deviations)}")
     else:
         print('Optional Requirements are up to date.')
+    if opt_deviations and config.getoption('update_all_reqs'):
+        updater.install_opt_deviations()
 
-    if updater.req_deviation or updater.opt_deviation:
+    if req_deviations or opt_deviations:
         print(
             "To update requirements, run the pytest with "
             "'--update-required-reqs' OR '--update-all-reqs' option."