Skip to content

Bugfix unstable database order - #137

Open
Louwrensth wants to merge 1 commit into
iterorganization:developfrom
Louwrensth:bugfix/unstable-database-order
Open

Louwrensth wants to merge 1 commit into
iterorganization:developfrom
Louwrensth:bugfix/unstable-database-order

Conversation

@Louwrensth

Copy link
Copy Markdown
Contributor

Summary

This PR fixes unstable pagination in the database layer (src/simdb/database/database.py) by making simulation listing order deterministic.

Changes

  • Make database pagination use a stable, deterministic sort order.
  • Raise DatabaseError when an unknown sort column is requested.
  • Update the database test fixture to simulate unordered query results by rotating rows on each Query.all() call.
  • Add regression coverage to show duplicate entries in query pagination (limit) when no stable sort is applied.

Validation

  • 24 passed in tests/database/test_metadata_queries.py

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant