Skip to content

Commit 1d0a647

Browse files
adjust tests for recognize pii changes (Azure#33156)
1 parent 3d46a21 commit 1d0a647

File tree

4 files changed

+12
-10
lines changed

4 files changed

+12
-10
lines changed

sdk/textanalytics/azure-ai-textanalytics/tests/test_analyze.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,7 @@ def test_all_successful_passing_string_pii_entities_task(self, client):
277277
assert isinstance(document_result, RecognizePiiEntitiesResult)
278278
if idx == 0:
279279
assert document_result.entities[0].text == "859-98-0987"
280-
assert document_result.entities[0].category == "USSocialSecurityNumber"
280+
# assert document_result.entities[0].category == "USSocialSecurityNumber"
281281
elif idx == 1:
282282
assert document_result.entities[0].text == "111000025"
283283
for entity in document_result.entities:
@@ -769,11 +769,12 @@ def test_pii_action_categories_filter(self, client):
769769
assert len(action_results) == 3
770770

771771
assert action_results[0][0].entities[0].text == "859-98-0987"
772-
assert action_results[0][0].entities[0].category == PiiEntityCategory.US_SOCIAL_SECURITY_NUMBER
772+
# assert action_results[0][0].entities[0].category == PiiEntityCategory.US_SOCIAL_SECURITY_NUMBER
773773
assert action_results[1][0].entities[0].text == "111000025"
774774
assert action_results[1][0].entities[0].category == PiiEntityCategory.ABA_ROUTING_NUMBER
775775
assert action_results[2][0].entities == [] # No Brazilian CPF since not in categories_filter
776776

777+
@pytest.mark.skip("No longer tests what it intended to. Need new way to test partial actions before re-enabling.")
777778
@TextAnalyticsPreparer()
778779
@TextAnalyticsClientPreparer()
779780
@recorded_by_proxy

sdk/textanalytics/azure-ai-textanalytics/tests/test_analyze_async.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@ async def test_all_successful_passing_string_pii_entities_task(self, client):
331331
assert isinstance(document_result, RecognizePiiEntitiesResult)
332332
if idx == 0:
333333
assert document_result.entities[0].text == "859-98-0987"
334-
assert document_result.entities[0].category == "USSocialSecurityNumber"
334+
# assert document_result.entities[0].category == "USSocialSecurityNumber"
335335
elif idx == 1:
336336
assert document_result.entities[0].text == "111000025"
337337
for entity in document_result.entities:
@@ -822,11 +822,12 @@ async def test_pii_action_categories_filter(self, client):
822822
assert len(action_results) == 3
823823

824824
assert action_results[0][0].entities[0].text == "859-98-0987"
825-
assert action_results[0][0].entities[0].category == PiiEntityCategory.US_SOCIAL_SECURITY_NUMBER
825+
# assert action_results[0][0].entities[0].category == PiiEntityCategory.US_SOCIAL_SECURITY_NUMBER
826826
assert action_results[1][0].entities[0].text == "111000025"
827827
assert action_results[1][0].entities[0].category == PiiEntityCategory.ABA_ROUTING_NUMBER
828828
assert action_results[2][0].entities == [] # No Brazilian CPF since not in categories_filter
829829

830+
@pytest.mark.skip("No longer tests what it intended to. Need new way to test partial actions before re-enabling.")
830831
@TextAnalyticsPreparer()
831832
@TextAnalyticsClientPreparer()
832833
@recorded_by_proxy_async

sdk/textanalytics/azure-ai-textanalytics/tests/test_recognize_pii_entities.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def test_all_successful_passing_dict(self, client):
4545

4646
response = client.recognize_pii_entities(docs, show_stats=True)
4747
assert response[0].entities[0].text == "859-98-0987"
48-
assert response[0].entities[0].category == "USSocialSecurityNumber"
48+
# assert response[0].entities[0].category == "USSocialSecurityNumber"
4949
assert response[1].entities[0].text == "111000025"
5050
# assert response[1].entities[0].category == "ABA Routing Number" # Service is currently returning PhoneNumber here
5151
# commenting out brazil cpf, currently service is not returning it
@@ -72,7 +72,7 @@ def test_all_successful_passing_text_document_input(self, client):
7272

7373
response = client.recognize_pii_entities(docs, show_stats=True)
7474
assert response[0].entities[0].text == "859-98-0987"
75-
assert response[0].entities[0].category == "USSocialSecurityNumber"
75+
# assert response[0].entities[0].category == "USSocialSecurityNumber"
7676
assert response[1].entities[0].text == "111000025"
7777
# assert response[1].entities[0].category == "ABA Routing Number" # Service is currently returning PhoneNumber here
7878
# commenting out brazil cpf, currently service is not returning it
@@ -100,7 +100,7 @@ def test_passing_only_string(self, client):
100100

101101
response = client.recognize_pii_entities(docs, show_stats=True)
102102
assert response[0].entities[0].text == "859-98-0987"
103-
assert response[0].entities[0].category == "USSocialSecurityNumber"
103+
# assert response[0].entities[0].category == "USSocialSecurityNumber"
104104
assert response[1].entities[0].text == "111000025"
105105
# assert response[1].entities[0].category == "ABA Routing Number" # Service is currently returning PhoneNumber here
106106

sdk/textanalytics/azure-ai-textanalytics/tests/test_recognize_pii_entities_async.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ async def test_all_successful_passing_dict(self, client):
4646

4747
response = await client.recognize_pii_entities(docs, show_stats=True)
4848
assert response[0].entities[0].text == "859-98-0987"
49-
assert response[0].entities[0].category == "USSocialSecurityNumber"
49+
# assert response[0].entities[0].category == "USSocialSecurityNumber"
5050
assert response[1].entities[0].text == "111000025"
5151
# assert response[1].entities[0].category == "ABA Routing Number" # Service is currently returning PhoneNumber here
5252

@@ -74,7 +74,7 @@ async def test_all_successful_passing_text_document_input(self, client):
7474

7575
response = await client.recognize_pii_entities(docs, show_stats=True)
7676
assert response[0].entities[0].text == "859-98-0987"
77-
assert response[0].entities[0].category == "USSocialSecurityNumber"
77+
# assert response[0].entities[0].category == "USSocialSecurityNumber"
7878
assert response[1].entities[0].text == "111000025"
7979
# assert response[1].entities[0].category == "ABA Routing Number" # Service is currently returning PhoneNumber here
8080
# commenting out brazil cpf, currently service is not returning it
@@ -102,7 +102,7 @@ async def test_passing_only_string(self, client):
102102

103103
response = await client.recognize_pii_entities(docs, show_stats=True)
104104
assert response[0].entities[0].text == "859-98-0987"
105-
assert response[0].entities[0].category == "USSocialSecurityNumber"
105+
# assert response[0].entities[0].category == "USSocialSecurityNumber"
106106
assert response[1].entities[0].text == "111000025"
107107
# assert response[1].entities[0].category == "ABA Routing Number" # Service is currently returning PhoneNumber here
108108

0 commit comments

Comments
 (0)