)]}'
{"nova/tests/db/test_db_api.py":[{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"b43e3b388203b91ce9de0e1032bf4c8898f8d19f","unresolved":false,"context_lines":[{"line_number":1914,"context_line":"        self.assertRaises(exception.InstanceNotFound,"},{"line_number":1915,"context_line":"                          db.instance_get, ctxt, inst_uuid)"},{"line_number":1916,"context_line":"        self.assertIsNone(db.instance_info_cache_get(ctxt, inst_uuid))"},{"line_number":1917,"context_line":"        self.assertEqual({}, db.instance_metadata_get(ctxt, inst_uuid))"},{"line_number":1918,"context_line":""},{"line_number":1919,"context_line":"    def test_instance_destroy_already_destroyed(self):"},{"line_number":1920,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Bfm4%3D","line":1917,"updated":"2013-11-25 14:39:36.000000000","message":"This is already being tested in test_delete_instance_metadata_on_instance_destroy, but I guess it doesn\u0027t hurt too much.\n\nAlso - why are we not testing the securitygroup association, and that it is cleaned up properly as well?","commit_id":"eeb29d86013b48d4bc1cec3979c8665603d57f01"},{"author":{"_account_id":5511,"name":"Nikola Dipanov","email":"ndipanov@redhat.com","username":"ndipanov"},"change_message_id":"2e3e468496a32eedf2f9771f5337fcb199596c8a","unresolved":false,"context_lines":[{"line_number":1914,"context_line":"        self.assertRaises(exception.InstanceNotFound,"},{"line_number":1915,"context_line":"                          db.instance_get, ctxt, inst_uuid)"},{"line_number":1916,"context_line":"        self.assertIsNone(db.instance_info_cache_get(ctxt, inst_uuid))"},{"line_number":1917,"context_line":"        self.assertEqual({}, db.instance_metadata_get(ctxt, inst_uuid))"},{"line_number":1918,"context_line":""},{"line_number":1919,"context_line":"    def test_instance_destroy_already_destroyed(self):"},{"line_number":1920,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2BQIQ%3D","line":1917,"in_reply_to":"AAAATn%2F%2BW%2FY%3D","updated":"2013-11-28 10:06:18.000000000","message":"k - agreed","commit_id":"eeb29d86013b48d4bc1cec3979c8665603d57f01"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"f2ad03bb89d96708b41500ead6da6bf715fa9116","unresolved":false,"context_lines":[{"line_number":1914,"context_line":"        self.assertRaises(exception.InstanceNotFound,"},{"line_number":1915,"context_line":"                          db.instance_get, ctxt, inst_uuid)"},{"line_number":1916,"context_line":"        self.assertIsNone(db.instance_info_cache_get(ctxt, inst_uuid))"},{"line_number":1917,"context_line":"        self.assertEqual({}, db.instance_metadata_get(ctxt, inst_uuid))"},{"line_number":1918,"context_line":""},{"line_number":1919,"context_line":"    def test_instance_destroy_already_destroyed(self):"},{"line_number":1920,"context_line":"        ctxt \u003d context.get_admin_context()"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2BW%2FY%3D","line":1917,"in_reply_to":"AAAATn%2F%2Bfm4%3D","updated":"2013-11-26 23:20:21.000000000","message":"About instance_system_metadata() I am going to test both here metadata and system_metadata() here. \n\nSo I think I will remove that method when I finish work around system_metadata. (in future patches)\n\n\nAbout secgroup association, I am not sure how to test it?)\nWith mock?\n\nImho probably it will be better just to merge this patch, because it fix small but bug..","commit_id":"eeb29d86013b48d4bc1cec3979c8665603d57f01"}]}