Keep json report in artifacts for easier debugging (#550)

* log wingha status and components

* log wingha status and components

* add json report to artifacts

* separate report for headed tests

* always use artifacts

* reset logging

Author: ben-c-at-moz

.cron.yml

@@ -1,43 +1,41 @@
 ---
-
 jobs:
   - name: new-beta-qa
     job:
       type: decision-task
       treeherder-symbol: STARb
       target-tasks-method: new_beta_qa
     when:
-      - {hour: 0, minute: 15}
-      - {hour: 1, minute: 15}
-      - {hour: 2, minute: 15}
-      - {hour: 3, minute: 15}
-      - {hour: 4, minute: 15}
-      - {hour: 5, minute: 15}
-      - {hour: 6, minute: 15}
-      - {hour: 7, minute: 15}
-      - {hour: 8, minute: 15}
-      - {hour: 9, minute: 15}
-      - {hour: 10, minute: 15}
-      - {hour: 11, minute: 15}
-      - {hour: 12, minute: 15}
-      - {hour: 13, minute: 45}
-      - {hour: 14, minute: 15}
-      - {hour: 15, minute: 15}
-      - {hour: 16, minute: 15}
-      - {hour: 17, minute: 15}
-      - {hour: 18, minute: 15}
-      - {hour: 19, minute: 15}
-      - {hour: 20, minute: 15}
-      - {hour: 21, minute: 15}
-      - {hour: 22, minute: 15}
-      - {hour: 23, minute: 15}
+      - { hour: 0, minute: 15 }
+      - { hour: 1, minute: 15 }
+      - { hour: 2, minute: 15 }
+      - { hour: 3, minute: 15 }
+      - { hour: 4, minute: 15 }
+      - { hour: 5, minute: 15 }
+      - { hour: 6, minute: 15 }
+      - { hour: 7, minute: 15 }
+      - { hour: 8, minute: 15 }
+      - { hour: 9, minute: 15 }
+      - { hour: 10, minute: 15 }
+      - { hour: 11, minute: 15 }
+      - { hour: 12, minute: 15 }
+      - { hour: 13, minute: 15 }
+      - { hour: 14, minute: 15 }
+      - { hour: 15, minute: 15 }
+      - { hour: 16, minute: 15 }
+      - { hour: 17, minute: 15 }
+      - { hour: 18, minute: 15 }
+      - { hour: 19, minute: 15 }
+      - { hour: 20, minute: 15 }
+      - { hour: 21, minute: 15 }
+      - { hour: 22, minute: 15 }
+      - { hour: 23, minute: 15 }
 
   - name: new-devedition-qa
     job:
       type: decision-task
       treeherder-symbol: STARd
       target-tasks-method: new_devedition_qa
     when:
-      - {hour: 11, minute: 15}
-      - {hour: 19, minute: 30}
-      - {hour: 23, minute: 15}
+      - { hour: 11, minute: 15 }
+      - { hour: 23, minute: 15 }

.github/workflows/smoke.yml

@@ -187,7 +187,6 @@ jobs:
     needs:
       - Smoke-Windows
       - Smoke-MacOS
-    if: ${{ always() && github.event_name == 'pull_request' }}
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4

ci_pyproject.toml

@@ -16,7 +16,7 @@ markers = [
 testpaths = [
     "tests"
 ]
-addopts = "-vs --ci --run-headless --json-report --reruns 3 --reruns-delay 3 -m 'not incident and not unstable and not headed' --html=artifacts/report.html"
+addopts = "-vs --ci --run-headless --json-report --json-report-file artifacts/report.json --reruns 3 --reruns-delay 3 -m 'not incident and not unstable and not headed' --html=artifacts/report.html"
 
 [tool.ruff]
 target-version = "py310"

ci_pyproject_headed.toml

@@ -16,7 +16,7 @@ markers = [
 testpaths = [
     "tests"
 ]
-addopts = "-vs --ci --json-report --reruns 3 --reruns-delay 2 -m 'not incident and not unstable and headed' --html=artifacts/report_headed.html"
+addopts = "-vs --ci --json-report --json-report-file artifacts/report_headed.json --reruns 3 --reruns-delay 2 -m 'not incident and not unstable and headed' --html=artifacts/report_headed.html"
 
 [tool.ruff]
 target-version = "py310"

modules/testrail.py

@@ -173,7 +173,7 @@ def update_test_case(self, case_id, **kwargs):
         """Given a test case id, update according to keyword args"""
         if not kwargs:
             return None
-        return self.client.send_post(f"update_case/{case_id}", **kwargs)
+        return self.client.send_post(f"update_case/{case_id}", kwargs)
 
     def create_test_run_on_plan_entry(
         self, plan_id, entry_id, config_ids, description=None, case_ids=None