)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"b5e4f7f1849e9504556e25886fc551a7d738522c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20e5e94a_51f8abba","updated":"2025-03-21 16:21:30.000000000","message":"I think this is a good improvement. In follow-up patches, we might consider if feasible to check for the number of actions as well to assert if the number of migrations is the expected (although this may prove complicated in some tests)","commit_id":"f9d1c38c786de8eba992c2b996596c4cd2536167"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"585bc2a90cb3cbc0c4fa0d875631b4760406d2b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5a074bf5_6258b312","updated":"2025-03-26 13:10:39.000000000","message":"Looks good, left few suggestions!","commit_id":"6145ccebc3c39f7325a42191977b1a3bb2985bd2"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"00aa5b750cf584316191cee408b044145875c3b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7caefe06_ca241111","updated":"2025-03-26 13:32:01.000000000","message":"Thanks, done","commit_id":"16c6d651634dfb0aef08872bbbb8bc6a2d95c2c7"}],"watcher_tempest_plugin/tests/scenario/base.py":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"585bc2a90cb3cbc0c4fa0d875631b4760406d2b3","unresolved":true,"context_lines":[{"line_number":645,"context_line":"        return True"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def execute_strategy(self, goal_name, strategy_name,"},{"line_number":648,"context_line":"                         expected_actions\u003dNone, **audit_kwargs):"},{"line_number":649,"context_line":"        \"\"\"Execute an action plan based on the specific strategy"},{"line_number":650,"context_line":""},{"line_number":651,"context_line":"        - create an audit template with the specific strategy"}],"source_content_type":"text/x-python","patch_set":2,"id":"d32e59bd_febd25bf","line":648,"range":{"start_line":648,"start_character":25,"end_line":648,"end_character":47},"updated":"2025-03-26 13:10:39.000000000","message":"Since we are stating `expected_actions` is a list, can we define the default value as list like this instead of None.\n```\nexpected_actions\u003d[]\n```","commit_id":"6145ccebc3c39f7325a42191977b1a3bb2985bd2"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"00aa5b750cf584316191cee408b044145875c3b5","unresolved":false,"context_lines":[{"line_number":645,"context_line":"        return True"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"    def execute_strategy(self, goal_name, strategy_name,"},{"line_number":648,"context_line":"                         expected_actions\u003dNone, **audit_kwargs):"},{"line_number":649,"context_line":"        \"\"\"Execute an action plan based on the specific strategy"},{"line_number":650,"context_line":""},{"line_number":651,"context_line":"        - create an audit template with the specific strategy"}],"source_content_type":"text/x-python","patch_set":2,"id":"9b1fc872_10a5d8ee","line":648,"range":{"start_line":648,"start_character":25,"end_line":648,"end_character":47},"in_reply_to":"d32e59bd_febd25bf","updated":"2025-03-26 13:32:01.000000000","message":"Done","commit_id":"6145ccebc3c39f7325a42191977b1a3bb2985bd2"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"585bc2a90cb3cbc0c4fa0d875631b4760406d2b3","unresolved":true,"context_lines":[{"line_number":651,"context_line":"        - create an audit template with the specific strategy"},{"line_number":652,"context_line":"        - run the audit to create an action plan"},{"line_number":653,"context_line":"        - get the action plan"},{"line_number":654,"context_line":"        - check all the action types in the expected_actions"},{"line_number":655,"context_line":"          list is found in the action plan"},{"line_number":656,"context_line":"        - run the action plan"},{"line_number":657,"context_line":"        - get results and make sure it succeeded"},{"line_number":658,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"c9d49803_3cb957c1","line":655,"range":{"start_line":654,"start_character":10,"end_line":655,"end_character":42},"updated":"2025-03-26 13:10:39.000000000","message":"I can improve the wording like this\n```\nVerify that all action types in the expected_actions list are present in the action plan.\n```","commit_id":"6145ccebc3c39f7325a42191977b1a3bb2985bd2"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"00aa5b750cf584316191cee408b044145875c3b5","unresolved":false,"context_lines":[{"line_number":651,"context_line":"        - create an audit template with the specific strategy"},{"line_number":652,"context_line":"        - run the audit to create an action plan"},{"line_number":653,"context_line":"        - get the action plan"},{"line_number":654,"context_line":"        - check all the action types in the expected_actions"},{"line_number":655,"context_line":"          list is found in the action plan"},{"line_number":656,"context_line":"        - run the action plan"},{"line_number":657,"context_line":"        - get results and make sure it succeeded"},{"line_number":658,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"94adc20f_c983c01d","line":655,"range":{"start_line":654,"start_character":10,"end_line":655,"end_character":42},"in_reply_to":"c9d49803_3cb957c1","updated":"2025-03-26 13:32:01.000000000","message":"Done","commit_id":"6145ccebc3c39f7325a42191977b1a3bb2985bd2"}]}