Skip to content

Add TCK for deterministic recognizers#2190

Open
andreaTP wants to merge 3 commits into
data-privacy-stack:mainfrom
andreaTP:add-tck-ci-and-release
Open

Add TCK for deterministic recognizers#2190
andreaTP wants to merge 3 commits into
data-privacy-stack:mainfrom
andreaTP:add-tck-ci-and-release

Conversation

@andreaTP

@andreaTP andreaTP commented Jul 22, 2026

Copy link
Copy Markdown

Portable YAML test suite extracted from presidio-analyzer parametrized tests. Adds a CI validation job and packages the TCK data as a release asset so other implementations can validate against the same test cases.

Change Description

This PR adds a Technology Compatibility Kit for deterministic recognizers — a portable YAML test suite extracted from the existing presidio-analyzer parametrized tests (74 recognizers, 1478 test cases). The goal is to let other implementations validate their pattern/regex/checksum recognizers against the same test cases Presidio uses.

Issue reference

I thought I directly propose this in a PR, let me know if I should open a tracking Issue

Fixes #XX

Checklist

  • I have reviewed the contribution guidelines
  • I agree to follow this project's Code of Conduct
  • I confirm that I have the right to submit this contribution and that it does not knowingly contain proprietary or confidential code.
  • My code includes unit tests
  • All unit tests and lint checks pass locally
  • My PR contains documentation updates / additions if required

andreaTP added 3 commits July 22, 2026 19:17
Portable YAML test suite extracted from presidio-analyzer parametrized
tests. Adds a CI validation job and packages the TCK data as a release
asset so other implementations can validate against the same test cases.
Comment thread tck/uv.lock

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The lock file makes this PR look huge, sorry for the noise

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant