OPS-157454: Upload test coverage to Codacy from CI

[davidnmbond]

Adds a coverage job to CI that collects test coverage and uploads it to Codacy.

• The test project runs as a Microsoft.Testing.Platform executable (UseMicrosoftTestingPlatformRunner) with Microsoft.Testing.Extensions.CodeCoverage, writing a Cobertura report.
• coverage.config ports the generated-code exclusions used across the estate, so figures stay comparable.
• The report is uploaded with codacy/codacy-coverage-reporter-action using the CODACY_PROJECT_TOKEN secret (continue-on-error, so a Codacy outage cannot turn a passing run red), and kept as a 7-day artifact.
• Runs on the self-hosted pdl-public runners, not a GitHub-hosted runner.

Excluded tests: none - every test is offline (847 tests).

CI runs only tests that need no credentials. Tests are selected by what passes in a clean Linux container with no user secrets, exactly as CI sees it.

Local line coverage: 85.4%

Jira: https://jira.panoramicdata.com/browse/OPS-157454

🤖 Generated with Claude Code

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Coverage ∅ diff coverage

Metric	Results
Coverage variation	Report missing for 2ede4f01
Diff coverage	✅ ∅ diff coverage

View coverage diff in Codacy

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (2ede4f0)	Report Missing	Report Missing	Report Missing
Head commit (d88c935)	3022	2542	84.12%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#52)	0	0	∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

¹ Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

_{TIP This summary will be updated as you push new changes.}

[codacy-production]

Pull Request Overview

The PR successfully implements the framework for coverage reporting but contains critical pathing errors and a missing dependency file that will prevent the CI job from completing as intended. While Codacy indicates the PR is up to standards, the 'coverage' job will likely fail in its current state.

The primary issues to address are the mismatch between the generated Cobertura report location and the paths specified in the upload steps, and the inclusion of the 'coverage.config' file referenced in the workflow. There is also an opportunity to improve CI performance by removing redundant test execution.

About this PR

• The workflow is configured to use 'PanoramicData.OData.Client.Test/coverage.config' for exclusions, but this file is missing from the repository. This contradicts the acceptance criteria regarding code exclusions and will lead to job failure.

Test suggestions

• Verify the CI job executes the test binary with coverage enabled and generates the Cobertura XML file.
• Verify that coverage exclusions defined in 'coverage.config' are successfully applied during the test run.
• Verify the coverage report is successfully uploaded to Codacy.
• Verify the coverage report is archived as a build artifact with a 7-day retention period.

Prompt proposal for missing tests

Consider implementing these tests if applicable:
1. Verify that coverage exclusions defined in 'coverage.config' are successfully applied during the test run.

_{TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback}

[codacy-production]

_{🔴 HIGH RISK}

The file 'PanoramicData.OData.Client.Test/coverage.config' is missing from the PR. Please include this file to ensure the coverage run can complete successfully with the intended exclusions.

[codacy-production]

_{🔴 HIGH RISK}

The coverage report path in the upload steps is incorrect. The test runner generates the report based on the working directory (root) or the --coverage-output flag (line 86), but the upload steps (lines 96, 104) look in the project's 'bin/Debug' folder. To fix this, explicitly set the results directory and use a consistent path:

1. Add --results-directory TestResults to the test command.
2. Update the upload steps to use 'TestResults/coverage.cobertura.xml'.