Skip to content

Commit c899229

Browse files
committed
fix: update parquet export tests to match new schema
- Change 'goal' to 'instruction' in column assertions - Change 'image_path' to 'screenshot_path' to match schema
1 parent 70749db commit c899229

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/test_parquet_export.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,13 +75,13 @@ def test_to_parquet_basic(self, sample_episodes, tmp_path):
7575
# Check expected columns exist
7676
expected_columns = [
7777
"episode_id",
78-
"goal",
78+
"instruction", # Changed from "goal" to match new schema
7979
"step_index",
8080
"timestamp",
8181
"action_type",
8282
"x",
8383
"y",
84-
"image_path",
84+
"screenshot_path", # Changed from "image_path" to match new schema
8585
]
8686
for col in expected_columns:
8787
assert col in table.column_names, f"Missing column: {col}"
@@ -108,7 +108,7 @@ def test_to_parquet_with_summary(self, sample_episodes, tmp_path):
108108
# Check summary columns
109109
assert "episode_id" in summary_table.column_names
110110
assert "step_count" in summary_table.column_names
111-
assert "goal" in summary_table.column_names
111+
assert "instruction" in summary_table.column_names # Changed from "goal"
112112

113113
def test_from_parquet_roundtrip(self, sample_episodes, tmp_path):
114114
"""Test Parquet roundtrip (lossy but reconstructable)."""

0 commit comments

Comments
 (0)