fix: fix failing e2e tests

[sohankshirsagar]

No description provided.

[cubic-dev-ai]

3 issues found across 9 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="drift/instrumentation/psycopg2/instrumentation.py">

<violation number="1" location="drift/instrumentation/psycopg2/instrumentation.py:222">
P1: InstrumentedConnection.cursor() unconditionally supplies cursor_factory based only on the cursor() argument, which can override a cursor_factory set on connect() and change cursor behavior (e.g., RealDictCursor no longer default). Preserve the connect()-time cursor_factory when cursor() is called without one (e.g., store it on the wrapper and use it as the base_factory).</violation>

<violation number="2" location="drift/instrumentation/psycopg2/instrumentation.py:613">
P2: In REPLAY executemany(), normalize params_list to a list before hashing/matching mocks to keep parity with RECORD mode and avoid iterator/serialization issues.</violation>

<violation number="3" location="drift/instrumentation/psycopg2/instrumentation.py:840">
P2: deserialize_db_value() is applied to all mocked row values and will coerce any ISO-looking string into a datetime (including date-only strings), potentially changing types for text columns. Consider narrowing parsing (e.g., require a time component like 'T' / ':'), or use column type info (type_code) to only deserialize known datetime/timestamp columns.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}