)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9ba3a4ca11e054116dcc18d31377be201dc13741","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aec3061c_22da94e2","updated":"2025-06-05 17:47:21.000000000","message":"lgtm, thanks for the details on commit msg","commit_id":"167fb61b4ec5d059871d3eb484d07d4976a06ef0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e1f777d523941035287f03143b60399cb8f431c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c4bfab62_dd3b58f2","updated":"2025-06-09 05:32:22.000000000","message":"thank you for adding the tests!","commit_id":"167fb61b4ec5d059871d3eb484d07d4976a06ef0"}],"watcher/tests/decision_engine/strategy/strategies/test_zone_migration.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"37a5557ac2f3fe58faaa841d422c5e151623392c","unresolved":true,"context_lines":[{"line_number":84,"context_line":"            new_callable\u003dmock.PropertyMock"},{"line_number":85,"context_line":"        )"},{"line_number":86,"context_line":"        self.m_with_attached_volume \u003d p_with_attached_volume.start()"},{"line_number":87,"context_line":"        self.addCleanup(p_with_attached_volume.stop)"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"        model \u003d self.fake_c_cluster.generate_scenario_1()"},{"line_number":90,"context_line":"        self.m_c_model.return_value \u003d model"}],"source_content_type":"text/x-python","patch_set":1,"id":"5c3cf004_78271afe","line":87,"updated":"2025-06-13 11:43:39.000000000","message":"as an aside we are oging to need to rewrite almost all of watcher untit est as this is not the correc tpattern to be following.\n\nWe should not be using property models like this.\n\nwe shjopudo be usein ga realy ZoneMigration object without any mocks fo the feils and only mockign in a limited fashion\n\nthis signifcanlty redusrce the value of the test cover to the point where it may as well not exist in some case.\n\n\nwhen i initally looked at the code i did not knotce this is the partter it was usign but there is a lot of technialy debt that we will need to adress.\n\nim sort of ok with adding this for now but goring forward for new test we shoudl aovid doint this.","commit_id":"167fb61b4ec5d059871d3eb484d07d4976a06ef0"}]}