Conversation
sameagen-MW
left a comment
sameagen-MW
left a comment
There was a problem hiding this comment.
Looks good! Do we have any test coverage for the summary showing up if the user uses runMATLABTests?
| @Test | ||
| public void verifyTestResultsSummary() throws Exception { | ||
| FreeStyleProject project = jenkins.createFreeStyleProject(); | ||
| project.setScm(new ExtractResourceSCM(Utilities.getRunMATLABTestsData())); |
There was a problem hiding this comment.
This is the existing system test utilities right?
There was a problem hiding this comment.
Yes, 'Utilities' is mainly used for system tests. I updated the method name to 'getURLForTestData' as it is more apt.
src/test/java/com/mathworks/ci/systemtests/TestResultVisualizationIT.java
Outdated
Show resolved
Hide resolved
src/test/java/com/mathworks/ci/systemtests/TestResultVisualizationIT.java
Outdated
Show resolved
Hide resolved
|
|
||
| Combination c = new Combination(new AxisList(new MatlabInstallationAxis(Arrays.asList("MATLAB_PATH_1"))), "MATLAB_PATH_1"); | ||
| MatrixRun run = build.getRun(c); | ||
| String[] buildResultSummary= getTestResultSummaryFromBuildStatusPage(run); |
There was a problem hiding this comment.
| String[] buildResultSummary= getTestResultSummaryFromBuildStatusPage(run); | |
| String[] buildResultSummaries = getTestResultSummaryFromBuildStatusPage(run); |
src/test/java/com/mathworks/ci/systemtests/TestResultVisualizationIT.java
Outdated
Show resolved
Hide resolved
Run MATLAB Tests is not supported yet. Once those source changes are in, I will add the tests for that. |
|
I have raised another PR under jenkinsci/matlab-plugin repo for same changes. Closing this PR, review can be continued from new PR. |
2 participants
Adding system tests for Test Result Visualization