Skip to content

Commit 5f0381d

Browse files
xiaofan-luanclaude
andcommitted
fix: add missing convert_timestamptz in timestamptz default value test
The test_milvus_client_timestamptz_add_field_with_default_value test was missing a convert_timestamptz() call to normalize expected timestamp values before comparison. This caused flaky failures when CI environments had different tzdata versions or when the server returned timestamps in a slightly different format. All other timestamptz tests already use this normalization pattern. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: xiaofanluan <xiaofan.luan@zilliz.com>
1 parent 1c0302a commit 5f0381d

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tests/python_client/milvus_client/test_milvus_client_timestamptz.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -746,6 +746,7 @@ def test_milvus_client_timestamptz_add_field_with_default_value(self):
746746
# step 3: query the rows
747747
for row in rows:
748748
row[default_timestamp_field_name] = default_timestamp_value
749+
rows = cf.convert_timestamptz(rows, default_timestamp_field_name, "UTC")
749750
self.query(client, collection_name, filter=f"{default_primary_key_field_name} >= 0",
750751
check_task=CheckTasks.check_query_results,
751752
check_items={exp_res: rows,

0 commit comments

Comments
 (0)