Merge pull request #21 from lsst/tickets/DM-44431

DM-44431: Fix middleware test breakage

tests/test_prod_outputs.py

@@ -152,19 +152,19 @@ def check_step1_manifest_checker(self, helper: OutputRepoTests) -> None:
         # are where we expect them to be, a second time.
         hr_summary_1 = report_1.to_summary_dict(helper.butler, human_readable=True)
         failures = hr_summary_1["_mock_calibrate"]["errors"]
-        failed_visits = failures["data_id"]["visit"]
-        self.assertEqual(failed_visits, 18202)
-        self.assertTrue(failures["error"][0].startswith("Execution of task '_mock_calibrate' on quantum"))
-        self.assertEqual(hr_summary_1["_mock_isr"]["outputs"]["_mock_postISRCCD"]["produced"], 36)
+        for failure in failures:
+            self.assertEqual(failure["data_id"]["visit"], 18202)
+            self.assertTrue(failure["error"][0].startswith("Execution of task '_mock_calibrate' on quantum"))
+            self.assertEqual(hr_summary_1["_mock_isr"]["outputs"]["_mock_postISRCCD"]["produced"], 36)
         # This task should have succeeded in attempt1 and should not have been
         # included in attempt2.
         qg_2 = helper.get_quantum_graph("step1", "i-attempt2")
         report_2 = QuantumGraphExecutionReport.make_reports(helper.butler, qg_2)
         summary_2 = report_2.to_summary_dict(helper.butler)
-        self.assertDictEqual(summary_2["_mock_calibrate"]["failed_quanta"], {})  # is empty ??
+        self.assertEqual(summary_2["_mock_calibrate"]["failed_quanta"], {})  # is empty ??
         # Making sure it works with the human-readable version,
         hr_summary_2 = report_2.to_summary_dict(helper.butler, human_readable=True)
-        self.assertDictEqual(hr_summary_2["_mock_calibrate"]["failed_quanta"], {})
+        self.assertEqual(hr_summary_2["_mock_calibrate"]["failed_quanta"], [])
 
     def test_step1_manifest_checker_qbb(self) -> None:
         self.check_step1_manifest_checker(self.qbb)