remove unused commmented out tests (Azure#22226)

Author: kristapratico

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

@@ -461,75 +461,6 @@ def test_poller_metadata(self, client):
         assert poller.total_actions_count == 1
         assert poller.id
 
-    ### TODO: Commenting out language tests. Right now analyze only supports language 'en', so no point to these tests yet
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    @recorded_by_proxy
-    # def test_whole_batch_language_hint(self, client):
-    #     def callback(resp):
-    #         language_str = "\"language\": \"fr\""
-    #         if resp.http_request.body:
-    #             language = resp.http_request.body.count(language_str)
-    #             assert language == 3
-
-    #     docs = [
-    #         u"This was the best day of my life.",
-    #         u"I did not like the hotel we stayed at. It was too expensive.",
-    #         u"The restaurant was not as good as I hoped."
-    #     ]
-
-    #     response = list(client.begin_analyze_actions(
-    #         docs,
-    #         actions=[
-    #             RecognizeEntitiesAction(),
-    #             ExtractKeyPhrasesAction(),
-    #             RecognizePiiEntitiesAction()
-    #         ],
-    #         language="fr",
-    #         polling_interval=self._interval(),
-    #         raw_response_hook=callback
-    #     ).result())
-
-    #     for document_result in response:
-    #         for doc in document_result.document_results:
-    #             assert not doc.is_error
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer(client_kwargs={
-    #     "default_language": "en"
-    # })
-    # def test_whole_batch_language_hint_and_obj_per_item_hints(self, client):
-    #     def callback(resp):
-    #         pass
-    #         # if resp.http_request.body:
-    #         #     language_str = "\"language\": \"es\""
-    #         #     language = resp.http_request.body.count(language_str)
-    #         #     assert language == 2
-    #         #     language_str = "\"language\": \"en\""
-    #         #     language = resp.http_request.body.count(language_str)
-    #         #     assert language == 1
-
-    #     docs = [
-    #         TextDocumentInput(id="1", text="I should take my cat to the veterinarian.", language="es"),
-    #         TextDocumentInput(id="2", text="Este es un document escrito en Español.", language="es"),
-    #         TextDocumentInput(id="3", text="猫は幸せ"),
-    #     ]
-
-    #     response = list(client.begin_analyze_actions(
-    #         docs,
-    #         actions=[
-    #             RecognizeEntitiesAction(),
-    #             ExtractKeyPhrasesAction(),
-    #             RecognizePiiEntitiesAction()
-    #         ],
-    #         polling_interval=self._interval(),
-    #     ).result())
-
-    #     for document_result in response:
-    #         for doc in document_result.document_results:
-    #             assert not doc.is_error
-
     @TextAnalyticsPreparer()
     @TextAnalyticsClientPreparer()
     @recorded_by_proxy

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

@@ -521,98 +521,6 @@ async def test_poller_metadata(self, client):
             assert poller.total_actions_count == 1
             assert poller.id
 
-    ### TODO: Commenting out language tests. Right now analyze only supports language 'en', so no point to these tests yet
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    # @recorded_by_proxy_async
-    # async def test_whole_batch_language_hint(self, client):
-    #     def callback(resp):
-    #         language_str = "\"language\": \"fr\""
-    #         if resp.http_request.body:
-    #             language = resp.http_request.body.count(language_str)
-    #             assert language == 3
-
-    #     docs = [
-    #         u"This was the best day of my life.",
-    #         u"I did not like the hotel we stayed at. It was too expensive.",
-    #         u"The restaurant was not as good as I hoped."
-    #     ]
-
-    #     async with client:
-    #         response = await (await client.begin_analyze_actions(
-    #             docs,
-    #             actions=[
-    #                 RecognizeEntitiesAction(),
-    #                 ExtractKeyPhrasesAction(),
-    #                 RecognizePiiEntitiesAction()
-    #             ],
-    #             language="fr",
-    #             polling_interval=self._interval(),
-    #             raw_response_hook=callback
-    #         )).result()
-
-    #         async for action_result in response:
-    #             for doc in action_result.document_results:
-    #                 assert not doc.is_error
-
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer(client_kwargs={
-    #     "default_language": "en"
-    # })
-    # async def test_whole_batch_language_hint_and_obj_per_item_hints(self, client):
-    #     def callback(resp):
-    #         if resp.http_request.body:
-    #             language_str = "\"language\": \"es\""
-    #             language = resp.http_request.body.count(language_str)
-    #             assert language == 2
-    #             language_str = "\"language\": \"en\""
-    #             language = resp.http_request.body.count(language_str)
-    #             assert language == 1
-
-    #     docs = [
-    #         TextDocumentInput(id="1", text="I should take my cat to the veterinarian.", language="es"),
-    #         TextDocumentInput(id="2", text="Este es un document escrito en Español.", language="es"),
-    #         TextDocumentInput(id="3", text="猫は幸せ"),
-    #     ]
-
-    #     async with client:
-    #         response = await (await client.begin_analyze_actions(
-    #             docs,
-    #             actions=[
-    #                 RecognizeEntitiesAction(),
-    #                 ExtractKeyPhrasesAction(),
-    #                 RecognizePiiEntitiesAction()
-    #             ],
-    #             language="en",
-    #             polling_interval=self._interval()
-    #         )).result()
-
-    #         async for action_result in response:
-    #             for doc in action_result.document_results:
-    #                 assert not doc.is_error
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    # @recorded_by_proxy_async
-    # async def test_invalid_language_hint_method(self, client):
-    #     async with client:
-    #         response = await (await client.begin_analyze_actions(
-    #             ["This should fail because we're passing in an invalid language hint"],
-    #             language="notalanguage",
-    #             actions=[
-    #                 RecognizeEntitiesAction(),
-    #                 ExtractKeyPhrasesAction(),
-    #                 RecognizePiiEntitiesAction()
-    #             ],
-    #             polling_interval=self._interval()
-    #         )).result()
-
-    #         async for action_result in response:
-    #             for doc in action_result.document_results:
-    #                 assert doc.is_error
-
     @TextAnalyticsPreparer()
     @TextAnalyticsClientPreparer()
     @recorded_by_proxy_async

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

@@ -291,56 +291,6 @@ def callback(pipeline_response, deserialized, _):
         ).result()
         assert res == "cls result"
 
-    """Commenting out multi page tests until service returns multiple pages"""
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    # @recorded_by_proxy
-    # def test_multiple_pages_of_results_returned_successfully(self, client):
-    #     single_doc = "hello world"
-    #     docs = [{"id": str(idx), "text": val} for (idx, val) in enumerate(list(itertools.repeat(single_doc, 10)))]
-    #     # Service now only accepts 10 documents for a job, and since the current default server-side value
-    #     # for records per page is 20, pagination logic will never be activated.  This is intended to change
-    #     # in the future but for now this test actually won't hit the pagination logic now.
-
-    #     result = client.begin_analyze_healthcare_entities(docs, show_stats=True, polling_interval=self._interval()).result()
-    #     response = list(result)
-
-    #     assert len(docs) == len(response)
-    #     assert result.statistics is not None
-
-    #     for (idx, doc) in enumerate(response):
-    #         assert docs[idx]["id"] == doc.id
-    #         assert doc.statistics is not None
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    # @recorded_by_proxy
-    # def test_multiple_pages_of_results_with_errors_returned_successfully(self, client):
-    #     single_doc = "hello world"
-    #     docs = [{"id": str(idx), "text": val} for (idx, val) in enumerate(list(itertools.repeat(single_doc, 9)))]
-    #     docs.append({"id": "9", "text": ""})
-    #     # Service now only accepts 10 documents for a job, and since the current default server-side value
-    #     # for records per page is 20, pagination logic will never be activated.  This is intended to change
-    #     # in the future but for now this test actually won't hit the pagination logic now.
-
-
-    #     result = client.begin_analyze_healthcare_entities(docs, show_stats=True, polling_interval=self._interval()).result()
-    #     response = list(result)
-
-    #     assert len(docs) == len(response)
-    #     assert result.statistics is not None
-
-    #     for (idx, doc) in enumerate(response):
-    #         assert docs[idx]["id"] == doc.id
-
-    #         if doc.id == "9":
-    #             assert doc.is_error
-
-    #         else:
-    #             assert not doc.is_error
-    #             assert doc.statistics is not None
-
     @TextAnalyticsPreparer()
     @TextAnalyticsClientPreparer()
     @recorded_by_proxy

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

@@ -317,61 +317,6 @@ def callback(pipeline_response, deserialized, _):
             )).result()
         assert res == "cls result"
 
-    """Commenting out multi page tests until service returns multiple pages"""
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    # @recorded_by_proxy_async
-    # async def test_multiple_pages_of_results_returned_successfully(self, client):
-    #     single_doc = "hello world"
-    #     docs = [{"id": str(idx), "text": val} for (idx, val) in enumerate(list(itertools.repeat(single_doc, 10)))]
-    #     # Service now only accepts 10 documents for a job, and since the current default server-side value
-    #     # for records per page is 20, pagination logic will never be activated.  This is intended to change
-    #     # in the future but for now this test actually won't hit the pagination logic now.
-
-    #     async with client:
-    #         poller = await client.begin_analyze_healthcare_entities(docs, show_stats=True, polling_interval=self._interval())
-    #         result = await poller.result()
-    #         response = []
-    #         async for r in result:
-    #             response.append(r)
-
-    #     assert len(docs) == len(response)
-    #     assert result.statistics is not None
-
-    #     for (idx, doc) in enumerate(response):
-    #         assert docs[idx]["id"] == doc.id
-
-    # @TextAnalyticsPreparer()
-    # @TextAnalyticsClientPreparer()
-    # @recorded_by_proxy_async
-    # async def test_multiple_pages_of_results_with_errors_returned_successfully(self, client):
-    #     single_doc = "hello world"
-    #     docs = [{"id": str(idx), "text": val} for (idx, val) in enumerate(list(itertools.repeat(single_doc, 9)))]
-    #     docs.append({"id": "9", "text": ""})
-    #     # Service now only accepts 10 documents for a job, and since the current default server-side value
-    #     # for records per page is 20, pagination logic will never be activated.  This is intended to change
-    #     # in the future but for now this test actually won't hit the pagination logic now.
-
-    #     async with client:
-    #         result = await (await client.begin_analyze_healthcare_entities(docs, show_stats=True, polling_interval=self._interval())).result()
-    #         response = []
-    #         async for r in result:
-    #             response.append(r)
-
-    #         assert len(docs) == len(response)
-    #         assert result.statistics is not None
-
-    #         for (idx, doc) in enumerate(response):
-    #             assert docs[idx]["id"] == doc.id
-
-    #             if doc.id == "9":
-    #                 assert doc.is_error
-
-    #             else:
-    #                 assert not doc.is_error
-    #                 assert doc.statistics is not None
-
     @TextAnalyticsPreparer()
     @TextAnalyticsClientPreparer()
     @recorded_by_proxy_async
@@ -389,7 +334,6 @@ async def test_cancellation(self, client):
             except HttpResponseError:
                 pass # expected if the operation was already in a terminal state.
 
-
     @TextAnalyticsPreparer()
     @TextAnalyticsClientPreparer()
     @recorded_by_proxy_async