)]}'
{"glance/db/sqlalchemy/api.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cb1395939809679014f0f0c168875d5dc7cedbf6","unresolved":false,"context_lines":[{"line_number":1462,"context_line":"    return _task_format(task_ref, task_ref.info)"},{"line_number":1463,"context_line":""},{"line_number":1464,"context_line":""},{"line_number":1465,"context_line":"def _task_soft_delete(context, session\u003dNone):"},{"line_number":1466,"context_line":"    \"\"\"Scrub task entities which are expired \"\"\""},{"line_number":1467,"context_line":"    expires_at \u003d models.Task.expires_at"},{"line_number":1468,"context_line":"    session \u003d session or get_session()"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f6a8fd7_dc806b49","line":1465,"range":{"start_line":1465,"start_character":22,"end_line":1465,"end_character":29},"updated":"2018-05-02 19:55:57.000000000","message":"looks like this parameter is no longer used -- maybe it should be removed?","commit_id":"8005c4e3970118df09456ff03d571caa00500c51"}],"glance/tests/functional/db/base.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fc97972502fcc1226a8a0e917c5fccc375623696","unresolved":false,"context_lines":[{"line_number":1593,"context_line":"        self.context \u003d context.RequestContext("},{"line_number":1594,"context_line":"            is_admin\u003dFalse, auth_token\u003d\u0027user:user:user\u0027, user\u003dself.owner_id)"},{"line_number":1595,"context_line":"        self.db_api \u003d db_tests.get_db(self.config)"},{"line_number":1596,"context_line":"        self.fixtures \u003d self.build_task_fixtures()"},{"line_number":1597,"context_line":"        db_tests.reset_db(self.db_api)"},{"line_number":1598,"context_line":""},{"line_number":1599,"context_line":"    def build_task_fixtures(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"5f7c97a3_6cc0e87d","line":1596,"updated":"2018-06-04 21:09:07.000000000","message":"I\u0027m a bit worried about this because now all the test fixtures contain a non-None expires_at, which is not the general case: expires_at is only populated when a task reaches a terminal status (success or failure).  I think a better way to do this is to leave expires_at unpopulated in the fixture, and then explicitly populate it in those tests where we know the field will definitely be populated.","commit_id":"4bacc8c5c6998643b85d627d6ee783866a6a4848"}]}
