)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":10,"context_line":"concerning the already-declared foreign keys."},{"line_number":11,"context_line":"I also modified the DB query building to now handle a new \u0027eager\u0027"},{"line_number":12,"context_line":"parameter that, if True, is responsible to also fetch the data"},{"line_number":13,"context_line":"relative to these \u0027parent\u0027 DB entities."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ieea181af9a4b173c54621dcc6c549161f5a35aeb"},{"line_number":16,"context_line":"Partially-Implements: blueprint watcher-versioned-objects"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a95cdbc_a31b1ec9","line":13,"updated":"2016-10-10 09:26:51.000000000","message":"please note that it is not recursive.","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":10,"context_line":"concerning the already-declared foreign keys."},{"line_number":11,"context_line":"I also modified the DB query building to now handle a new \u0027eager\u0027"},{"line_number":12,"context_line":"parameter that, if True, is responsible to also fetch the data"},{"line_number":13,"context_line":"relative to these \u0027parent\u0027 DB entities."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ieea181af9a4b173c54621dcc6c549161f5a35aeb"},{"line_number":16,"context_line":"Partially-Implements: blueprint watcher-versioned-objects"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_72a80a8e","line":13,"in_reply_to":"1a95cdbc_a31b1ec9","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/db/api.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        :param context: The security context"},{"line_number":80,"context_line":"        :param goal_id: The ID of a goal"},{"line_number":81,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":82,"context_line":"        :returns: A goal"},{"line_number":83,"context_line":"        :raises: :py:class:`~.GoalNotFound`"},{"line_number":84,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_183efdd9","line":81,"range":{"start_line":81,"start_character":42,"end_line":81,"end_character":54},"updated":"2016-10-10 09:26:51.000000000","message":"Term \u0027Foreign key\u0027 refers to a SQL constraint. Maybe this term is not well used here.","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        :param context: The security context"},{"line_number":80,"context_line":"        :param goal_id: The ID of a goal"},{"line_number":81,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":82,"context_line":"        :returns: A goal"},{"line_number":83,"context_line":"        :raises: :py:class:`~.GoalNotFound`"},{"line_number":84,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_92ae7e7f","line":81,"range":{"start_line":81,"start_character":42,"end_line":81,"end_character":54},"in_reply_to":"1a95cdbc_183efdd9","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":220,"context_line":"        \"\"\"Destroy a strategy."},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        :param strategy_uuid: The UUID of a strategy"},{"line_number":223,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":224,"context_line":"        :raises: :py:class:`~.StrategyNotFound`"},{"line_number":225,"context_line":"        \"\"\""},{"line_number":226,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_c3002a04","line":223,"range":{"start_line":223,"start_character":0,"end_line":223,"end_character":76},"updated":"2016-10-10 09:26:51.000000000","message":"remove this line.","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":220,"context_line":"        \"\"\"Destroy a strategy."},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        :param strategy_uuid: The UUID of a strategy"},{"line_number":223,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":224,"context_line":"        :raises: :py:class:`~.StrategyNotFound`"},{"line_number":225,"context_line":"        \"\"\""},{"line_number":226,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_92e59edd","line":223,"range":{"start_line":223,"start_character":0,"end_line":223,"end_character":76},"in_reply_to":"1a95cdbc_c3002a04","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":411,"context_line":"        \"\"\"Destroy an audit and all associated action plans."},{"line_number":412,"context_line":""},{"line_number":413,"context_line":"        :param audit_id: The id or uuid of an audit."},{"line_number":414,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":415,"context_line":"        :raises: :py:class:`~.AuditNotFound`"},{"line_number":416,"context_line":"        \"\"\""},{"line_number":417,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_23b70ef6","line":414,"range":{"start_line":414,"start_character":0,"end_line":414,"end_character":76},"updated":"2016-10-10 09:26:51.000000000","message":"remove this line too.","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":411,"context_line":"        \"\"\"Destroy an audit and all associated action plans."},{"line_number":412,"context_line":""},{"line_number":413,"context_line":"        :param audit_id: The id or uuid of an audit."},{"line_number":414,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":415,"context_line":"        :raises: :py:class:`~.AuditNotFound`"},{"line_number":416,"context_line":"        \"\"\""},{"line_number":417,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_f23eda26","line":414,"range":{"start_line":414,"start_character":0,"end_line":414,"end_character":76},"in_reply_to":"1a95cdbc_23b70ef6","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":500,"context_line":"        \"\"\"Destroy a action and all associated interfaces."},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"        :param action_id: The id or uuid of a action."},{"line_number":503,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":504,"context_line":"        :raises: :py:class:`~.ActionNotFound`"},{"line_number":505,"context_line":"        :raises: :py:class:`~.ActionReferenced`"},{"line_number":506,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_03a93290","line":503,"range":{"start_line":503,"start_character":29,"end_line":503,"end_character":30},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":500,"context_line":"        \"\"\"Destroy a action and all associated interfaces."},{"line_number":501,"context_line":""},{"line_number":502,"context_line":"        :param action_id: The id or uuid of a action."},{"line_number":503,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":504,"context_line":"        :raises: :py:class:`~.ActionNotFound`"},{"line_number":505,"context_line":"        :raises: :py:class:`~.ActionReferenced`"},{"line_number":506,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_5214e69e","line":503,"range":{"start_line":503,"start_character":29,"end_line":503,"end_character":30},"in_reply_to":"1a95cdbc_03a93290","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":581,"context_line":"        \"\"\"Destroy an action plan and all associated interfaces."},{"line_number":582,"context_line":""},{"line_number":583,"context_line":"        :param action_plan_id: The id or uuid of a action plan."},{"line_number":584,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":585,"context_line":"        :raises: :py:class:`~.ActionPlanNotFound`"},{"line_number":586,"context_line":"        :raises: :py:class:`~.ActionPlanReferenced`"},{"line_number":587,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_038052f6","line":584,"range":{"start_line":584,"start_character":0,"end_line":584,"end_character":76},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":581,"context_line":"        \"\"\"Destroy an action plan and all associated interfaces."},{"line_number":582,"context_line":""},{"line_number":583,"context_line":"        :param action_plan_id: The id or uuid of a action plan."},{"line_number":584,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":585,"context_line":"        :raises: :py:class:`~.ActionPlanNotFound`"},{"line_number":586,"context_line":"        :raises: :py:class:`~.ActionPlanReferenced`"},{"line_number":587,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_920afe77","line":584,"range":{"start_line":584,"start_character":0,"end_line":584,"end_character":76},"in_reply_to":"1a95cdbc_038052f6","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":689,"context_line":"        \"\"\"Destroy an efficacy indicator."},{"line_number":690,"context_line":""},{"line_number":691,"context_line":"        :param efficacy_indicator_uuid: The UUID of an efficacy indicator"},{"line_number":692,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":693,"context_line":"        :raises: :py:class:`~.EfficacyIndicatorNotFound`"},{"line_number":694,"context_line":"        \"\"\""},{"line_number":695,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_23b38eea","line":692,"range":{"start_line":692,"start_character":0,"end_line":692,"end_character":76},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":689,"context_line":"        \"\"\"Destroy an efficacy indicator."},{"line_number":690,"context_line":""},{"line_number":691,"context_line":"        :param efficacy_indicator_uuid: The UUID of an efficacy indicator"},{"line_number":692,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":693,"context_line":"        :raises: :py:class:`~.EfficacyIndicatorNotFound`"},{"line_number":694,"context_line":"        \"\"\""},{"line_number":695,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_b20f0287","line":692,"range":{"start_line":692,"start_character":0,"end_line":692,"end_character":76},"in_reply_to":"1a95cdbc_23b38eea","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":777,"context_line":"        \"\"\"Destroy a scoring engine."},{"line_number":778,"context_line":""},{"line_number":779,"context_line":"        :param scoring_engine_id: The id of a scoring engine."},{"line_number":780,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":781,"context_line":"        :raises: :py:class:`~.ScoringEngineNotFound`"},{"line_number":782,"context_line":"        \"\"\""},{"line_number":783,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_c3de6a10","line":780,"range":{"start_line":780,"start_character":0,"end_line":780,"end_character":76},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":777,"context_line":"        \"\"\"Destroy a scoring engine."},{"line_number":778,"context_line":""},{"line_number":779,"context_line":"        :param scoring_engine_id: The id of a scoring engine."},{"line_number":780,"context_line":"        :param eager: If True, also loads foreign keys data (Default: False)"},{"line_number":781,"context_line":"        :raises: :py:class:`~.ScoringEngineNotFound`"},{"line_number":782,"context_line":"        \"\"\""},{"line_number":783,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_f225fa08","line":780,"range":{"start_line":780,"start_character":0,"end_line":780,"end_character":76},"in_reply_to":"1a95cdbc_c3de6a10","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/db/sqlalchemy/api.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    :param query: Initial query to add filter to."},{"line_number":86,"context_line":"    :param value: Value for filtering results by."},{"line_number":87,"context_line":"    :return: Modified query."},{"line_number":88,"context_line":"    \"\"\""},{"line_number":89,"context_line":"    if utils.is_int_like(value):"},{"line_number":90,"context_line":"        return query.filter_by(id\u003dvalue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_c129cfe7","line":87,"updated":"2016-10-10 09:26:51.000000000","message":"please, add eager param into docstring","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    :param query: Initial query to add filter to."},{"line_number":86,"context_line":"    :param value: Value for filtering results by."},{"line_number":87,"context_line":"    :return: Modified query."},{"line_number":88,"context_line":"    \"\"\""},{"line_number":89,"context_line":"    if utils.is_int_like(value):"},{"line_number":90,"context_line":"        return query.filter_by(id\u003dvalue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_92cf5e28","line":87,"in_reply_to":"1a95cdbc_c129cfe7","updated":"2016-10-18 15:27:53.000000000","message":"The eager param was actually not necessary here so I removed it.","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"            query.delete()"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"    def _add_goals_filters(self, query, filters, eager\u003dFalse):"},{"line_number":302,"context_line":"        if filters is None:"},{"line_number":303,"context_line":"            filters \u003d {}"},{"line_number":304,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_4163ff41","line":301,"range":{"start_line":301,"start_character":49,"end_line":301,"end_character":54},"updated":"2016-10-10 09:26:51.000000000","message":"not used in the method","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":298,"context_line":""},{"line_number":299,"context_line":"            query.delete()"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":"    def _add_goals_filters(self, query, filters, eager\u003dFalse):"},{"line_number":302,"context_line":"        if filters is None:"},{"line_number":303,"context_line":"            filters \u003d {}"},{"line_number":304,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_d2dd56e1","line":301,"range":{"start_line":301,"start_character":49,"end_line":301,"end_character":54},"in_reply_to":"1a95cdbc_4163ff41","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":308,"context_line":"            query\u003dquery, model\u003dmodels.Goal, filters\u003dfilters,"},{"line_number":309,"context_line":"            plain_fields\u003dplain_fields)"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    def _add_strategies_filters(self, query, filters, eager\u003dFalse):"},{"line_number":312,"context_line":"        plain_fields \u003d [\u0027uuid\u0027, \u0027name\u0027, \u0027display_name\u0027, \u0027goal_id\u0027]"},{"line_number":313,"context_line":"        join_fieldmap \u003d JoinMap("},{"line_number":314,"context_line":"            goal_uuid\u003dNaturalJoinFilter("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_61663b52","line":311,"range":{"start_line":311,"start_character":54,"end_line":311,"end_character":59},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":308,"context_line":"            query\u003dquery, model\u003dmodels.Goal, filters\u003dfilters,"},{"line_number":309,"context_line":"            plain_fields\u003dplain_fields)"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    def _add_strategies_filters(self, query, filters, eager\u003dFalse):"},{"line_number":312,"context_line":"        plain_fields \u003d [\u0027uuid\u0027, \u0027name\u0027, \u0027display_name\u0027, \u0027goal_id\u0027]"},{"line_number":313,"context_line":"        join_fieldmap \u003d JoinMap("},{"line_number":314,"context_line":"            goal_uuid\u003dNaturalJoinFilter("}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_f2da5ae7","line":311,"range":{"start_line":311,"start_character":54,"end_line":311,"end_character":59},"in_reply_to":"1a95cdbc_61663b52","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":319,"context_line":"            query\u003dquery, model\u003dmodels.Strategy, filters\u003dfilters,"},{"line_number":320,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":321,"context_line":""},{"line_number":322,"context_line":"    def _add_audit_templates_filters(self, query, filters, eager\u003dFalse):"},{"line_number":323,"context_line":"        if filters is None:"},{"line_number":324,"context_line":"            filters \u003d {}"},{"line_number":325,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_c1570f56","line":322,"range":{"start_line":322,"start_character":59,"end_line":322,"end_character":64},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":319,"context_line":"            query\u003dquery, model\u003dmodels.Strategy, filters\u003dfilters,"},{"line_number":320,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":321,"context_line":""},{"line_number":322,"context_line":"    def _add_audit_templates_filters(self, query, filters, eager\u003dFalse):"},{"line_number":323,"context_line":"        if filters is None:"},{"line_number":324,"context_line":"            filters \u003d {}"},{"line_number":325,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_52f06667","line":322,"range":{"start_line":322,"start_character":59,"end_line":322,"end_character":64},"in_reply_to":"1a95cdbc_c1570f56","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":340,"context_line":"            query\u003dquery, model\u003dmodels.AuditTemplate, filters\u003dfilters,"},{"line_number":341,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    def _add_audits_filters(self, query, filters, eager\u003dFalse):"},{"line_number":344,"context_line":"        if filters is None:"},{"line_number":345,"context_line":"            filters \u003d {}"},{"line_number":346,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_214993b7","line":343,"range":{"start_line":343,"start_character":50,"end_line":343,"end_character":55},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":340,"context_line":"            query\u003dquery, model\u003dmodels.AuditTemplate, filters\u003dfilters,"},{"line_number":341,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"    def _add_audits_filters(self, query, filters, eager\u003dFalse):"},{"line_number":344,"context_line":"        if filters is None:"},{"line_number":345,"context_line":"            filters \u003d {}"},{"line_number":346,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_72f56a55","line":343,"range":{"start_line":343,"start_character":50,"end_line":343,"end_character":55},"in_reply_to":"1a95cdbc_214993b7","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":357,"context_line":"            query\u003dquery, model\u003dmodels.Audit, filters\u003dfilters,"},{"line_number":358,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"    def _add_action_plans_filters(self, query, filters, eager\u003dFalse):"},{"line_number":361,"context_line":"        if filters is None:"},{"line_number":362,"context_line":"            filters \u003d {}"},{"line_number":363,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_c1de2fdc","line":360,"range":{"start_line":360,"start_character":56,"end_line":360,"end_character":61},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":357,"context_line":"            query\u003dquery, model\u003dmodels.Audit, filters\u003dfilters,"},{"line_number":358,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":"    def _add_action_plans_filters(self, query, filters, eager\u003dFalse):"},{"line_number":361,"context_line":"        if filters is None:"},{"line_number":362,"context_line":"            filters \u003d {}"},{"line_number":363,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_d204764b","line":360,"range":{"start_line":360,"start_character":56,"end_line":360,"end_character":61},"in_reply_to":"1a95cdbc_c1de2fdc","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":375,"context_line":"            query\u003dquery, model\u003dmodels.ActionPlan, filters\u003dfilters,"},{"line_number":376,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":377,"context_line":""},{"line_number":378,"context_line":"    def _add_actions_filters(self, query, filters, eager\u003dFalse):"},{"line_number":379,"context_line":"        if filters is None:"},{"line_number":380,"context_line":"            filters \u003d {}"},{"line_number":381,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_e1d96bf4","line":378,"range":{"start_line":378,"start_character":51,"end_line":378,"end_character":56},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":375,"context_line":"            query\u003dquery, model\u003dmodels.ActionPlan, filters\u003dfilters,"},{"line_number":376,"context_line":"            plain_fields\u003dplain_fields, join_fieldmap\u003djoin_fieldmap)"},{"line_number":377,"context_line":""},{"line_number":378,"context_line":"    def _add_actions_filters(self, query, filters, eager\u003dFalse):"},{"line_number":379,"context_line":"        if filters is None:"},{"line_number":380,"context_line":"            filters \u003d {}"},{"line_number":381,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_f2017a39","line":378,"range":{"start_line":378,"start_character":51,"end_line":378,"end_character":56},"in_reply_to":"1a95cdbc_e1d96bf4","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":397,"context_line":""},{"line_number":398,"context_line":"        return query"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"    def _add_efficacy_indicators_filters(self, query, filters, eager\u003dFalse):"},{"line_number":401,"context_line":"        if filters is None:"},{"line_number":402,"context_line":"            filters \u003d {}"},{"line_number":403,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_21e0b395","line":400,"range":{"start_line":400,"start_character":63,"end_line":400,"end_character":68},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":397,"context_line":""},{"line_number":398,"context_line":"        return query"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"    def _add_efficacy_indicators_filters(self, query, filters, eager\u003dFalse):"},{"line_number":401,"context_line":"        if filters is None:"},{"line_number":402,"context_line":"            filters \u003d {}"},{"line_number":403,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_929d9e0c","line":400,"range":{"start_line":400,"start_character":63,"end_line":400,"end_character":68},"in_reply_to":"1a95cdbc_21e0b395","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":781,"context_line":""},{"line_number":782,"context_line":"        return self._do_update_action(action_id, values)"},{"line_number":783,"context_line":""},{"line_number":784,"context_line":"    def _do_update_action(self, action_id, values, eager\u003dFalse):"},{"line_number":785,"context_line":"        session \u003d get_session()"},{"line_number":786,"context_line":"        with session.begin():"},{"line_number":787,"context_line":"            query \u003d model_query(models.Action, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_1453f339","line":784,"range":{"start_line":784,"start_character":51,"end_line":784,"end_character":56},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":781,"context_line":""},{"line_number":782,"context_line":"        return self._do_update_action(action_id, values)"},{"line_number":783,"context_line":""},{"line_number":784,"context_line":"    def _do_update_action(self, action_id, values, eager\u003dFalse):"},{"line_number":785,"context_line":"        session \u003d get_session()"},{"line_number":786,"context_line":"        with session.begin():"},{"line_number":787,"context_line":"            query \u003d model_query(models.Action, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_cd6f7729","line":784,"range":{"start_line":784,"start_character":51,"end_line":784,"end_character":56},"in_reply_to":"1a95cdbc_1453f339","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":884,"context_line":""},{"line_number":885,"context_line":"        return self._do_update_action_plan(action_plan_id, values)"},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"    def _do_update_action_plan(self, action_plan_id, values, eager\u003dFalse):"},{"line_number":888,"context_line":"        session \u003d get_session()"},{"line_number":889,"context_line":"        with session.begin():"},{"line_number":890,"context_line":"            query \u003d model_query(models.ActionPlan, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_943d43df","line":887,"range":{"start_line":887,"start_character":61,"end_line":887,"end_character":66},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":884,"context_line":""},{"line_number":885,"context_line":"        return self._do_update_action_plan(action_plan_id, values)"},{"line_number":886,"context_line":""},{"line_number":887,"context_line":"    def _do_update_action_plan(self, action_plan_id, values, eager\u003dFalse):"},{"line_number":888,"context_line":"        session \u003d get_session()"},{"line_number":889,"context_line":"        with session.begin():"},{"line_number":890,"context_line":"            query \u003d model_query(models.ActionPlan, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_cd56975b","line":887,"range":{"start_line":887,"start_character":61,"end_line":887,"end_character":66},"in_reply_to":"1a95cdbc_943d43df","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":994,"context_line":""},{"line_number":995,"context_line":"    # ### SCORING ENGINES ### #"},{"line_number":996,"context_line":""},{"line_number":997,"context_line":"    def _add_scoring_engine_filters(self, query, filters, eager\u003dFalse):"},{"line_number":998,"context_line":"        if filters is None:"},{"line_number":999,"context_line":"            filters \u003d {}"},{"line_number":1000,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_7411975c","line":997,"range":{"start_line":997,"start_character":58,"end_line":997,"end_character":63},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":994,"context_line":""},{"line_number":995,"context_line":"    # ### SCORING ENGINES ### #"},{"line_number":996,"context_line":""},{"line_number":997,"context_line":"    def _add_scoring_engine_filters(self, query, filters, eager\u003dFalse):"},{"line_number":998,"context_line":"        if filters is None:"},{"line_number":999,"context_line":"            filters \u003d {}"},{"line_number":1000,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_ada24349","line":997,"range":{"start_line":997,"start_character":58,"end_line":997,"end_character":63},"in_reply_to":"1a95cdbc_7411975c","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"    def get_scoring_engine_list("},{"line_number":1008,"context_line":"            self, context, columns\u003dNone, filters\u003dNone, limit\u003dNone,"},{"line_number":1009,"context_line":"            marker\u003dNone, sort_key\u003dNone, sort_dir\u003dNone, eager\u003dFalse):"},{"line_number":1010,"context_line":"        query \u003d model_query(models.ScoringEngine)"},{"line_number":1011,"context_line":"        query \u003d self._add_scoring_engine_filters(query, filters)"},{"line_number":1012,"context_line":"        if not context.show_deleted:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_970b8523","line":1009,"range":{"start_line":1009,"start_character":55,"end_line":1009,"end_character":60},"updated":"2016-10-10 09:26:51.000000000","message":"ditto","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"    def get_scoring_engine_list("},{"line_number":1008,"context_line":"            self, context, columns\u003dNone, filters\u003dNone, limit\u003dNone,"},{"line_number":1009,"context_line":"            marker\u003dNone, sort_key\u003dNone, sort_dir\u003dNone, eager\u003dFalse):"},{"line_number":1010,"context_line":"        query \u003d model_query(models.ScoringEngine)"},{"line_number":1011,"context_line":"        query \u003d self._add_scoring_engine_filters(query, filters)"},{"line_number":1012,"context_line":"        if not context.show_deleted:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_0d924ff6","line":1009,"range":{"start_line":1009,"start_character":55,"end_line":1009,"end_character":60},"in_reply_to":"1a95cdbc_970b8523","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":427,"context_line":"        return _paginate_query(models.Goal, limit, marker,"},{"line_number":428,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":429,"context_line":""},{"line_number":430,"context_line":"    def create_goal(self, values):"},{"line_number":431,"context_line":"        # ensure defaults are present for new goals"},{"line_number":432,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":433,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_e38140c5","line":430,"range":{"start_line":430,"start_character":8,"end_line":430,"end_character":19},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":427,"context_line":"        return _paginate_query(models.Goal, limit, marker,"},{"line_number":428,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":429,"context_line":""},{"line_number":430,"context_line":"    def create_goal(self, values):"},{"line_number":431,"context_line":"        # ensure defaults are present for new goals"},{"line_number":432,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":433,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_a762c6ab","line":430,"range":{"start_line":430,"start_character":8,"end_line":430,"end_character":19},"in_reply_to":"fa6399be_e38140c5","updated":"2016-10-18 13:54:13.000000000","message":"IMHO, we should keep it as is because otherwise, we would have to change the signature of the method in watcher/dp/api.py as well.","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":496,"context_line":"        return _paginate_query(models.Strategy, limit, marker,"},{"line_number":497,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":498,"context_line":""},{"line_number":499,"context_line":"    def create_strategy(self, values):"},{"line_number":500,"context_line":"        # ensure defaults are present for new strategies"},{"line_number":501,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":502,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_43502c5f","line":499,"range":{"start_line":499,"start_character":8,"end_line":499,"end_character":23},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":496,"context_line":"        return _paginate_query(models.Strategy, limit, marker,"},{"line_number":497,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":498,"context_line":""},{"line_number":499,"context_line":"    def create_strategy(self, values):"},{"line_number":500,"context_line":"        # ensure defaults are present for new strategies"},{"line_number":501,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":502,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_276f9693","line":499,"range":{"start_line":499,"start_character":8,"end_line":499,"end_character":23},"in_reply_to":"fa6399be_43502c5f","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":566,"context_line":"        return _paginate_query(models.AuditTemplate, limit, marker,"},{"line_number":567,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":568,"context_line":""},{"line_number":569,"context_line":"    def create_audit_template(self, values):"},{"line_number":570,"context_line":"        # ensure defaults are present for new audit_templates"},{"line_number":571,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":572,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_23da58ea","line":569,"range":{"start_line":569,"start_character":8,"end_line":569,"end_character":29},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":566,"context_line":"        return _paginate_query(models.AuditTemplate, limit, marker,"},{"line_number":567,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":568,"context_line":""},{"line_number":569,"context_line":"    def create_audit_template(self, values):"},{"line_number":570,"context_line":"        # ensure defaults are present for new audit_templates"},{"line_number":571,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":572,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_e713de0b","line":569,"range":{"start_line":569,"start_character":8,"end_line":569,"end_character":29},"in_reply_to":"fa6399be_23da58ea","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        return _paginate_query(models.Audit, limit, marker,"},{"line_number":653,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"    def create_audit(self, values):"},{"line_number":656,"context_line":"        # ensure defaults are present for new audits"},{"line_number":657,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":658,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_c3f19c6c","line":655,"range":{"start_line":655,"start_character":8,"end_line":655,"end_character":20},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        return _paginate_query(models.Audit, limit, marker,"},{"line_number":653,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"    def create_audit(self, values):"},{"line_number":656,"context_line":"        # ensure defaults are present for new audits"},{"line_number":657,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":658,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_a73006a0","line":655,"range":{"start_line":655,"start_character":8,"end_line":655,"end_character":20},"in_reply_to":"fa6399be_c3f19c6c","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":738,"context_line":"        return _paginate_query(models.Action, limit, marker,"},{"line_number":739,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":740,"context_line":""},{"line_number":741,"context_line":"    def create_action(self, values):"},{"line_number":742,"context_line":"        # ensure defaults are present for new actions"},{"line_number":743,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":744,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_03ec748e","line":741,"range":{"start_line":741,"start_character":8,"end_line":741,"end_character":21},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":738,"context_line":"        return _paginate_query(models.Action, limit, marker,"},{"line_number":739,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":740,"context_line":""},{"line_number":741,"context_line":"    def create_action(self, values):"},{"line_number":742,"context_line":"        # ensure defaults are present for new actions"},{"line_number":743,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":744,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_e72afe2b","line":741,"range":{"start_line":741,"start_character":8,"end_line":741,"end_character":21},"in_reply_to":"fa6399be_03ec748e","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":766,"context_line":"        return self._get_action("},{"line_number":767,"context_line":"            context, fieldname\u003d\"uuid\", value\u003daction_uuid, eager\u003deager)"},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"    def destroy_action(self, action_id):"},{"line_number":770,"context_line":"        session \u003d get_session()"},{"line_number":771,"context_line":"        with session.begin():"},{"line_number":772,"context_line":"            query \u003d model_query(models.Action, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_43fe6c39","line":769,"range":{"start_line":769,"start_character":8,"end_line":769,"end_character":22},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":766,"context_line":"        return self._get_action("},{"line_number":767,"context_line":"            context, fieldname\u003d\"uuid\", value\u003daction_uuid, eager\u003deager)"},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"    def destroy_action(self, action_id):"},{"line_number":770,"context_line":"        session \u003d get_session()"},{"line_number":771,"context_line":"        with session.begin():"},{"line_number":772,"context_line":"            query \u003d model_query(models.Action, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_271dd60f","line":769,"range":{"start_line":769,"start_character":8,"end_line":769,"end_character":22},"in_reply_to":"fa6399be_43fe6c39","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":797,"context_line":"            ref.update(values)"},{"line_number":798,"context_line":"        return ref"},{"line_number":799,"context_line":""},{"line_number":800,"context_line":"    def soft_delete_action(self, action_id):"},{"line_number":801,"context_line":"        session \u003d get_session()"},{"line_number":802,"context_line":"        with session.begin():"},{"line_number":803,"context_line":"            query \u003d model_query(models.Action, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_a393c8ff","line":800,"range":{"start_line":800,"start_character":8,"end_line":800,"end_character":26},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":797,"context_line":"            ref.update(values)"},{"line_number":798,"context_line":"        return ref"},{"line_number":799,"context_line":""},{"line_number":800,"context_line":"    def soft_delete_action(self, action_id):"},{"line_number":801,"context_line":"        session \u003d get_session()"},{"line_number":802,"context_line":"        with session.begin():"},{"line_number":803,"context_line":"            query \u003d model_query(models.Action, session\u003dsession)"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_471aca18","line":800,"range":{"start_line":800,"start_character":8,"end_line":800,"end_character":26},"in_reply_to":"fa6399be_a393c8ff","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":1021,"context_line":"        return _paginate_query(models.ScoringEngine, limit, marker,"},{"line_number":1022,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":1023,"context_line":""},{"line_number":1024,"context_line":"    def create_scoring_engine(self, values):"},{"line_number":1025,"context_line":"        # ensure defaults are present for new scoring engines"},{"line_number":1026,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":1027,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_43858cab","line":1024,"range":{"start_line":1024,"start_character":8,"end_line":1024,"end_character":29},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":1021,"context_line":"        return _paginate_query(models.ScoringEngine, limit, marker,"},{"line_number":1022,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":1023,"context_line":""},{"line_number":1024,"context_line":"    def create_scoring_engine(self, values):"},{"line_number":1025,"context_line":"        # ensure defaults are present for new scoring engines"},{"line_number":1026,"context_line":"        if not values.get(\u0027uuid\u0027):"},{"line_number":1027,"context_line":"            values[\u0027uuid\u0027] \u003d utils.generate_uuid()"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_87d22233","line":1024,"range":{"start_line":1024,"start_character":8,"end_line":1024,"end_character":29},"in_reply_to":"fa6399be_43858cab","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"46642ba4be4c975dee6dfcacee1ab783f531af2b","unresolved":false,"context_lines":[{"line_number":1107,"context_line":"        return _paginate_query(models.Service, limit, marker,"},{"line_number":1108,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":1109,"context_line":""},{"line_number":1110,"context_line":"    def create_service(self, values):"},{"line_number":1111,"context_line":"        service \u003d models.Service()"},{"line_number":1112,"context_line":"        service.update(values)"},{"line_number":1113,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_e3b4e083","line":1110,"range":{"start_line":1110,"start_character":8,"end_line":1110,"end_character":22},"updated":"2016-10-17 15:27:29.000000000","message":"@staticmethod","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"65218bdad0f35bf9331c32aca76e3117afb8fd76","unresolved":false,"context_lines":[{"line_number":1107,"context_line":"        return _paginate_query(models.Service, limit, marker,"},{"line_number":1108,"context_line":"                               sort_key, sort_dir, query)"},{"line_number":1109,"context_line":""},{"line_number":1110,"context_line":"    def create_service(self, values):"},{"line_number":1111,"context_line":"        service \u003d models.Service()"},{"line_number":1112,"context_line":"        service.update(values)"},{"line_number":1113,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa6399be_c7dc1a49","line":1110,"range":{"start_line":1110,"start_character":8,"end_line":1110,"end_character":22},"in_reply_to":"fa6399be_e3b4e083","updated":"2016-10-18 13:54:13.000000000","message":"ditto","commit_id":"4ce361308b8772ec0efe5d610d416e56f4cd8fd9"}],"watcher/tests/db/test_action.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":251,"context_line":"        res \u003d self.dbapi.get_action_list(self.context)"},{"line_number":252,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":253,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    def test_get_action_list_eager(self):"},{"line_number":256,"context_line":"        _action_plan \u003d utils.get_test_action_plan()"},{"line_number":257,"context_line":"        action_plan \u003d self.dbapi.create_action_plan(_action_plan)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_97b2a5e3","line":254,"updated":"2016-10-10 09:26:51.000000000","message":"maybe you can add an assert test to verify that action-plan is not in the query result (eager\u003dFalse by default)","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":251,"context_line":"        res \u003d self.dbapi.get_action_list(self.context)"},{"line_number":252,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":253,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    def test_get_action_list_eager(self):"},{"line_number":256,"context_line":"        _action_plan \u003d utils.get_test_action_plan()"},{"line_number":257,"context_line":"        action_plan \u003d self.dbapi.create_action_plan(_action_plan)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_495b3bf6","line":254,"in_reply_to":"1a95cdbc_97b2a5e3","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":265,"context_line":"        actions \u003d self.dbapi.get_action_list(self.context, eager\u003dTrue)"},{"line_number":266,"context_line":"        action_map \u003d {a.uuid: a for a in actions}"},{"line_number":267,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_map.keys()))"},{"line_number":268,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_map.keys()))"},{"line_number":269,"context_line":"        eager_action \u003d action_map[action.uuid]"},{"line_number":270,"context_line":"        self.assertEqual("},{"line_number":271,"context_line":"            action_plan.as_dict(), eager_action.action_plan.as_dict())"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_f7192923","line":268,"range":{"start_line":268,"start_character":8,"end_line":268,"end_character":66},"updated":"2016-10-10 09:26:51.000000000","message":"doublon","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":265,"context_line":"        actions \u003d self.dbapi.get_action_list(self.context, eager\u003dTrue)"},{"line_number":266,"context_line":"        action_map \u003d {a.uuid: a for a in actions}"},{"line_number":267,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_map.keys()))"},{"line_number":268,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_map.keys()))"},{"line_number":269,"context_line":"        eager_action \u003d action_map[action.uuid]"},{"line_number":270,"context_line":"        self.assertEqual("},{"line_number":271,"context_line":"            action_plan.as_dict(), eager_action.action_plan.as_dict())"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_cd2777c8","line":268,"range":{"start_line":268,"start_character":8,"end_line":268,"end_character":66},"in_reply_to":"1a95cdbc_f7192923","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/tests/db/test_action_plan.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        res \u003d self.dbapi.get_action_plan_list(self.context)"},{"line_number":250,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":251,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    def test_get_action_plan_list_eager(self):"},{"line_number":254,"context_line":"        _strategy \u003d utils.get_test_strategy()"},{"line_number":255,"context_line":"        strategy \u003d self.dbapi.create_strategy(_strategy)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_b72961e8","line":252,"updated":"2016-10-10 09:26:51.000000000","message":"Maybe add 2 assert checks (audit and strategies are not in query result) to verify: eager\u003dFalse;","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        res \u003d self.dbapi.get_action_plan_list(self.context)"},{"line_number":250,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":251,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    def test_get_action_plan_list_eager(self):"},{"line_number":254,"context_line":"        _strategy \u003d utils.get_test_strategy()"},{"line_number":255,"context_line":"        strategy \u003d self.dbapi.create_strategy(_strategy)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_a96a177f","line":252,"in_reply_to":"1a95cdbc_b72961e8","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":265,"context_line":"            self.context, eager\u003dTrue)"},{"line_number":266,"context_line":"        action_plan_map \u003d {a.uuid: a for a in action_plans}"},{"line_number":267,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_plan_map.keys()))"},{"line_number":268,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_plan_map.keys()))"},{"line_number":269,"context_line":"        eager_action_plan \u003d action_plan_map[action_plan.uuid]"},{"line_number":270,"context_line":"        self.assertEqual("},{"line_number":271,"context_line":"            strategy.as_dict(), eager_action_plan.strategy.as_dict())"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_f74849ce","line":268,"range":{"start_line":268,"start_character":0,"end_line":268,"end_character":71},"updated":"2016-10-10 09:26:51.000000000","message":"remote this doublon","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":265,"context_line":"            self.context, eager\u003dTrue)"},{"line_number":266,"context_line":"        action_plan_map \u003d {a.uuid: a for a in action_plans}"},{"line_number":267,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_plan_map.keys()))"},{"line_number":268,"context_line":"        self.assertEqual(sorted(uuids), sorted(action_plan_map.keys()))"},{"line_number":269,"context_line":"        eager_action_plan \u003d action_plan_map[action_plan.uuid]"},{"line_number":270,"context_line":"        self.assertEqual("},{"line_number":271,"context_line":"            strategy.as_dict(), eager_action_plan.strategy.as_dict())"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_69fb9fc1","line":268,"range":{"start_line":268,"start_character":0,"end_line":268,"end_character":71},"in_reply_to":"1a95cdbc_f74849ce","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/tests/db/test_audit.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":257,"context_line":""},{"line_number":258,"context_line":"    def test_get_audit_list(self):"},{"line_number":259,"context_line":"        uuids \u003d []"},{"line_number":260,"context_line":"        for _ in range(1, 6):"},{"line_number":261,"context_line":"            audit \u003d utils.create_test_audit(uuid\u003dw_utils.generate_uuid())"},{"line_number":262,"context_line":"            uuids.append(six.text_type(audit[\u0027uuid\u0027]))"},{"line_number":263,"context_line":"        res \u003d self.dbapi.get_audit_list(self.context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_52058b0e","line":260,"range":{"start_line":260,"start_character":26,"end_line":260,"end_character":27},"updated":"2016-10-10 09:26:51.000000000","message":"4","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":257,"context_line":""},{"line_number":258,"context_line":"    def test_get_audit_list(self):"},{"line_number":259,"context_line":"        uuids \u003d []"},{"line_number":260,"context_line":"        for _ in range(1, 6):"},{"line_number":261,"context_line":"            audit \u003d utils.create_test_audit(uuid\u003dw_utils.generate_uuid())"},{"line_number":262,"context_line":"            uuids.append(six.text_type(audit[\u0027uuid\u0027]))"},{"line_number":263,"context_line":"        res \u003d self.dbapi.get_audit_list(self.context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_e9896f27","line":260,"range":{"start_line":260,"start_character":26,"end_line":260,"end_character":27},"in_reply_to":"1a95cdbc_52058b0e","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        res \u003d self.dbapi.get_audit_list(self.context)"},{"line_number":264,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":265,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def test_get_audit_list_eager(self):"},{"line_number":268,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":269,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_97b8e576","line":266,"updated":"2016-10-10 09:26:51.000000000","message":"why not add 2 assert checks (on goal and strategy) to verify eager\u003dFalse ?","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        res \u003d self.dbapi.get_audit_list(self.context)"},{"line_number":264,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":265,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def test_get_audit_list_eager(self):"},{"line_number":268,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":269,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_c99aeb6d","line":266,"in_reply_to":"1a95cdbc_97b8e576","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":271,"context_line":"        strategy \u003d self.dbapi.create_strategy(_strategy)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"        uuids \u003d []"},{"line_number":274,"context_line":"        for i in range(1, 6):"},{"line_number":275,"context_line":"            audit \u003d utils.create_test_audit("},{"line_number":276,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(),"},{"line_number":277,"context_line":"                goal_id\u003dgoal.id, strategy_id\u003dstrategy.id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_b216cf49","line":274,"range":{"start_line":274,"start_character":26,"end_line":274,"end_character":27},"updated":"2016-10-10 09:26:51.000000000","message":"4","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":271,"context_line":"        strategy \u003d self.dbapi.create_strategy(_strategy)"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"        uuids \u003d []"},{"line_number":274,"context_line":"        for i in range(1, 6):"},{"line_number":275,"context_line":"            audit \u003d utils.create_test_audit("},{"line_number":276,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(),"},{"line_number":277,"context_line":"                goal_id\u003dgoal.id, strategy_id\u003dstrategy.id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_69971f48","line":274,"range":{"start_line":274,"start_character":26,"end_line":274,"end_character":27},"in_reply_to":"1a95cdbc_b216cf49","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":279,"context_line":"        audits \u003d self.dbapi.get_audit_list(self.context, eager\u003dTrue)"},{"line_number":280,"context_line":"        audit_map \u003d {a.uuid: a for a in audits}"},{"line_number":281,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_map.keys()))"},{"line_number":282,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_map.keys()))"},{"line_number":283,"context_line":"        eager_audit \u003d audit_map[audit.uuid]"},{"line_number":284,"context_line":"        self.assertEqual(goal.as_dict(), eager_audit.goal.as_dict())"},{"line_number":285,"context_line":"        self.assertEqual(strategy.as_dict(), eager_audit.strategy.as_dict())"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_f7a92925","line":282,"range":{"start_line":282,"start_character":0,"end_line":282,"end_character":65},"updated":"2016-10-10 09:26:51.000000000","message":"doublon","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":279,"context_line":"        audits \u003d self.dbapi.get_audit_list(self.context, eager\u003dTrue)"},{"line_number":280,"context_line":"        audit_map \u003d {a.uuid: a for a in audits}"},{"line_number":281,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_map.keys()))"},{"line_number":282,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_map.keys()))"},{"line_number":283,"context_line":"        eager_audit \u003d audit_map[audit.uuid]"},{"line_number":284,"context_line":"        self.assertEqual(goal.as_dict(), eager_audit.goal.as_dict())"},{"line_number":285,"context_line":"        self.assertEqual(strategy.as_dict(), eager_audit.strategy.as_dict())"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_098f832a","line":282,"range":{"start_line":282,"start_character":0,"end_line":282,"end_character":65},"in_reply_to":"1a95cdbc_f7a92925","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        res \u003d self.dbapi.get_audit_list("},{"line_number":302,"context_line":"            self.context,"},{"line_number":303,"context_line":"            filters\u003d{\u0027audit_type\u0027: objects.audit.AuditType.ONESHOT.value})"},{"line_number":304,"context_line":"        self.assertEqual([audit1[\u0027id\u0027]], [r.id for r in res])"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        res \u003d self.dbapi.get_audit_list(self.context,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_374d519a","line":303,"range":{"start_line":303,"start_character":59,"end_line":303,"end_character":72},"updated":"2016-10-10 09:26:51.000000000","message":"why not ONESHOT ?","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":300,"context_line":""},{"line_number":301,"context_line":"        res \u003d self.dbapi.get_audit_list("},{"line_number":302,"context_line":"            self.context,"},{"line_number":303,"context_line":"            filters\u003d{\u0027audit_type\u0027: objects.audit.AuditType.ONESHOT.value})"},{"line_number":304,"context_line":"        self.assertEqual([audit1[\u0027id\u0027]], [r.id for r in res])"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        res \u003d self.dbapi.get_audit_list(self.context,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_e9b7afd5","line":303,"range":{"start_line":303,"start_character":59,"end_line":303,"end_character":72},"in_reply_to":"1a95cdbc_374d519a","updated":"2016-10-18 15:27:53.000000000","message":"AuditType.ONESHOT.value returns \u0027ONESHOT\u0027 whereas AuditType.ONESHOT returns the enum object.","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/tests/db/test_audit_template.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":232,"context_line":""},{"line_number":233,"context_line":"    def test_get_audit_template_list(self):"},{"line_number":234,"context_line":"        uuids \u003d []"},{"line_number":235,"context_line":"        for i in range(1, 6):"},{"line_number":236,"context_line":"            audit_template \u003d utils.create_test_audit_template("},{"line_number":237,"context_line":"                id\u003di,"},{"line_number":238,"context_line":"                uuid\u003dw_utils.generate_uuid(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_7711995c","line":235,"range":{"start_line":235,"start_character":26,"end_line":235,"end_character":27},"updated":"2016-10-10 09:26:51.000000000","message":"4","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":232,"context_line":""},{"line_number":233,"context_line":"    def test_get_audit_template_list(self):"},{"line_number":234,"context_line":"        uuids \u003d []"},{"line_number":235,"context_line":"        for i in range(1, 6):"},{"line_number":236,"context_line":"            audit_template \u003d utils.create_test_audit_template("},{"line_number":237,"context_line":"                id\u003di,"},{"line_number":238,"context_line":"                uuid\u003dw_utils.generate_uuid(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_a9d93724","line":235,"range":{"start_line":235,"start_character":26,"end_line":235,"end_character":27},"in_reply_to":"1a95cdbc_7711995c","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":241,"context_line":"        res \u003d self.dbapi.get_audit_template_list(self.context)"},{"line_number":242,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":243,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"    def test_get_audit_template_list_eager(self):"},{"line_number":246,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":247,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_97462579","line":244,"updated":"2016-10-10 09:26:51.000000000","message":"why not add 2 assert checks (on goal and strategy) to verify eager\u003dFalse ?","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":241,"context_line":"        res \u003d self.dbapi.get_audit_template_list(self.context)"},{"line_number":242,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":243,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"    def test_get_audit_template_list_eager(self):"},{"line_number":246,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":247,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_c9d66bf0","line":244,"in_reply_to":"1a95cdbc_97462579","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        strategy \u003d self.dbapi.create_strategy(_strategy)"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"        uuids \u003d []"},{"line_number":252,"context_line":"        for i in range(1, 6):"},{"line_number":253,"context_line":"            audit_template \u003d utils.create_test_audit_template("},{"line_number":254,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(),"},{"line_number":255,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(i),"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_57373dfe","line":252,"range":{"start_line":252,"start_character":26,"end_line":252,"end_character":27},"updated":"2016-10-10 09:26:51.000000000","message":"4","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        strategy \u003d self.dbapi.create_strategy(_strategy)"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":"        uuids \u003d []"},{"line_number":252,"context_line":"        for i in range(1, 6):"},{"line_number":253,"context_line":"            audit_template \u003d utils.create_test_audit_template("},{"line_number":254,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(),"},{"line_number":255,"context_line":"                name\u003d\u0027My Audit Template {0}\u0027.format(i),"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_69d39f04","line":252,"range":{"start_line":252,"start_character":26,"end_line":252,"end_character":27},"in_reply_to":"1a95cdbc_57373dfe","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":259,"context_line":"            self.context, eager\u003dTrue)"},{"line_number":260,"context_line":"        audit_template_map \u003d {a.uuid: a for a in audit_templates}"},{"line_number":261,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_template_map.keys()))"},{"line_number":262,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_template_map.keys()))"},{"line_number":263,"context_line":"        eager_audit_template \u003d audit_template_map[audit_template.uuid]"},{"line_number":264,"context_line":"        self.assertEqual(goal.as_dict(), eager_audit_template.goal.as_dict())"},{"line_number":265,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_373471fa","line":262,"range":{"start_line":262,"start_character":0,"end_line":262,"end_character":74},"updated":"2016-10-10 09:26:51.000000000","message":"doublon","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":259,"context_line":"            self.context, eager\u003dTrue)"},{"line_number":260,"context_line":"        audit_template_map \u003d {a.uuid: a for a in audit_templates}"},{"line_number":261,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_template_map.keys()))"},{"line_number":262,"context_line":"        self.assertEqual(sorted(uuids), sorted(audit_template_map.keys()))"},{"line_number":263,"context_line":"        eager_audit_template \u003d audit_template_map[audit_template.uuid]"},{"line_number":264,"context_line":"        self.assertEqual(goal.as_dict(), eager_audit_template.goal.as_dict())"},{"line_number":265,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_49017b69","line":262,"range":{"start_line":262,"start_character":0,"end_line":262,"end_character":74},"in_reply_to":"1a95cdbc_373471fa","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/tests/db/test_efficacy_indicator.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        res \u003d self.dbapi.get_efficacy_indicator_list(self.context)"},{"line_number":264,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":265,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def test_get_efficacy_indicator_list_eager(self):"},{"line_number":268,"context_line":"        _action_plan \u003d utils.get_test_action_plan()"},{"line_number":269,"context_line":"        action_plan \u003d self.dbapi.create_action_plan(_action_plan)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_5253cb16","line":266,"updated":"2016-10-10 09:26:51.000000000","message":"Why not add an assert check on action-plan not present in query result for eager\u003dFalse ?","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        res \u003d self.dbapi.get_efficacy_indicator_list(self.context)"},{"line_number":264,"context_line":"        res_uuids \u003d [r.uuid for r in res]"},{"line_number":265,"context_line":"        self.assertEqual(uuids.sort(), res_uuids.sort())"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    def test_get_efficacy_indicator_list_eager(self):"},{"line_number":268,"context_line":"        _action_plan \u003d utils.get_test_action_plan()"},{"line_number":269,"context_line":"        action_plan \u003d self.dbapi.create_action_plan(_action_plan)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_c944abb8","line":266,"in_reply_to":"1a95cdbc_5253cb16","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        action_plan \u003d self.dbapi.create_action_plan(_action_plan)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        uuids \u003d []"},{"line_number":272,"context_line":"        for i in range(1, 6):"},{"line_number":273,"context_line":"            efficacy_indicator \u003d utils.create_test_efficacy_indicator("},{"line_number":274,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(),"},{"line_number":275,"context_line":"                action_plan_id\u003daction_plan.id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_522cab98","line":272,"range":{"start_line":272,"start_character":26,"end_line":272,"end_character":27},"updated":"2016-10-10 09:26:51.000000000","message":"4","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":269,"context_line":"        action_plan \u003d self.dbapi.create_action_plan(_action_plan)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        uuids \u003d []"},{"line_number":272,"context_line":"        for i in range(1, 6):"},{"line_number":273,"context_line":"            efficacy_indicator \u003d utils.create_test_efficacy_indicator("},{"line_number":274,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(),"},{"line_number":275,"context_line":"                action_plan_id\u003daction_plan.id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_a94777ba","line":272,"range":{"start_line":272,"start_character":26,"end_line":272,"end_character":27},"in_reply_to":"1a95cdbc_522cab98","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":278,"context_line":"            self.context, eager\u003dTrue)"},{"line_number":279,"context_line":"        efficacy_indicator_map \u003d {a.uuid: a for a in efficacy_indicators}"},{"line_number":280,"context_line":"        self.assertEqual(sorted(uuids), sorted(efficacy_indicator_map.keys()))"},{"line_number":281,"context_line":"        self.assertEqual(sorted(uuids), sorted(efficacy_indicator_map.keys()))"},{"line_number":282,"context_line":"        eager_efficacy_indicator \u003d efficacy_indicator_map["},{"line_number":283,"context_line":"            efficacy_indicator.uuid]"},{"line_number":284,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_72560707","line":281,"range":{"start_line":281,"start_character":0,"end_line":281,"end_character":78},"updated":"2016-10-10 09:26:51.000000000","message":"doublon","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":278,"context_line":"            self.context, eager\u003dTrue)"},{"line_number":279,"context_line":"        efficacy_indicator_map \u003d {a.uuid: a for a in efficacy_indicators}"},{"line_number":280,"context_line":"        self.assertEqual(sorted(uuids), sorted(efficacy_indicator_map.keys()))"},{"line_number":281,"context_line":"        self.assertEqual(sorted(uuids), sorted(efficacy_indicator_map.keys()))"},{"line_number":282,"context_line":"        eager_efficacy_indicator \u003d efficacy_indicator_map["},{"line_number":283,"context_line":"            efficacy_indicator.uuid]"},{"line_number":284,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_290a8779","line":281,"range":{"start_line":281,"start_character":0,"end_line":281,"end_character":78},"in_reply_to":"1a95cdbc_72560707","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}],"watcher/tests/db/test_strategy.py":[{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        res \u003d self.dbapi.get_strategy_list(self.context)"},{"line_number":257,"context_line":"        res_ids \u003d [r.display_name for r in res]"},{"line_number":258,"context_line":"        self.assertEqual(ids.sort(), res_ids.sort())"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"    def test_get_strategy_list_eager(self):"},{"line_number":261,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":262,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_527aeb88","line":259,"updated":"2016-10-10 09:26:51.000000000","message":"Why not add an assert check about goal not present in query result for eager\u003dFalse ?","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":256,"context_line":"        res \u003d self.dbapi.get_strategy_list(self.context)"},{"line_number":257,"context_line":"        res_ids \u003d [r.display_name for r in res]"},{"line_number":258,"context_line":"        self.assertEqual(ids.sort(), res_ids.sort())"},{"line_number":259,"context_line":""},{"line_number":260,"context_line":"    def test_get_strategy_list_eager(self):"},{"line_number":261,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":262,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_6928ffe2","line":259,"in_reply_to":"1a95cdbc_527aeb88","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":261,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":262,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"},{"line_number":263,"context_line":"        uuids \u003d []"},{"line_number":264,"context_line":"        for i in range(1, 6):"},{"line_number":265,"context_line":"            strategy \u003d utils.create_test_strategy("},{"line_number":266,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(), goal_id\u003dgoal.id)"},{"line_number":267,"context_line":"            uuids.append(six.text_type(strategy[\u0027uuid\u0027]))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_f26137d4","line":264,"range":{"start_line":264,"start_character":26,"end_line":264,"end_character":27},"updated":"2016-10-10 09:26:51.000000000","message":"4","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":261,"context_line":"        _goal \u003d utils.get_test_goal()"},{"line_number":262,"context_line":"        goal \u003d self.dbapi.create_goal(_goal)"},{"line_number":263,"context_line":"        uuids \u003d []"},{"line_number":264,"context_line":"        for i in range(1, 6):"},{"line_number":265,"context_line":"            strategy \u003d utils.create_test_strategy("},{"line_number":266,"context_line":"                id\u003di, uuid\u003dw_utils.generate_uuid(), goal_id\u003dgoal.id)"},{"line_number":267,"context_line":"            uuids.append(six.text_type(strategy[\u0027uuid\u0027]))"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_8925530a","line":264,"range":{"start_line":264,"start_character":26,"end_line":264,"end_character":27},"in_reply_to":"1a95cdbc_f26137d4","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":12394,"name":"David TARDIVEL","email":"David.TARDIVEL@b-com.com","username":"David.T"},"change_message_id":"628505bc87029b47ff39a445a5ccfd6336626a06","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        strategys \u003d self.dbapi.get_strategy_list(self.context, eager\u003dTrue)"},{"line_number":269,"context_line":"        strategy_map \u003d {a.uuid: a for a in strategys}"},{"line_number":270,"context_line":"        self.assertEqual(sorted(uuids), sorted(strategy_map.keys()))"},{"line_number":271,"context_line":"        self.assertEqual(sorted(uuids), sorted(strategy_map.keys()))"},{"line_number":272,"context_line":"        eager_strategy \u003d strategy_map[strategy.uuid]"},{"line_number":273,"context_line":"        self.assertEqual(goal.as_dict(), eager_strategy.goal.as_dict())"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a95cdbc_b26b2fb6","line":271,"range":{"start_line":271,"start_character":0,"end_line":271,"end_character":68},"updated":"2016-10-10 09:26:51.000000000","message":"doublon","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"},{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"a0718ec4d229ef17196ed259f62a6bbe5f3bc8ef","unresolved":false,"context_lines":[{"line_number":268,"context_line":"        strategys \u003d self.dbapi.get_strategy_list(self.context, eager\u003dTrue)"},{"line_number":269,"context_line":"        strategy_map \u003d {a.uuid: a for a in strategys}"},{"line_number":270,"context_line":"        self.assertEqual(sorted(uuids), sorted(strategy_map.keys()))"},{"line_number":271,"context_line":"        self.assertEqual(sorted(uuids), sorted(strategy_map.keys()))"},{"line_number":272,"context_line":"        eager_strategy \u003d strategy_map[strategy.uuid]"},{"line_number":273,"context_line":"        self.assertEqual(goal.as_dict(), eager_strategy.goal.as_dict())"},{"line_number":274,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_a91e97b8","line":271,"range":{"start_line":271,"start_character":0,"end_line":271,"end_character":68},"in_reply_to":"1a95cdbc_b26b2fb6","updated":"2016-10-18 15:27:53.000000000","message":"Done","commit_id":"a4c01048f7c4e90e7f8fe7ae10dd1c6722676942"}]}