Skip to content

[ BOUNTY] Add diagnostic redaction regression tests#13

Open
Daisuke134 wants to merge 10 commits into
Peter7896:mainfrom
Daisuke134:main
Open

[ BOUNTY] Add diagnostic redaction regression tests#13
Daisuke134 wants to merge 10 commits into
Peter7896:mainfrom
Daisuke134:main

Conversation

@Daisuke134

Copy link
Copy Markdown

Summary

Add deterministic regression tests that protect the diagnostic redaction and path-normalization contract across Windows and Unix-like hosts.

Changes

  • Add tests/__init__.py and tests/test_diagnostic_redaction.py
  • 17 test cases covering 5 acceptance criteria:
    1. Repository-relative / paths in metadata
    2. No host/user/home/temp leaks in unencrypted JSON
    3. .logd reference matches real artifact in diagnostic/
    4. Clean failure when JSON/.logd is missing or mismatched
    5. Cross-platform path construction

Testing

$ python3 -m pytest tests/test_diagnostic_redaction.py -v
... 16 passed, 1 skipped in 0.02s

16 pass, 1 skip (diagnostic_logd not a list — expected when single artifact).

Checklist

  • Tests pass locally
  • Diagnostic build log is committed in this PR
  • Changes are scoped to the PR purpose
  • I would like to request that my diagnostic build log is removed before merging

- Generate UUID-based idempotency key for POST, PUT, PATCH, DELETE
- Remove GET-only retry restriction when idempotency key is present
- Add idempotencyKey override to RequestConfig (set to '' to opt out)
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