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.

Sign up for GitHub

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

Jump to bottom

[BUG] - Fix AnalyzeOperationBatchWithStatisticsTest in TextAnalytics #16839

Closed
suhas92 opened this issue Nov 10, 2020 · 3 comments
Closed

[BUG] - Fix AnalyzeOperationBatchWithStatisticsTest in TextAnalytics #16839

suhas92 opened this issue Nov 10, 2020 · 3 comments
Assignees
@a-noyass
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics test-reliability Issue that causes tests to be unreliable
Milestone
[2021] March

Comments

@suhas92
Copy link
Member

suhas92 commented Nov 10, 2020

Describe the bug
We have a test ignored due to a bug on the TA service. The statistics are not showing up in the response.

Expected behavior
What is the expected behavior?
The test should not be ignored

Actual behavior (include Exception or Stack Trace)
What is the actual behavior?
The test is ignored
#16785 (comment)

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Environment:

  • Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.2.0]
  • Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): [e.g. Azure AppService or Windows 10 .NET Framework 4.8]
  • IDE and version : [e.g. Visual Studio 16.3]
@ghost ghost added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 10, 2020
@jsquire jsquire added Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics needs-team-triage Workflow: This issue needs the team to triage. test-reliability Issue that causes tests to be unreliable labels Nov 10, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Nov 10, 2020
@jsquire
Copy link
Member

jsquire commented Nov 10, 2020

//cc: @maririos

@maririos maririos removed the needs-team-triage Workflow: This issue needs the team to triage. label Nov 10, 2020
@maririos maririos added this to the [2021] February milestone Nov 10, 2020
@maririos maririos modified the milestones: [2021] February, [2021] March Feb 3, 2021
@maririos
Copy link
Member

maririos commented Feb 4, 2021

Stats are now returned from the service. Check and enable test https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/textanalytics/Azure.AI.TextAnalytics/tests/AnalyzeOperationTests.cs#L370

@maririos maririos assigned a-noyass and unassigned suhas92 Feb 10, 2021
@maririos
Copy link
Member

Fixed in #18516

@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@a-noyass a-noyass

Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Text Analytics test-reliability Issue that causes tests to be unreliable
Projects
None yet
Milestone
[2021] March
Development

No branches or pull requests
4 participants
@jsquire @maririos @suhas92 @a-noyass