)]}'
{"nova/tests/compute/test_compute.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"8b31143986c5f4868afccbb2499449289f1978e9","unresolved":false,"context_lines":[{"line_number":6519,"context_line":""},{"line_number":6520,"context_line":"        self.compute.confirm_resize(self.context, instance,"},{"line_number":6521,"context_line":"                                    migration\u003dmigration)"},{"line_number":6522,"context_line":"        self.compute.terminate_instance(self.context, instance\u003dinstance)"},{"line_number":6523,"context_line":""},{"line_number":6524,"context_line":"    def test_resize_revert_through_api(self):"},{"line_number":6525,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2Fsdk%3D","line":6522,"updated":"2013-06-12 16:44:46.000000000","message":"This test doesn\u0027t assert anything.  Are you just looking for lack of exceptions?","commit_id":"dc97214bd30177b056144199cded5bc3b514a8dd"},{"author":{"_account_id":7532,"name":"Aditi Raveesh","email":"aditi.raveesh@gmail.com","username":"aditiraveesh"},"change_message_id":"7e30d58b5a67db52fce77787300bc5578358abb7","unresolved":false,"context_lines":[{"line_number":6519,"context_line":""},{"line_number":6520,"context_line":"        self.compute.confirm_resize(self.context, instance,"},{"line_number":6521,"context_line":"                                    migration\u003dmigration)"},{"line_number":6522,"context_line":"        self.compute.terminate_instance(self.context, instance\u003dinstance)"},{"line_number":6523,"context_line":""},{"line_number":6524,"context_line":"    def test_resize_revert_through_api(self):"},{"line_number":6525,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FsZ0%3D","line":6522,"in_reply_to":"AAAAPX%2F%2FsaM%3D","updated":"2013-06-12 17:02:57.000000000","message":"If thats the case, the terminate_instance step isn\u0027t needed,\nsince that is not what I\u0027m trying to test. So I will remove that check.","commit_id":"dc97214bd30177b056144199cded5bc3b514a8dd"},{"author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"change_message_id":"75f765476c78361975f464e16f83e692cface25d","unresolved":false,"context_lines":[{"line_number":6519,"context_line":""},{"line_number":6520,"context_line":"        self.compute.confirm_resize(self.context, instance,"},{"line_number":6521,"context_line":"                                    migration\u003dmigration)"},{"line_number":6522,"context_line":"        self.compute.terminate_instance(self.context, instance\u003dinstance)"},{"line_number":6523,"context_line":""},{"line_number":6524,"context_line":"    def test_resize_revert_through_api(self):"},{"line_number":6525,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FsaM%3D","line":6522,"in_reply_to":"AAAAPX%2F%2FscM%3D","updated":"2013-06-12 16:59:49.000000000","message":"So I think the pattern in a lot of the existing tests is flawed. An assert Not Raises at the very least would be nice.","commit_id":"dc97214bd30177b056144199cded5bc3b514a8dd"},{"author":{"_account_id":7532,"name":"Aditi Raveesh","email":"aditi.raveesh@gmail.com","username":"aditiraveesh"},"change_message_id":"ce10bb5c2da9a129a1951d8a5ddc36daae98f168","unresolved":false,"context_lines":[{"line_number":6519,"context_line":""},{"line_number":6520,"context_line":"        self.compute.confirm_resize(self.context, instance,"},{"line_number":6521,"context_line":"                                    migration\u003dmigration)"},{"line_number":6522,"context_line":"        self.compute.terminate_instance(self.context, instance\u003dinstance)"},{"line_number":6523,"context_line":""},{"line_number":6524,"context_line":"    def test_resize_revert_through_api(self):"},{"line_number":6525,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FscM%3D","line":6522,"in_reply_to":"AAAAPX%2F%2Fsdk%3D","updated":"2013-06-12 16:50:32.000000000","message":"This seems to be a pattern in the rest of the tests in this class, since they are almost like functional tests, so stuck to that flow.","commit_id":"dc97214bd30177b056144199cded5bc3b514a8dd"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"0cb3c44c5a3013b7b0039834ea396bb30d2fa682","unresolved":false,"context_lines":[{"line_number":6518,"context_line":"                                       \u0027system_metadata\u0027: sys_meta})"},{"line_number":6519,"context_line":""},{"line_number":6520,"context_line":"        self.compute.confirm_resize(self.context, instance,"},{"line_number":6521,"context_line":"                                    migration\u003dmigration)"},{"line_number":6522,"context_line":""},{"line_number":6523,"context_line":"    def test_resize_revert_through_api(self):"},{"line_number":6524,"context_line":"        instance \u003d jsonutils.to_primitive(self._create_fake_instance())"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2FsNw%3D","line":6521,"updated":"2013-06-12 18:20:45.000000000","message":"So, there\u0027s still no asserts in this test…","commit_id":"45e4330726f9e65c19b54480282e25e19e81fb49"}]}