)]}'
{"nova/tests/unit/api/openstack/compute/test_block_device_mapping_v1.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"a83c038d9624c4b1b24d665e2bf1fa2ff3b13fd9","unresolved":false,"context_lines":[{"line_number":321,"context_line":"               {\u0027device_name\u0027: \u0027foo2\u0027, \u0027volume_id\u0027: fakes.FAKE_UUID,"},{"line_number":322,"context_line":"                \u0027delete_on_termination\u0027: True},"},{"line_number":323,"context_line":"               {\u0027device_name\u0027: \u0027foo3\u0027, \u0027volume_id\u0027: fakes.FAKE_UUID,"},{"line_number":324,"context_line":"                \u0027delete_on_termination\u0027: \u0027invalid\u0027},"},{"line_number":325,"context_line":"               {\u0027device_name\u0027: \u0027foo4\u0027, \u0027volume_id\u0027: fakes.FAKE_UUID,"},{"line_number":326,"context_line":"                \u0027delete_on_termination\u0027: False},"},{"line_number":327,"context_line":"               {\u0027device_name\u0027: \u0027foo5\u0027, \u0027volume_id\u0027: fakes.FAKE_UUID,"}],"source_content_type":"text/x-python","patch_set":7,"id":"da4df55a_5389e6c9","side":"PARENT","line":324,"range":{"start_line":324,"start_character":42,"end_line":324,"end_character":49},"updated":"2016-12-22 07:34:32.000000000","message":"Humm, it is necessary to change here to string \u0027False\u0027 instead of removing because 1st case checks the string on the test.","commit_id":"6f1a39d7dcb64efb1208bc0ac6d4a244316f86c5"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"99addafc6b73297c65a25ea5340a45b69f8e373a","unresolved":false,"context_lines":[{"line_number":364,"context_line":"            self.assertEqual(kwargs[\u0027block_device_mapping\u0027], bdm)"},{"line_number":365,"context_line":"            return old_create(*args, **kwargs)"},{"line_number":366,"context_line":""},{"line_number":367,"context_line":"        self.stubs.Set(compute_api.API, \u0027create\u0027, create)"},{"line_number":368,"context_line":"        self.assertRaises(self.validation_error,"},{"line_number":369,"context_line":"                          self._test_create, params)"},{"line_number":370,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"da4df55a_6e37dde3","line":367,"range":{"start_line":367,"start_character":8,"end_line":367,"end_character":57},"updated":"2016-12-22 07:39:38.000000000","message":"We need to use mock instead of stubs.Set()..","commit_id":"28b2376abb7e3a1337e38d24a2b5d24aa8ceb8a5"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"99addafc6b73297c65a25ea5340a45b69f8e373a","unresolved":false,"context_lines":[{"line_number":413,"context_line":"        self.assertRaises(exc.HTTPBadRequest, self._test_create, params,"},{"line_number":414,"context_line":"                          override_controller\u003dboth_controllers)"},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"    def test_create_instance_with_invalid_bdm_in_2nd_dict(self):"},{"line_number":417,"context_line":"        bdm \u003d [{\u0027delete_on_termination\u0027: True,"},{"line_number":418,"context_line":"                \u0027device_name\u0027: \u0027vda\u0027,"},{"line_number":419,"context_line":"                \u0027volume_size\u0027: 1,"}],"source_content_type":"text/x-python","patch_set":8,"id":"da4df55a_ee510d10","line":416,"updated":"2016-12-22 07:39:38.000000000","message":"I don\u0027t think this test is mandatory because the above test seems enough to detect the bug.","commit_id":"28b2376abb7e3a1337e38d24a2b5d24aa8ceb8a5"}]}