Skip to content

Isolate and strengthen field-sorting tests#578

Draft
claell wants to merge 1 commit into
sciunto-org:mainfrom
claell:test/stable-field-sorting
Draft

Isolate and strengthen field-sorting tests#578
claell wants to merge 1 commit into
sciunto-org:mainfrom
claell:test/stable-field-sorting

Conversation

@claell

@claell claell commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Isolate in-place field-sorting cases with fresh entry fixtures.
  • Verify stable ordering for fields omitted from a custom order.
  • Exercise custom sorting through the public writer without source mutation.
  • Reject ambiguous custom orders, including case-only duplicates.

Closes #577.

Validation

  • pytest -p no:cacheprovider -W error tests/middleware_tests/test_sorting_entry_fields.py: 8 passed.
  • Complete upstream test suite: 2,579 passed, 12 skipped, with the four existing deprecation warnings in tests/test_entrypoint.py.
  • Complete pre-commit hook suite: passed.

Review note

This PR changes only tests. In particular, it replaces an execution-order-dependent expected value with the stable ordering produced from a fresh input. Maintainer review should confirm that unspecified fields retaining their original relative order is the intended contract.

AI assistance

This pull request was prepared with ChatGPT Codex using GPT-5.6 Sol with high reasoning effort. Codex assisted with test isolation, behavior analysis, branch preparation, and validation. Automated validation is not a substitute for maintainer review.

* Isolate in-place cases with fresh entry fixtures.
* Verify stable ordering for unspecified fields and public writer integration.
* Reject ambiguous custom orders, including case-only duplicates.
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.

Isolate and strengthen field-sorting tests

1 participant