Skip to content

test: add safe prose privacy coverage - #372

Merged
ashrafee-dev merged 1 commit into
ashrafee-dev:mainfrom
sivaadithya25:issue-180-safe-prose
Sep 29, 2026
Merged

ashrafee-dev merged 1 commit into
ashrafee-dev:mainfrom
sivaadithya25:issue-180-safe-prose

Conversation

@sivaadithya25

Copy link
Copy Markdown
Contributor

Summary

Adds focused regression tests for privacy filtering to ensure benign prose is preserved exactly.

Changes

  • Added tests/roadmap/privacy/test_safe_prose.py
  • Added deterministic tests for ordinary benign prose
  • Verified exact preservation with filter_sensitive(text) == text
  • Covered punctuation such as commas, dashes, and parentheses
  • No runtime behavior was changed

Verification

  • uv run pytest tests/roadmap/privacy/test_safe_prose.py -q → 4 passed
  • uv run pylint app tests → 10.00/10
  • bash scripts/check.sh could not be run in the Windows environment because /bin/bash is unavailable

Related: #180

Copilot AI balanced review requested due to automatic review settings September 29, 2026 05:33
@vercel

vercel Bot commented Sep 29, 2026

Copy link
Copy Markdown

@sivaadithya25 is attempting to deploy a commit to the ashrafee-dev's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@sivaadithya25

Copy link
Copy Markdown
Contributor Author

Hi @ashrafee-dev, I’ve completed the changes for #180 and opened this PR for review. The focused tests pass (4 passed) and pylint reports 10.00/10. The two Vercel checks currently show “Authorization required to deploy.”

Please review when you get a chance. Thank you!

@ashrafee-dev
ashrafee-dev merged commit 343df1f into ashrafee-dev:main Sep 29, 2026
3 of 5 checks passed
@ashrafee-dev

Copy link
Copy Markdown
Owner

Awesome! Thank you for your contribution! @sivaadithya25

@sivaadithya25
sivaadithya25 deleted the issue-180-safe-prose branch September 29, 2026 09:45
@sivaadithya25

Copy link
Copy Markdown
Contributor Author

It is grate to contribute for you

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.

3 participants