From 93138f0483eb1522fba9686776f0099abf7c4771 Mon Sep 17 00:00:00 2001
From: Jason Staack <jason@staack.com>
Date: Sun, 15 Mar 2026 06:23:14 -0500
Subject: [PATCH] fix(ci): clean up test data before AND after each test

Prevents stale data from prior tests/runs from causing false failures
like test_list_devices_empty finding leftover devices.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 backend/tests/integration/conftest.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/backend/tests/integration/conftest.py b/backend/tests/integration/conftest.py
index 0189b83..c2cad1a 100644
--- a/backend/tests/integration/conftest.py
+++ b/backend/tests/integration/conftest.py
@@ -165,6 +165,13 @@ async def admin_session(admin_engine) -> AsyncGenerator[AsyncSession, None]:
     Cleanup deletes all rows from test tables after the test.
     """
     session = AsyncSession(admin_engine, expire_on_commit=False)
+    # Clean up any leftover data from previous tests/runs BEFORE yielding
+    for table in _CLEANUP_TABLES:
+        try:
+            await session.execute(text(f"DELETE FROM {table}"))
+        except Exception:
+            pass
+    await session.commit()
     try:
         yield session
     finally: