)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"0ba8b57b57e81684fb9c37828a374ab57db53d60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"25f80f27_97c8c82b","updated":"2025-07-10 10:17:41.000000000","message":"thanks for improving the test coverage, looks good.","commit_id":"d80dce7507d73042a931d8970061d9842960bae5"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"527874c72363bd0fcf4945d40c006739941b4403","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a56ca4cd_43a2b7a3","updated":"2025-09-16 18:04:36.000000000","message":"there is a lot more that can be done to improve the unit tests but this is a good start.","commit_id":"821147547816888bf8e63ab8c03635792a0f564f"}],"watcher/tests/decision_engine/strategy/strategies/test_zone_migration.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a6c0adb44fec7f4afdda109dece332f8303977f9","unresolved":true,"context_lines":[{"line_number":40,"context_line":"        self.m_s_model \u003d p_s_model.start()"},{"line_number":41,"context_line":"        self.addCleanup(p_s_model.stop)"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"        model \u003d self.fake_c_cluster.generate_scenario_1()"},{"line_number":44,"context_line":"        self.m_c_model.return_value \u003d model"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        model \u003d self.fake_s_cluster.generate_scenario_1()"},{"line_number":47,"context_line":"        self.m_s_model.return_value \u003d model"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        self.input_parameters \u003d {"},{"line_number":50,"context_line":"            \"storage_pools\": ["}],"source_content_type":"text/x-python","patch_set":3,"id":"e0139c72_c2e609d9","line":47,"range":{"start_line":43,"start_character":0,"end_line":47,"end_character":43},"updated":"2025-09-16 18:35:26.000000000","message":"I was trying to understand how these resources are connected with the ones created in tests. They don\u0027t match (names/ids), so I\u0027m not sure how this model part is working. Both instance_list and volume_list don\u0027t seems in sync with the scenario_1? I also think that there are still place for improvements in this class..","commit_id":"821147547816888bf8e63ab8c03635792a0f564f"}]}
