)]}'
{"nova/tests/compute/test_compute.py":[{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"d27826012e530d4efaeabc5078295674ad554c42","unresolved":false,"context_lines":[{"line_number":2782,"context_line":""},{"line_number":2783,"context_line":"        self.compute.terminate_instance(self.context, instance, [], [])"},{"line_number":2784,"context_line":""},{"line_number":2785,"context_line":"    def test_get_spice_console_not_implemented(self):"},{"line_number":2786,"context_line":"        self.flags(vnc_enabled\u003dFalse)"},{"line_number":2787,"context_line":"        self.flags(enabled\u003dTrue, group\u003d\u0027spice\u0027)"},{"line_number":2788,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAATn%2F9U94%3D","line":2785,"updated":"2013-12-12 16:19:25.000000000","message":"Looks like there is a lot of replication here with the above test case - could then be combined to a common _test_get_console_no_implemented(self, method,  type)","commit_id":"daf450948afd045148bf9b7b9ec4c9755c498430"},{"author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"change_message_id":"d27826012e530d4efaeabc5078295674ad554c42","unresolved":false,"context_lines":[{"line_number":2799,"context_line":"                          self.context, \u0027spice-html5\u0027, instance\u003dinstance)"},{"line_number":2800,"context_line":""},{"line_number":2801,"context_line":"        self.compute \u003d utils.ExceptionHelper(self.compute)"},{"line_number":2802,"context_line":""},{"line_number":2803,"context_line":"        self.assertRaises(NotImplementedError,"},{"line_number":2804,"context_line":"                          self.compute.get_spice_console,"},{"line_number":2805,"context_line":"                          self.context, \u0027spice-html5\u0027, instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAATn%2F9U9s%3D","line":2802,"updated":"2013-12-12 16:19:25.000000000","message":"Minor nit - its not really obvious why the test is checking for two different exceptions here - a comment explaining would help with future maintenance.","commit_id":"daf450948afd045148bf9b7b9ec4c9755c498430"}]}