openqa-bats-review: Add --dry-run option

Author: ricardobranco777

openqa-bats-review

@@ -136,7 +136,7 @@ def resolve_clone_chain(openqa_host: str, job_id: int) -> list[int]:
     return chain
 
 
-def main(url: str) -> None:
+def main(url: str, dry_run: bool = False) -> None:
     """
     Main function
     """
@@ -197,21 +197,23 @@ def main(url: str) -> None:
     common_failures: set[str] = set.intersection(*all_failures)
 
     if not common_failures:
-        log.info("No common failures across clone chain. Tagging as PASSED.")
-        # TODO: Tag job as passed
+        if not dry_run:
+            log.info("No common failures across clone chain. Would tag as PASSED.")
+            # TODO: Tag job as passed
+        else:
+            log.info("No common failures across clone chain. Tagging as PASSED.")
     else:
         log.info("Common failures found across clone chain: %s", " ".join(common_failures))
         sys.exit(1)
 
 
 def parse_args() -> argparse.Namespace:
-    """
-    Parse args
-    """
     parser = argparse.ArgumentParser()
+    parser.add_argument("-n", "--dry-run", action="store_true", help="dry run")
     parser.add_argument("url", help="URL to openQA jobs")
     return parser.parse_args()
 
 
 if __name__ == "__main__":
-    main(parse_args().url)
+    args = parse_args()
+    main(args.url, dry_run=args.dry_run)

tests/openqa_bats_review_test.py

@@ -61,7 +61,7 @@ def mock_job_response(url):
         mock_get_job.side_effect = mock_job_response
 
         with pytest.raises(KeyError):
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
 
 
 class TestRegexPattern:
@@ -381,7 +381,7 @@ def test_main_no_clones(self, mock_log, mock_resolve_clone_chain):
         mock_resolve_clone_chain.return_value = [123]
 
         with pytest.raises(SystemExit) as exc_info:
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
 
         assert exc_info.value.code == 0
         mock_log.info.assert_called_with("No clones. Exiting")
@@ -412,7 +412,7 @@ def mock_job_response(url):
             {"file1:test3", "file1:test4"},  # Job 122 failures (different)
         ]
 
-        main("http://openqa.example.com/tests/123")
+        main("http://openqa.example.com/tests/123", dry_run=True)
 
         mock_log.info.assert_any_call("Processing clone chain: %s", "123 -> 122")
         mock_log.info.assert_any_call(
@@ -436,7 +436,7 @@ def mock_job_response(url):
         mock_get_job.side_effect = mock_job_response
 
         with pytest.raises(SystemExit) as exc_info:
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
 
         assert exc_info.value.code == 0
         mock_log.info.assert_any_call("Job %s has no TAP logs, skipping", 123)
@@ -465,7 +465,7 @@ def mock_job_response(url):
         mock_get_job.side_effect = mock_job_response
 
         with pytest.raises(SystemExit) as exc_info:
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
 
         assert exc_info.value.code == 0
         mock_log.info.assert_any_call("Job %s has only %d TAP logs, skipping", 123, 2)
@@ -512,7 +512,7 @@ def mock_job_response(url):
                 {"file1:test2"},  # Job 122 failures (different)
             ]
 
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
 
             mock_log.info.assert_any_call(
                 "No common failures across clone chain. Tagging as PASSED."
@@ -535,7 +535,7 @@ def mock_job_response(url):
         mock_get_job.side_effect = mock_job_response
 
         with pytest.raises(SystemExit) as exc_info:
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
 
         assert exc_info.value.code == 0
         mock_log.info.assert_any_call("No logs found in chain. Exiting")
@@ -564,7 +564,7 @@ def mock_job_response(url):
             {"file1:test2"},  # Job 122 failures (different)
         ]
 
-        main("http://openqa.example.com/tests/123")
+        main("http://openqa.example.com/tests/123", dry_run=True)
 
         mock_log.info.assert_any_call(
             "No common failures across clone chain. Tagging as PASSED."
@@ -576,7 +576,7 @@ def test_main_url_normalization(self):
             mock_resolve_clone_chain.return_value = [123]
 
             with pytest.raises(SystemExit):
-                main("openqa.example.com/tests/123")  # No scheme
+                main("openqa.example.com/tests/123", dry_run=True)  # No scheme
 
             # Verify that the URL was normalized to include https://
             mock_resolve_clone_chain.assert_called_with(
@@ -649,7 +649,7 @@ def mock_get(url, **kwargs):
         mock_session.get.side_effect = mock_get
 
         with patch("bats_review.log") as mock_log:
-            main("http://openqa.example.com/tests/123")
+            main("http://openqa.example.com/tests/123", dry_run=True)
             mock_log.info.assert_any_call(
                 "No common failures across clone chain. Tagging as PASSED."
             )