Skip to content

Commit 634cfa1

Browse files
committed
fix: batch executor formatting + test status string fix
1 parent 33a6d14 commit 634cfa1

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

redisvl/migration/batch_executor.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,9 @@ def _migrate_single_index(
221221
)
222222

223223
# Sanitize index_name to prevent path traversal
224-
safe_name = index_name.replace("/", "_").replace("\\", "_").replace("..", "_")
224+
safe_name = (
225+
index_name.replace("/", "_").replace("\\", "_").replace("..", "_")
226+
)
225227
report_file = report_dir / f"{safe_name}_report.yaml"
226228
write_yaml(report.model_dump(exclude_none=True), str(report_file))
227229

tests/unit/test_batch_migration.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -831,7 +831,7 @@ def test_resume_from_checkpoint(self, tmp_path):
831831
completed=[
832832
BatchIndexState(
833833
name="idx1",
834-
status="succeeded",
834+
status="success",
835835
completed_at="2026-03-20T10:05:00Z",
836836
)
837837
],
@@ -970,7 +970,7 @@ def test_retry_failed_on_resume(self, tmp_path):
970970
name="idx1", status="failed", completed_at="2026-03-20T10:03:00Z"
971971
),
972972
BatchIndexState(
973-
name="idx2", status="succeeded", completed_at="2026-03-20T10:05:00Z"
973+
name="idx2", status="success", completed_at="2026-03-20T10:05:00Z"
974974
),
975975
],
976976
current_index=None,
@@ -1046,7 +1046,7 @@ def progress_callback(index_name, position, total, status):
10461046
"index": "idx1",
10471047
"pos": 1,
10481048
"total": 3,
1049-
"status": "succeeded",
1049+
"status": "success",
10501050
}
10511051

10521052

0 commit comments

Comments
 (0)