test_storage: Use the db_transaction context manager instead of raw connections

This minimizes the risk of dangling db connections, which in turn reduces the chance of cascading test failures.

Depends on !258 (closed)

Test Plan

tox runs without issue

---

Migrated from D2068 (view on Phabricator)