)]}'
{"oslo_db/sqlalchemy/test_migrations.py":[{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"d89c0632d298bafddebea0df5d308ae4c2cbb13e","unresolved":false,"context_lines":[{"line_number":595,"context_line":"                          \u0027 for running of this test: %s\u0027 % e)"},{"line_number":596,"context_line":""},{"line_number":597,"context_line":"        # drop all tables after a test run"},{"line_number":598,"context_line":"        self.addCleanup(functools.partial(self.db.backend.drop_all_objects,"},{"line_number":599,"context_line":"                                          self.get_engine()))"},{"line_number":600,"context_line":""},{"line_number":601,"context_line":"        # run migration scripts"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_35603fa5","line":598,"updated":"2016-12-05 09:40:06.000000000","message":"Ah, good catch! \n\nCan we try to fix this a bit differently: instead of restoring the previous assumption create an instance of Backend explicitly here? (as you correctly point out, previously we assumed self.db is present, but may not be the case, if the test case does not use oslo.db fixtures or base classes)","commit_id":"7a65f4d23be578de9ff4b49252a396a1e4f9f660"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"94459b6b8d4ef2f7e99bd254b27fe196bdc74e04","unresolved":false,"context_lines":[{"line_number":595,"context_line":"                          \u0027 for running of this test: %s\u0027 % e)"},{"line_number":596,"context_line":""},{"line_number":597,"context_line":"        # drop all tables after a test run"},{"line_number":598,"context_line":"        self.addCleanup(functools.partial(self.db.backend.drop_all_objects,"},{"line_number":599,"context_line":"                                          self.get_engine()))"},{"line_number":600,"context_line":""},{"line_number":601,"context_line":"        # run migration scripts"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a71b18c_d0c39987","line":598,"in_reply_to":"3a71b18c_35603fa5","updated":"2016-12-05 09:57:48.000000000","message":"I have noticed the same thing, perhaps https://review.openstack.org/#/c/406861/ is a better approach.","commit_id":"7a65f4d23be578de9ff4b49252a396a1e4f9f660"}]}