)]}'
{"nova/tests/unit/compute/test_compute.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"f189314fc773d269e0cbc13f20a77ff138c8e1a9","unresolved":false,"context_lines":[{"line_number":9957,"context_line":"                min_count\u003dnum_instances, max_count\u003dnum_instances,"},{"line_number":9958,"context_line":"                display_name\u003d\u0027x\u0027)"},{"line_number":9959,"context_line":"        for i in range(num_instances):"},{"line_number":9960,"context_line":"            name \u003d \u0027x-%s\u0027 % (i + 1,)"},{"line_number":9961,"context_line":"            self.assertEqual(refs[i][\u0027display_name\u0027], name)"},{"line_number":9962,"context_line":"            self.assertEqual(refs[i][\u0027hostname\u0027], name)"},{"line_number":9963,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"bfb3d3c7_cd08d6da","line":9960,"range":{"start_line":9960,"start_character":34,"end_line":9960,"end_character":35},"updated":"2019-05-21 21:10:03.000000000","message":"[unrelated] this is a weird and unnecessary comma.","commit_id":"59784cfa6c44d1c330842593809f15c3de3fbad4"}],"nova/tests/unit/compute/test_compute_cells.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"fe90ff141010e6a7994a8e5db8bad8b9893884d2","unresolved":false,"context_lines":[{"line_number":455,"context_line":"        # one targeted message should have been created"},{"line_number":456,"context_line":"        self.assertEqual(1, mock_msg.call_count)"},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"    def test_populate_instance_for_create(self):"},{"line_number":459,"context_line":"        self.skipTest(\"Removing cells v1\")"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    def test_multi_instance_display_name(self):"},{"line_number":462,"context_line":"        self.skipTest(\"Removing cells v1\")"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"    @ddt.data(True, False)"},{"line_number":465,"context_line":"    def test_rdp_console(self, enabled_consoleauth):"}],"source_content_type":"text/x-python","patch_set":9,"id":"dfbec78f_fb70800a","line":462,"range":{"start_line":458,"start_character":0,"end_line":462,"end_character":42},"updated":"2019-05-14 22:35:50.000000000","message":"Why do we need to make these tests skipped instead of just removing?","commit_id":"81ce428e4a6f8332548ce6584223ed3e0e200b73"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d34abc1b9bd7534e7237639720e87880c8d6e1f1","unresolved":false,"context_lines":[{"line_number":455,"context_line":"        # one targeted message should have been created"},{"line_number":456,"context_line":"        self.assertEqual(1, mock_msg.call_count)"},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"    def test_populate_instance_for_create(self):"},{"line_number":459,"context_line":"        self.skipTest(\"Removing cells v1\")"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    def test_multi_instance_display_name(self):"},{"line_number":462,"context_line":"        self.skipTest(\"Removing cells v1\")"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"    @ddt.data(True, False)"},{"line_number":465,"context_line":"    def test_rdp_console(self, enabled_consoleauth):"}],"source_content_type":"text/x-python","patch_set":9,"id":"dfbec78f_75432a43","line":462,"range":{"start_line":458,"start_character":0,"end_line":462,"end_character":42},"in_reply_to":"dfbec78f_cadd9ae0","updated":"2019-05-16 01:49:36.000000000","message":"I see, thanks for your explanation.","commit_id":"81ce428e4a6f8332548ce6584223ed3e0e200b73"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9b166853dfe09c15c621d95bea57fd6555f8eb30","unresolved":false,"context_lines":[{"line_number":455,"context_line":"        # one targeted message should have been created"},{"line_number":456,"context_line":"        self.assertEqual(1, mock_msg.call_count)"},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"    def test_populate_instance_for_create(self):"},{"line_number":459,"context_line":"        self.skipTest(\"Removing cells v1\")"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    def test_multi_instance_display_name(self):"},{"line_number":462,"context_line":"        self.skipTest(\"Removing cells v1\")"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"    @ddt.data(True, False)"},{"line_number":465,"context_line":"    def test_rdp_console(self, enabled_consoleauth):"}],"source_content_type":"text/x-python","patch_set":9,"id":"dfbec78f_cadd9ae0","line":462,"range":{"start_line":458,"start_character":0,"end_line":462,"end_character":42},"in_reply_to":"dfbec78f_fb70800a","updated":"2019-05-15 14:14:14.000000000","message":"It\u0027s because this module inherits from and overrides \u0027nova.tests.unit.compute.test_compute.ComputeAPITestCase\u0027. Without this, we\u0027d run the version of the tests found in the base class which don\u0027t work with cells v1.","commit_id":"81ce428e4a6f8332548ce6584223ed3e0e200b73"}]}