Skip to content

Add self-validation to integration tests#36

Merged
Malcolmnixon merged 3 commits intomainfrom
copilot/add-sonarmark-self-validation
Jan 21, 2026
Merged

Add self-validation to integration tests#36
Malcolmnixon merged 3 commits intomainfrom
copilot/add-sonarmark-self-validation

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

Integration tests now run SonarMark's built-in self-validation suite and capture TRX results for each OS/runtime combination.

Changes

  • New step: Run SonarMark self-validation - Executes sonarmark --validate --results integration-test-${{ matrix.os }}-dotnet${{ matrix.dotnet-version }}.trx to generate test results and report pass/fail to console
  • New step: Upload integration test results - Uploads TRX files as artifacts named integration-test-results-${{ matrix.os }}-dotnet${{ matrix.dotnet-version }} with if: always() to capture results even on failure

Self-validation tests verify quality gate status retrieval, issues retrieval, hot-spots retrieval, and markdown report generation using mock data.

Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature]: Run Self-Validation</issue_title>
<issue_description>### Problem Statement

The integration tests should include running the SonarMark self-validation and saving the test results

Proposed Solution

Modify the integration-test job of build.yaml to include two extra steps:

  1. Run the SonarMark self-validation to generate integration-test-${{ matrix.os }}-dotnet${{ matrix.dotnet-version }}.trx and to report the pass/fail to the console
  2. Upload the TRX results file to an artifact called integration-test-results-${{ matrix.os }}-dotnet${{ matrix.dotnet-version }}

Alternatives Considered

No response

Additional Context

No response

Priority

High - Blocking current work

Contribution

  • I would be willing to submit a PR for this feature

Code of Conduct

  • I agree to follow this project's Code of Conduct</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits January 21, 2026 20:52
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Add self-validation step to integration tests Add self-validation to integration tests Jan 21, 2026
Copilot AI requested a review from Malcolmnixon January 21, 2026 20:54
@Malcolmnixon Malcolmnixon marked this pull request as ready for review January 21, 2026 21:16
@Malcolmnixon Malcolmnixon merged commit 974a0ab into main Jan 21, 2026
9 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/add-sonarmark-self-validation branch January 21, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Run Self-Validation

2 participants