[HIVE-25676] Uncaught exception in QTestDatabaseHandler#afterTest causes unrelated test failures

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-1
• Component/s: Testing Infrastructure
• Labels: pull-request-available
• Target Version/s: 4.0.0
• Assignee: Stamatis Zampetakis
• Reporter: Stamatis Zampetakis
• Created: 05/Nov/21 12:59
• Updated: 17/Nov/22 08:49
• Resolved: 15/Nov/21 16:17

Description

When for some reason we fail to cleanup a database after running a test using the qt:database option an exception is raised and propagates up the stack. Not catching it in QTestDatabaseHandler#afterTest disrupts subsequent cleanup actions, which are not executed, and leads to failures in subsequent tests which are not related.

Moreover, the exception leaves QTestDatabaseHandler in an invalid state since the internal map holding the running databases is not updated.

Issue Links

• links to: GitHub Pull Request #2766