)]}'
{"tempest/tests/cmd/test_run.py":[{"author":{"_account_id":25695,"name":"Deepak Mourya","email":"deepakmoriya7@gmail.com","username":"deepak_mourya"},"change_message_id":"1893cf1165474c27ff19fba55e87a898ea085500","unresolved":false,"context_lines":[{"line_number":157,"context_line":"class TestTakeAction(base.TestCase):"},{"line_number":158,"context_line":"    def setUp(self):"},{"line_number":159,"context_line":"        super(TestTakeAction, self).setUp()"},{"line_number":160,"context_line":"        self.name \u003d data_utils.rand_uuid()"},{"line_number":161,"context_line":"        self.path \u003d tempfile.mkdtemp()"},{"line_number":162,"context_line":"        self.addCleanup(shutil.rmtree, self.path, ignore_errors\u003dTrue)"},{"line_number":163,"context_line":"        store_dir \u003d tempfile.mkdtemp()"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f79a3b5_b3b72b87","line":160,"range":{"start_line":160,"start_character":36,"end_line":160,"end_character":42},"updated":"2018-08-07 08:47:56.000000000","message":"I think rand_uuid will return random id so it should not be captured in the self.name.","commit_id":"cb5b9ea38b983fab7623e3f0e7656891c9dce281"},{"author":{"_account_id":25695,"name":"Deepak Mourya","email":"deepakmoriya7@gmail.com","username":"deepak_mourya"},"change_message_id":"4d1148fedff440ed16b31b90452045ff382e9572","unresolved":false,"context_lines":[{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    def test_no_config_file_no_workspace(self):"},{"line_number":222,"context_line":"        # Setup test dirs"},{"line_number":223,"context_line":"        self.directory \u003d tempfile.mkdtemp(prefix\u003d\u0027tempest-unit\u0027)"},{"line_number":224,"context_line":"        self.addCleanup(shutil.rmtree, self.directory)"},{"line_number":225,"context_line":"        self.test_dir \u003d os.path.join(self.directory, \u0027tests\u0027)"},{"line_number":226,"context_line":"        os.mkdir(self.test_dir)"},{"line_number":227,"context_line":"        # Change directory, run wrapper and check result"},{"line_number":228,"context_line":"        self.addCleanup(os.chdir, os.path.abspath(os.curdir))"},{"line_number":229,"context_line":"        os.chdir(self.directory)"},{"line_number":230,"context_line":"        tempest_run \u003d run.TempestRun(app\u003dmock.Mock(), app_args\u003dmock.Mock())"},{"line_number":231,"context_line":"        parsed_args \u003d mock.Mock()"},{"line_number":232,"context_line":"        parsed_args.config_file \u003d []"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_afec69f3","line":229,"range":{"start_line":223,"start_character":6,"end_line":229,"end_character":32},"updated":"2018-08-09 09:03:35.000000000","message":"I think these lines of codes are repeating in below tests also. Can we make a common helper function for this?","commit_id":"e689c17ad95e1c8e95005dad26417b774ea84d1d"},{"author":{"_account_id":25695,"name":"Deepak Mourya","email":"deepakmoriya7@gmail.com","username":"deepak_mourya"},"change_message_id":"4d1148fedff440ed16b31b90452045ff382e9572","unresolved":false,"context_lines":[{"line_number":241,"context_line":""},{"line_number":242,"context_line":"    def test_config_file_workspace_registered(self):"},{"line_number":243,"context_line":"        # Setup test dirs"},{"line_number":244,"context_line":"        self.directory \u003d tempfile.mkdtemp(prefix\u003d\u0027tempest-unit\u0027)"},{"line_number":245,"context_line":"        self.addCleanup(shutil.rmtree, self.directory)"},{"line_number":246,"context_line":"        self.test_dir \u003d os.path.join(self.directory, \u0027tests\u0027)"},{"line_number":247,"context_line":"        os.mkdir(self.test_dir)"},{"line_number":248,"context_line":"        # Change directory, run wrapper and check result"},{"line_number":249,"context_line":"        self.addCleanup(os.chdir, os.path.abspath(os.curdir))"},{"line_number":250,"context_line":"        os.chdir(self.directory)"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        tempest_run \u003d run.TempestRun(app\u003dmock.Mock(), app_args\u003dmock.Mock())"},{"line_number":253,"context_line":"        parsed_args \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_6fc09151","line":250,"range":{"start_line":244,"start_character":7,"end_line":250,"end_character":32},"updated":"2018-08-09 09:03:35.000000000","message":"ditto","commit_id":"e689c17ad95e1c8e95005dad26417b774ea84d1d"},{"author":{"_account_id":25695,"name":"Deepak Mourya","email":"deepakmoriya7@gmail.com","username":"deepak_mourya"},"change_message_id":"4d1148fedff440ed16b31b90452045ff382e9572","unresolved":false,"context_lines":[{"line_number":266,"context_line":"    def test_config_file_workspace_registered_state_enable("},{"line_number":267,"context_line":"            self, mock_init_state):"},{"line_number":268,"context_line":"        # Setup test dirs"},{"line_number":269,"context_line":"        self.directory \u003d tempfile.mkdtemp(prefix\u003d\u0027tempest-unit\u0027)"},{"line_number":270,"context_line":"        self.addCleanup(shutil.rmtree, self.directory)"},{"line_number":271,"context_line":"        self.test_dir \u003d os.path.join(self.directory, \u0027tests\u0027)"},{"line_number":272,"context_line":"        os.mkdir(self.test_dir)"},{"line_number":273,"context_line":"        # Change directory, run wrapper and check result"},{"line_number":274,"context_line":"        self.addCleanup(os.chdir, os.path.abspath(os.curdir))"},{"line_number":275,"context_line":"        os.chdir(self.directory)"},{"line_number":276,"context_line":""},{"line_number":277,"context_line":"        tempest_run \u003d run.TempestRun(app\u003dmock.Mock(), app_args\u003dmock.Mock())"},{"line_number":278,"context_line":"        parsed_args \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_cfd0c522","line":275,"range":{"start_line":269,"start_character":8,"end_line":275,"end_character":32},"updated":"2018-08-09 09:03:35.000000000","message":"ditto","commit_id":"e689c17ad95e1c8e95005dad26417b774ea84d1d"},{"author":{"_account_id":25695,"name":"Deepak Mourya","email":"deepakmoriya7@gmail.com","username":"deepak_mourya"},"change_message_id":"4d1148fedff440ed16b31b90452045ff382e9572","unresolved":false,"context_lines":[{"line_number":292,"context_line":"    @mock.patch(\u0027tempest.cmd.run.TempestRun._init_state\u0027)"},{"line_number":293,"context_line":"    def test_no_config_file_no_workspace_state_enable(self, mock_init_state):"},{"line_number":294,"context_line":"        # Setup test dirs"},{"line_number":295,"context_line":"        self.directory \u003d tempfile.mkdtemp(prefix\u003d\u0027tempest-unit\u0027)"},{"line_number":296,"context_line":"        self.addCleanup(shutil.rmtree, self.directory)"},{"line_number":297,"context_line":"        self.test_dir \u003d os.path.join(self.directory, \u0027tests\u0027)"},{"line_number":298,"context_line":"        os.mkdir(self.test_dir)"},{"line_number":299,"context_line":"        # Change directory, run wrapper and check result"},{"line_number":300,"context_line":"        self.addCleanup(os.chdir, os.path.abspath(os.curdir))"},{"line_number":301,"context_line":"        os.chdir(self.directory)"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        tempest_run \u003d run.TempestRun(app\u003dmock.Mock(), app_args\u003dmock.Mock())"},{"line_number":304,"context_line":"        parsed_args \u003d mock.Mock()"}],"source_content_type":"text/x-python","patch_set":5,"id":"3f79a3b5_2fba99dd","line":301,"range":{"start_line":295,"start_character":7,"end_line":301,"end_character":32},"updated":"2018-08-09 09:03:35.000000000","message":"ditto","commit_id":"e689c17ad95e1c8e95005dad26417b774ea84d1d"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"a870a6f2cdacf3ede68d851bdfa07900d9e3a269","unresolved":false,"context_lines":[{"line_number":169,"context_line":""},{"line_number":170,"context_line":"    def _setup_test_dirs(self):"},{"line_number":171,"context_line":"        self.directory \u003d tempfile.mkdtemp(prefix\u003d\u0027tempest-unit\u0027)"},{"line_number":172,"context_line":"        self.addCleanup(shutil.rmtree, self.directory)"},{"line_number":173,"context_line":"        self.test_dir \u003d os.path.join(self.directory, \u0027tests\u0027)"},{"line_number":174,"context_line":"        os.mkdir(self.test_dir)"},{"line_number":175,"context_line":"        # Change directory, run wrapper and check result"}],"source_content_type":"text/x-python","patch_set":6,"id":"3f79a3b5_ac543e55","line":172,"updated":"2018-09-18 03:16:31.000000000","message":"Why not add ignore_errors\u003dTrue to L172?","commit_id":"0743386ba5c7001f805c1d3b2f50545aa8db6bd8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"a870a6f2cdacf3ede68d851bdfa07900d9e3a269","unresolved":false,"context_lines":[{"line_number":223,"context_line":"        self._setup_test_dirs()"},{"line_number":224,"context_line":"        tempest_run \u003d run.TempestRun(app\u003dmock.Mock(), app_args\u003dmock.Mock())"},{"line_number":225,"context_line":"        parsed_args \u003d mock.Mock()"},{"line_number":226,"context_line":"        parsed_args.config_file \u003d []"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        parsed_args.workspace \u003d None"},{"line_number":229,"context_line":"        parsed_args.state \u003d None"}],"source_content_type":"text/x-python","patch_set":6,"id":"3f79a3b5_ac6bde1c","line":226,"range":{"start_line":226,"start_character":8,"end_line":226,"end_character":36},"updated":"2018-09-18 03:16:31.000000000","message":"Redundant with L231","commit_id":"0743386ba5c7001f805c1d3b2f50545aa8db6bd8"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"f60fd7d99721650085850170f0770cbf71981807","unresolved":false,"context_lines":[{"line_number":289,"context_line":"            m.assert_called()"},{"line_number":290,"context_line":"        mock_init_state.assert_not_called()"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    @mock.patch(\u0027tempest.cmd.run.TempestRun._init_state\u0027)"},{"line_number":293,"context_line":"    def test_config_file_workspace_registered_state_true("},{"line_number":294,"context_line":"            self, mock_init_state):"},{"line_number":295,"context_line":"        self._setup_test_dirs()"},{"line_number":296,"context_line":"        tempest_run \u003d run.TempestRun(app\u003dmock.Mock(), app_args\u003dmock.Mock())"},{"line_number":297,"context_line":"        parsed_args \u003d mock.Mock()"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        parsed_args.workspace \u003d self.name"},{"line_number":300,"context_line":"        parsed_args.workspace_path \u003d self.store_file"},{"line_number":301,"context_line":"        parsed_args.state \u003d True"},{"line_number":302,"context_line":"        parsed_args.list_tests \u003d False"},{"line_number":303,"context_line":"        parsed_args.config_file \u003d \u0027.stestr.conf\u0027"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"        with mock.patch(\u0027stestr.commands.run_command\u0027) as m:"},{"line_number":306,"context_line":"            m.return_value \u003d 0"},{"line_number":307,"context_line":"            self.assertEqual(0, tempest_run.take_action(parsed_args))"},{"line_number":308,"context_line":"            m.assert_called()"},{"line_number":309,"context_line":"        mock_init_state.assert_called()"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    @mock.patch(\u0027tempest.cmd.run.TempestRun._init_state\u0027)"},{"line_number":312,"context_line":"    def test_no_config_file_no_workspace_state_true(self, mock_init_state):"}],"source_content_type":"text/x-python","patch_set":9,"id":"3f79a3b5_6ef0fa66","line":309,"range":{"start_line":292,"start_character":0,"end_line":309,"end_character":39},"updated":"2018-10-17 03:46:37.000000000","message":"nit: I do not think there is much value in this test as it is the same one as on L260 except that it has parsed_args.state \u003d True -- but this functionality is tested many times below (and besides that, it is mocked out). So this is essentially already covered.","commit_id":"833ea84bb3a35ca872c8d0ce3a609b56ef0ed5a2"}]}