Skip to content

Test duplicate key namespaces#576

Draft
claell wants to merge 1 commit into
sciunto-org:mainfrom
claell:test/duplicate-key-namespaces
Draft

Test duplicate key namespaces#576
claell wants to merge 1 commit into
sciunto-org:mainfrom
claell:test/duplicate-key-namespaces

Conversation

@claell

@claell claell commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Cover strict duplicate detection for @string blocks.
  • Verify that entries and strings use separate key namespaces.
  • Keep the change test-only and document the behavior each regression protects.

Closes #575.

Validation

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

Review note

This PR makes currently passing behavior explicit; it does not change production code. Maintainer review should confirm that separate entry and string namespaces are the intended public contract.

AI assistance

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

* Cover strict duplicate detection for string blocks.
* Verify entries and strings may share a key without conflict.
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.

Test duplicate key namespaces

1 participant