[IngestionClient] Unit test coverage as part of the CI/CD pipeline

[ross-p-smith]

Purpose

This PR is actually #2503 but with the comments fixed. Original work has been carried out by @komalg1

This adds Code Coverage report into the project so we can baseline the tests

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

[ross-p-smith]

@HenryvanderVegte this PR is using the GITHUB_TOKEN which will not work on a fork, as you cannot access that Token from a fork. I believe the theory is that this workflow when run under the repo context (not fork) will have access to the token and will be able to write the Test Coverage as a comment in the PR

[HenryvanderVegte]

/azurepipelines run

[azure-pipelines]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[zhouwangzw]

/azurepipelines run

[azure-pipelines]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[yulin-li]

@ross-p-smith @HenryvanderVegte @zhouwangzw The azure pipeline build from a forked repo is disabled now for some security reasons, we are contact to ado org admin to see if we can enable it again.

[yulin-li]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[HenryvanderVegte]

@ross-p-smith @HenryvanderVegte @zhouwangzw The azure pipeline build from a forked repo is disabled now for some security reasons, we are contact to ado org admin to see if we can enable it again.

thanks a lot @yulin-li !