Skip to content

Commit

Permalink
Fix another DROP DATABASE test flake (#6637)
Browse files Browse the repository at this point in the history
In #4629 we introduced a `drop_tb` testbase function for dropping
databases in tests that works around #4567, but #5032 added some new
tests that used `DROP DATABASE` directly.

This caused a test flake on a 3.x release build today; hopefully it
passes on the retry without me needing to cherry-pick this fix first.
  • Loading branch information
msullivan authored Dec 22, 2023
1 parent 90f9059 commit ec8e38e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/test_database.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ async def test_database_drop_recreate(self):
await conn.aclose()

finally:
await self.con.execute('DROP DATABASE test_db_drop;')
await tb.drop_db(self.con, 'test_db_drop')

async def test_database_non_exist_template(self):
if not self.has_create_database:
Expand All @@ -141,4 +141,4 @@ async def test_database_non_exist_template(self):
await conn.aclose()

finally:
await self.con.execute('DROP DATABASE test_tpl;')
await tb.drop_db(self.con, 'test_tpl')

0 comments on commit ec8e38e

Please sign in to comment.