Test for yaml config for custom recognizers#1720
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds test coverage for custom recognizers defined via YAML configuration through the AnalyzerEngineProvider. The primary purpose is to validate that custom pattern-based recognizers can be successfully loaded and utilized from YAML configuration files.
- Adds a new test function to verify custom recognizer functionality
- Creates a YAML configuration file defining a ZIP code recognizer with patterns and context
- Imports
PatternRecognizerclass to support the new test functionality
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| presidio-analyzer/tests/test_analyzer_engine_provider.py | Adds import for PatternRecognizer and new test function for custom recognizer validation |
| presidio-analyzer/tests/conf/custom_recognizer_yaml.yaml | New YAML configuration file defining a ZIP code recognizer with pattern matching and context |
SharonHart
approved these changes
Sep 10, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change Description
Testing that custom recognizers can be successfully defined in YAML through the
AnalyzerEngineProviderIssue reference
Fixes #1696
Checklist