)]}'
{"watcher/tests/decision_engine/audit/test_audit_handlers.py":[{"author":{"_account_id":27947,"name":"Egor Panfilov","email":"edpanfilov@sbcloud.ru","username":"erakli"},"change_message_id":"788b00dd936b11194d23db4c944e3f52530fe8d3","unresolved":false,"context_lines":[{"line_number":424,"context_line":"        audit_handler.launch_audits_periodically()"},{"line_number":425,"context_line":"        m_remove_job.assert_called()"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"    @mock.patch.object(weight.WeightPlanner, \"schedule\")"},{"line_number":428,"context_line":"    def test_execute_audit(self, m_schedule):"},{"line_number":429,"context_line":"        audit_handler \u003d continuous.ContinuousAuditHandler()"},{"line_number":430,"context_line":"        audit \u003d self.audits[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_73cbe452","line":427,"range":{"start_line":427,"start_character":23,"end_line":427,"end_character":55},"updated":"2018-04-16 21:39:03.000000000","message":"Why not to mock AuditHandler\u0027s \u0027planner\u0027 property?\n\nFor example, do something like this:\n@mock.patch.object(continuous.ContinuousAuditHandler, \u0027planner\u0027)\n\nIt would be more convenient, IMO.","commit_id":"3b5c00433b860568abec30c09550662f944b6ffb"},{"author":{"_account_id":21692,"name":"licanwei","email":"li.canwei2@zte.com.cn","username":"licanwei"},"change_message_id":"03a351870e0a3770490c957d8ac2b4ee0930f501","unresolved":false,"context_lines":[{"line_number":424,"context_line":"        audit_handler.launch_audits_periodically()"},{"line_number":425,"context_line":"        m_remove_job.assert_called()"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"    @mock.patch.object(weight.WeightPlanner, \"schedule\")"},{"line_number":428,"context_line":"    def test_execute_audit(self, m_schedule):"},{"line_number":429,"context_line":"        audit_handler \u003d continuous.ContinuousAuditHandler()"},{"line_number":430,"context_line":"        audit \u003d self.audits[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_811e550a","line":427,"range":{"start_line":427,"start_character":23,"end_line":427,"end_character":55},"in_reply_to":"9f6a8fd7_73cbe452","updated":"2018-04-17 11:21:21.000000000","message":"Done","commit_id":"3b5c00433b860568abec30c09550662f944b6ffb"},{"author":{"_account_id":27947,"name":"Egor Panfilov","email":"edpanfilov@sbcloud.ru","username":"erakli"},"change_message_id":"788b00dd936b11194d23db4c944e3f52530fe8d3","unresolved":false,"context_lines":[{"line_number":425,"context_line":"        m_remove_job.assert_called()"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"    @mock.patch.object(weight.WeightPlanner, \"schedule\")"},{"line_number":428,"context_line":"    def test_execute_audit(self, m_schedule):"},{"line_number":429,"context_line":"        audit_handler \u003d continuous.ContinuousAuditHandler()"},{"line_number":430,"context_line":"        audit \u003d self.audits[0]"},{"line_number":431,"context_line":"        audit_handler.execute_audit(audit, self.context)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_f34cd477","line":428,"range":{"start_line":428,"start_character":33,"end_line":428,"end_character":43},"updated":"2018-04-16 21:39:03.000000000","message":"Argument not used","commit_id":"3b5c00433b860568abec30c09550662f944b6ffb"},{"author":{"_account_id":27947,"name":"Egor Panfilov","email":"edpanfilov@sbcloud.ru","username":"erakli"},"change_message_id":"aa62efe7c8e8a9073f90b33de4c6e10c8414e54c","unresolved":false,"context_lines":[{"line_number":425,"context_line":"        m_remove_job.assert_called()"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"    @mock.patch.object(weight.WeightPlanner, \"schedule\")"},{"line_number":428,"context_line":"    def test_execute_audit(self, m_schedule):"},{"line_number":429,"context_line":"        audit_handler \u003d continuous.ContinuousAuditHandler()"},{"line_number":430,"context_line":"        audit \u003d self.audits[0]"},{"line_number":431,"context_line":"        audit_handler.execute_audit(audit, self.context)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_3d8f5c2b","line":428,"range":{"start_line":428,"start_character":33,"end_line":428,"end_character":43},"in_reply_to":"9f6a8fd7_e12ab1e5","updated":"2018-04-17 19:44:01.000000000","message":"Got it. I misread that patch decorator also passes created mock as an argument.","commit_id":"3b5c00433b860568abec30c09550662f944b6ffb"},{"author":{"_account_id":21692,"name":"licanwei","email":"li.canwei2@zte.com.cn","username":"licanwei"},"change_message_id":"03a351870e0a3770490c957d8ac2b4ee0930f501","unresolved":false,"context_lines":[{"line_number":425,"context_line":"        m_remove_job.assert_called()"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"    @mock.patch.object(weight.WeightPlanner, \"schedule\")"},{"line_number":428,"context_line":"    def test_execute_audit(self, m_schedule):"},{"line_number":429,"context_line":"        audit_handler \u003d continuous.ContinuousAuditHandler()"},{"line_number":430,"context_line":"        audit \u003d self.audits[0]"},{"line_number":431,"context_line":"        audit_handler.execute_audit(audit, self.context)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f6a8fd7_e12ab1e5","line":428,"range":{"start_line":428,"start_character":33,"end_line":428,"end_character":43},"in_reply_to":"9f6a8fd7_f34cd477","updated":"2018-04-17 11:21:21.000000000","message":"It\u0027s the mock of schedule. It is invoked when running execute_audit.","commit_id":"3b5c00433b860568abec30c09550662f944b6ffb"}]}