Skip to content

test: add precheck API policy regression coverage#32

Merged
Shaivpidadi merged 3 commits into
devfrom
feat/vega-gov-589-precheck-policy-regression
May 12, 2026
Merged

test: add precheck API policy regression coverage#32
Shaivpidadi merged 3 commits into
devfrom
feat/vega-gov-589-precheck-policy-regression

Conversation

@Shaivpidadi

@Shaivpidadi Shaivpidadi commented Apr 24, 2026

Copy link
Copy Markdown
Member

Summary

  • cover every tool in policy.tool_access.yaml across both static-YAML and dynamic-payload precheck paths
  • preserve legacy allowed:/tokenized:/redacted: reason codes alongside pii.* reason codes for backward compatibility
  • lock static-path budget enforcement, unknown-tool default-redact reasons, and dashed/undashed SSN tokenization

Files touched

  • app/policies.py
  • tests/api/init.py
  • tests/api/test_precheck_policy_regression.py

Verification

  • cd precheck && venv/bin/pytest --no-cov -q tests

Related

  • MUL-589

@Shaivpidadi
Shaivpidadi merged commit d4aea59 into dev May 12, 2026
5 checks passed
@Shaivpidadi
Shaivpidadi deleted the feat/vega-gov-589-precheck-policy-regression branch May 12, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant