)]}'
{"rally/cli/commands/task.py":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"0ca6ad5a948bb84954d4f89ea5baf03c6f6d4a29","unresolved":false,"context_lines":[{"line_number":218,"context_line":"                           abort_on_sla_failure\u003dabort_on_sla_failure)"},{"line_number":219,"context_line":"            self.detailed(task_id\u003dtask[\"uuid\"])"},{"line_number":220,"context_line":"        except KeyboardInterrupt:"},{"line_number":221,"context_line":"            print(\"Keyboard interrupt received, aborting task...\\n\")"},{"line_number":222,"context_line":"            api.Task.abort(task[\"uuid\"])"},{"line_number":223,"context_line":"        except exceptions.InvalidConfigException:"},{"line_number":224,"context_line":"            return(1)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba15a1d1_a4a22099","line":221,"updated":"2015-09-20 21:43:32.000000000","message":"it will be better to show task uuid in this message.","commit_id":"bf8bbcc8cd471fc63b4f23844a8a25d264caa21e"}],"tests/unit/test_api.py":[{"author":{"_account_id":14764,"name":"Yuki Nishiwaki","email":"uckey.1067@gmail.com","username":"ukinau"},"change_message_id":"535f011f87dd996eb6a3fe7a065dd55336fce798","unresolved":false,"context_lines":[{"line_number":160,"context_line":"    @mock.patch(\"rally.api.engine.BenchmarkEngine\")"},{"line_number":161,"context_line":"    def test_start_abort(self, mock_benchmark_engine):"},{"line_number":162,"context_line":"        mock_benchmark_engine().run.side_effect \u003d KeyboardInterrupt"},{"line_number":163,"context_line":"        self.assertRaises(NotImplementedError, api.Task.abort, self.task_uuid)"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"    def test_abort(self):"},{"line_number":166,"context_line":"        self.assertRaises(NotImplementedError, api.Task.abort, self.task_uuid)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba15a1d1_9f383b3f","line":163,"updated":"2015-09-22 13:18:15.000000000","message":"I think this test is not correct for this time change.\n\nThis test mean that \"if rally.api.engine.BenchmarkEngine.run method raise KeyboardInterrupt in api.Task.abort method, api.Task.abort method should raise NotImplementedError\"\n\u003c- This test has no relation to the change of this patch.\n\nSo you must write test for confirming \"If api.Task.start method raise KeyboardInterrupt exception, api.Task.abort method should be called\".","commit_id":"f7b666765f02768e855884164c86a63c7c725d74"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"52c1a8303f87a2db3d1efa8a9e6013f8e53af6b3","unresolved":false,"context_lines":[{"line_number":160,"context_line":"    @mock.patch(\"rally.api.engine.BenchmarkEngine\")"},{"line_number":161,"context_line":"    def test_start_abort(self, mock_benchmark_engine):"},{"line_number":162,"context_line":"        mock_benchmark_engine().run.side_effect \u003d KeyboardInterrupt"},{"line_number":163,"context_line":"        self.assertRaises(NotImplementedError, api.Task.abort, self.task_uuid)"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"    def test_abort(self):"},{"line_number":166,"context_line":"        self.assertRaises(NotImplementedError, api.Task.abort, self.task_uuid)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba15a1d1_a2bf145f","line":163,"in_reply_to":"ba15a1d1_9f383b3f","updated":"2015-09-22 19:03:12.000000000","message":"Thanks for clarifying this.","commit_id":"f7b666765f02768e855884164c86a63c7c725d74"}]}