Skip to content

test(packaging): pin native manifest field refusals - #418

Closed
harshitethic wants to merge 1 commit into
Hebbian-Robotics:mainfrom
harshitethic:test/native-overlay-field-refusals
Closed

test(packaging): pin native manifest field refusals#418
harshitethic wants to merge 1 commit into
Hebbian-Robotics:mainfrom
harshitethic:test/native-overlay-field-refusals

Conversation

@harshitethic

Copy link
Copy Markdown
Contributor

Summary

Follow up on the field-validation refusals explicitly left out of #408 by adding focused coverage for the native-overlay manifest scalar/path validators.

The tests pin refusal behavior for:

  • object and exact-field validation;
  • integer, positive and non-negative integer constraints;
  • printable strings and dotted Python identifiers;
  • POSIX-only relative paths and root-escape rejection;
  • canonical lowercase SHA-256 values.

A final acceptance case records the canonical values each helper accepts. This is deliberately separate from the real-overlay parse-stage coverage in #417: these are small boundary validators and do not need a compiler-backed overlay build for every scalar case.

Related: #408.

Validation

The change is test-only and network-free. GitHub Actions is the authoritative full-suite validation for this branch.

AI assistance disclosure: AI assistance was used to inspect the deferred scope in #408, prepare the focused parametrized tests, and review the resulting change.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

👋 Hi @harshitethic — thanks for the contribution! To keep starter issues available
for other contributors and give every pull request a real review, we accept
1 open pull request per contributor at a time.

You already have #415 open, so this one is being closed automatically.
Once your open pull request is merged or closed, feel free to reopen this one —
no work is lost.

@github-actions github-actions Bot closed this Sep 5, 2026
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