Skip to content

Conversation

mw-kapilg
Copy link
Member

@mw-kapilg mw-kapilg commented Sep 11, 2025

Only for testing Tarun's changes

@taguptamw taguptamw changed the title draft PR Add Code Coverage, Model Coverage and Test results report for HTML Oct 9, 2025
@taguptamw taguptamw requested review from davidbuzinski, mcafaro, mw-hrastega and sameagen-MW and removed request for davidbuzinski October 9, 2025 08:34
@taguptamw taguptamw marked this pull request as ready for review October 9, 2025 08:37
Copy link
Member

@sameagen-MW sameagen-MW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

:shipit:

displayName: Run MATLAB tests producing artifacts
inputs:
testResultsJUnit: test-results/matlab/results.xml
testResultsHTML: test-results/matlab/resultsHTML
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as GHA, this is just the folder-based HTML?

Comment on lines -276 to +290
grep -q "TestRunner.withTextOutput('OutputDetail', 3)" console.log
grep -q "('OutputDetail', 3)" console.log
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this due to a change to genscript?

{
"name": "testResultsHTML",
"type": "string",
"label": "HTML test report ",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems potentially misleading if this is the folder-based html report, since when I see report I'd probably expect a file rather than a directory.

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.

5 participants