Azure-Samples / cognitive-services-speech-sdk

Sample code for the Microsoft Cognitive Services Speech SDK
MIT License
2.92k stars 1.85k forks source link

[IngestionClient] Add a unit test coverage build. #2447

Closed superhindupur closed 1 month ago

superhindupur commented 4 months ago

Is your feature request related to a problem? Please describe. There is no check currently that safeguards the amount of unit test coverage in IngestionClient. This could result in new code being added without safeguarding it with unit tests.

Describe the solution you'd like A unit test coverage check added as part of the CI/CD build for IngestionClient. Ideally this would run on every PR created against the main branch, and on merges to the main branch.

github-actions[bot] commented 3 months ago

This item has been open without activity for 19 days. Provide a comment on status and remove "update needed" label.

pankopon commented 2 months ago

@HenryvanderVegte There are several "[IngestionClient]" items open, are these actual customer issues or more general backlog work items? Is there a plan for them? Please update.

HenryvanderVegte commented 1 month ago

As discussed in https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2521, moving item to internal backlog. Not an actual customer issue, but general backlog work item.