)]}'
{"watcher_tempest_plugin/services/infra_optim/v1/json/client.py":[{"author":{"_account_id":13289,"name":"Taylor Peoples","email":"tpeoples@us.ibm.com","username":"tpeoples"},"change_message_id":"dd030d9cda103db182d92ea41cc51113fe8754a2","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        return self._list_request(\u0027/actions/detail\u0027, **kwargs)"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    @base.handle_errors"},{"line_number":271,"context_line":"    def filter_actions_by_goal(self, action_plan_uuid):"},{"line_number":272,"context_line":"        \"\"\"Gets an audit template associated with given action_plan_uuid."},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"        :param action_plan_uuid: action_plan_uuid identifier"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a68dd71_f75175d9","line":271,"range":{"start_line":271,"start_character":23,"end_line":271,"end_character":30},"updated":"2016-01-27 06:03:44.000000000","message":"by_action_plan, and is this method needed?  Seems like list_actions can get us the same thing via the kwargs.","commit_id":"d5358914589e00ee960da18715f74465cf523c96"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"8a634b80dcb9b2898bd32cde715762c10d0d6d2d","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        return self._list_request(\u0027/actions/detail\u0027, **kwargs)"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    @base.handle_errors"},{"line_number":271,"context_line":"    def filter_actions_by_goal(self, action_plan_uuid):"},{"line_number":272,"context_line":"        \"\"\"Gets an audit template associated with given action_plan_uuid."},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"        :param action_plan_uuid: action_plan_uuid identifier"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_06dbfa1c","line":271,"range":{"start_line":271,"start_character":23,"end_line":271,"end_character":30},"in_reply_to":"9a68dd71_f75175d9","updated":"2016-01-27 10:04:48.000000000","message":"Well, it wasn\u0027t needed. As for the other filtering methods, I ended up removing them as well so we keep the amount of code we maintain to a minimum.","commit_id":"d5358914589e00ee960da18715f74465cf523c96"},{"author":{"_account_id":13289,"name":"Taylor Peoples","email":"tpeoples@us.ibm.com","username":"tpeoples"},"change_message_id":"dd030d9cda103db182d92ea41cc51113fe8754a2","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    @base.handle_errors"},{"line_number":271,"context_line":"    def filter_actions_by_goal(self, action_plan_uuid):"},{"line_number":272,"context_line":"        \"\"\"Gets an audit template associated with given action_plan_uuid."},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"        :param action_plan_uuid: action_plan_uuid identifier"},{"line_number":275,"context_line":"        :return: Serialized audit template as a dictionary."},{"line_number":276,"context_line":"        \"\"\""},{"line_number":277,"context_line":"        return self._list_request(\u0027/actions\u0027,"},{"line_number":278,"context_line":"                                  action_plan_uuid\u003daction_plan_uuid)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a68dd71_7287d3d7","line":275,"range":{"start_line":272,"start_character":8,"end_line":275,"end_character":59},"updated":"2016-01-27 06:03:44.000000000","message":"Needs to be updated if we\u0027re keeping this.","commit_id":"d5358914589e00ee960da18715f74465cf523c96"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"8a634b80dcb9b2898bd32cde715762c10d0d6d2d","unresolved":false,"context_lines":[{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    @base.handle_errors"},{"line_number":271,"context_line":"    def filter_actions_by_goal(self, action_plan_uuid):"},{"line_number":272,"context_line":"        \"\"\"Gets an audit template associated with given action_plan_uuid."},{"line_number":273,"context_line":""},{"line_number":274,"context_line":"        :param action_plan_uuid: action_plan_uuid identifier"},{"line_number":275,"context_line":"        :return: Serialized audit template as a dictionary."},{"line_number":276,"context_line":"        \"\"\""},{"line_number":277,"context_line":"        return self._list_request(\u0027/actions\u0027,"},{"line_number":278,"context_line":"                                  action_plan_uuid\u003daction_plan_uuid)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_e6081e94","line":275,"range":{"start_line":272,"start_character":8,"end_line":275,"end_character":59},"in_reply_to":"9a68dd71_7287d3d7","updated":"2016-01-27 10:04:48.000000000","message":"Done","commit_id":"d5358914589e00ee960da18715f74465cf523c96"}],"watcher_tempest_plugin/tests/api/admin/base.py":[{"author":{"_account_id":13289,"name":"Taylor Peoples","email":"tpeoples@us.ibm.com","username":"tpeoples"},"change_message_id":"dd030d9cda103db182d92ea41cc51113fe8754a2","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    @classmethod"},{"line_number":182,"context_line":"    @creates(\u0027action_plan\u0027)"},{"line_number":183,"context_line":"    def start_action_plan(cls, audit_uuid, type\u003d\u0027ONESHOT\u0027,"},{"line_number":184,"context_line":"                          state\u003d\u0027PENDING\u0027, deadline\u003dNone):"},{"line_number":185,"context_line":"        \"\"\"Wrapper utility for creating a test action plan"},{"line_number":186,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a68dd71_b2930b4f","side":"PARENT","line":183,"updated":"2016-01-27 06:03:44.000000000","message":"Seems like this might be a good name for a function that wraps the update_action_plan call with the state updated to TRIGGERED, but just a nit.  OK to remove this since it isn\u0027t being used.","commit_id":"1aa49ee334ed2fdf05181ae7f3d651e58644285f"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"8a634b80dcb9b2898bd32cde715762c10d0d6d2d","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    @classmethod"},{"line_number":182,"context_line":"    @creates(\u0027action_plan\u0027)"},{"line_number":183,"context_line":"    def start_action_plan(cls, audit_uuid, type\u003d\u0027ONESHOT\u0027,"},{"line_number":184,"context_line":"                          state\u003d\u0027PENDING\u0027, deadline\u003dNone):"},{"line_number":185,"context_line":"        \"\"\"Wrapper utility for creating a test action plan"},{"line_number":186,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7a5de9d1_49db45ee","side":"PARENT","line":183,"in_reply_to":"9a68dd71_b2930b4f","updated":"2016-01-27 10:04:48.000000000","message":"I agree on the good naming, but the thing is that this function will probably never be used in the API tests apart from the one and only test which does start an action plan for to test that the PATCH request works. So just like for the filtering methods that I removed, I guess that I\u0027ll remove it for now and add it back later on if we actually need to test that bit in more detail.","commit_id":"1aa49ee334ed2fdf05181ae7f3d651e58644285f"}]}