Skip to content

Commit 0526956

Browse files
Clean up null-values test comments
Remove bug investigation comments from the null-values endpoint since the issue is now resolved and the test is part of the E2E suite.
1 parent 8ad42ac commit 0526956

2 files changed

Lines changed: 2 additions & 10 deletions

File tree

drift/instrumentation/psycopg/e2e-tests/src/app.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -733,10 +733,7 @@ def test_binary_format():
733733

734734
@app.route("/test/null-values")
735735
def test_null_values():
736-
"""Test handling of NULL values in results.
737-
738-
BUG INVESTIGATION: NULL value serialization/deserialization may have issues.
739-
"""
736+
"""Test handling of NULL values in results."""
740737
try:
741738
with psycopg.connect(get_conn_string()) as conn, conn.cursor() as cur:
742739
# Create temp table with nullable columns

drift/instrumentation/psycopg/e2e-tests/src/test_requests.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -69,12 +69,7 @@
6969
make_request("GET", "/test/scalar-row-factory")
7070
make_request("GET", "/test/binary-format")
7171

72-
# =====================================================================
73-
# PREVIOUSLY BUG-EXPOSING TEST ENDPOINTS
74-
# See BUG_TRACKING.md for detailed analysis.
75-
# =====================================================================
76-
77-
# Test 3: NULL values handling - FIXED (lazy capture mechanism)
72+
# Test: NULL values handling (integrated into E2E suite)
7873
make_request("GET", "/test/null-values")
7974

8075
# These still need investigation:

0 commit comments

Comments
 (0)