Skip to content

Commit

Permalink
Ensure pipeline tests are deterministic
Browse files Browse the repository at this point in the history
Small bug fix for this test to actually assign the sorted value ingest
input back. Rather than just sorting and using the old ordering!
  • Loading branch information
olaughter committed Sep 12, 2024
1 parent 2c9f6f1 commit bfa7172
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/unit/app/core/test_pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ def test_generate_pipeline_ingest_input(data_db: Session):
state_rows = generate_pipeline_ingest_input(data_db)
assert len(state_rows) == 2
# Sort to ensure order is consistent across tests
sorted(state_rows, key=lambda d: d.import_id)
state_rows = sorted(state_rows, key=lambda d: d.import_id)

# Now test one field from each table we've queried
# Check family title
Expand Down

0 comments on commit bfa7172

Please sign in to comment.