Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix failing test due to optional package #1258

Merged
merged 24 commits into from
Jan 22, 2024
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
4664e7b
Added a new Azure AI language recognizer
omri374 Dec 10, 2023
f6e8343
exception text change
omri374 Dec 10, 2023
81aeab7
Update supported_entities.md
omri374 Dec 10, 2023
dfbba97
flake updates
omri374 Dec 11, 2023
9ae0db1
Merge remote-tracking branch 'origin/omri/azure_language_nlp_engine' …
omri374 Dec 11, 2023
d66de47
renamed the TA service
omri374 Dec 11, 2023
0106b95
removed previous implementations
omri374 Dec 11, 2023
5d3adaa
removed previous implemetations
omri374 Dec 11, 2023
c0e9004
updated Azure AI Language doc
omri374 Dec 11, 2023
687242f
Update index.md
omri374 Dec 11, 2023
cabaa8c
Merge branch 'main' into omri/azure_language_nlp_engine
omri374 Dec 14, 2023
57e42cf
Update docs/samples/python/text_analytics/index.md
omri374 Dec 24, 2023
327d422
Merge branch 'main' into omri/azure_language_nlp_engine
omri374 Dec 24, 2023
06fcc8d
Update supported_entities.md
omri374 Dec 31, 2023
bca6a5e
revert unneeded changes
omri374 Jan 8, 2024
c16d485
Merge remote-tracking branch 'origin/omri/azure_language_nlp_engine' …
omri374 Jan 8, 2024
fcd26d6
Merge branch 'main' into omri/azure_language_nlp_engine
omri374 Jan 8, 2024
8419b47
fixed failing test due to ta_client available but key absent
omri374 Jan 9, 2024
aa3ba77
Merge remote-tracking branch 'origin/omri/azure_language_nlp_engine' …
omri374 Jan 9, 2024
fbc1d90
skip test if optional package is not installed
omri374 Jan 22, 2024
9baf050
merged with main
omri374 Jan 22, 2024
eee7672
Update index.md
omri374 Jan 22, 2024
32d5db0
Update test_azure_ai_language_recognizer.py
omri374 Jan 22, 2024
3959876
Update test_azure_ai_language_recognizer.py
omri374 Jan 22, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 11 additions & 6 deletions presidio-analyzer/tests/test_azure_ai_language_recognizer.py
Original file line number Diff line number Diff line change
@@ -1,14 +1,11 @@
import os
from unittest.mock import patch, MagicMock
import importlib
from unittest.mock import MagicMock

import pytest
from azure.ai.textanalytics import PiiEntity, TextAnalyticsClient, \
RecognizePiiEntitiesResult
from azure.core.credentials import AzureKeyCredential

from presidio_analyzer.predefined_recognizers import AzureAILanguageRecognizer
import dotenv

from presidio_analyzer.predefined_recognizers import AzureAILanguageRecognizer

dotenv.load_dotenv()

Expand Down Expand Up @@ -61,6 +58,14 @@ def test_analyze_subset_of_entities_on_init():


def test_mocked_entities_match_recognizer_results():
try:
importlib.import_module("azure.ai.textanalytics")
except ImportError:
pytest.skip("Skipping test because 'azure.ai.textanalytics' is not installed")

from azure.ai.textanalytics import PiiEntity, TextAnalyticsClient, \
RecognizePiiEntitiesResult
from azure.core.credentials import AzureKeyCredential

ent1 = PiiEntity(text="Raj", category="Person",
length=3, offset=0, confidence_score=0.8)
Expand Down