Skip to content

Commit

Permalink
Set Merge Test Results to true by default (#20718)
Browse files Browse the repository at this point in the history
* merge test results to true.

* revert package lock

* revert make options.

* Update package-lock.json

* Update .npmrc

* version bump
  • Loading branch information
KathanS authored Dec 13, 2024
1 parent 6a4f7c4 commit 80d79fd
Show file tree
Hide file tree
Showing 15 changed files with 25 additions and 78 deletions.
3 changes: 2 additions & 1 deletion Tasks/AzureTestPlanV0/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,5 @@ export const AUTOMATED_EXECUTION = "AutomatedExecutionPhase";
export const AUTOMATED_PUBLISHING = "PublishingAutomatedResultsPhase";
export const MANUALTESTS_PUBLISHING = "ManualTestResultsPublishingPhase";
export const INSTALL_GOTESTSUM = "install gotest.tools/gotestsum@latest";
export const INSTALL_JESTJUNIT = "i jest-junit";
export const INSTALL_JESTJUNIT = "i jest-junit";
export const MERGE_TEST_RESULTS = true;
2 changes: 1 addition & 1 deletion Tasks/AzureTestPlanV0/publishAutomatedTests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export async function publishAutomatedTestResult(listOfAutomatedTestPoints: stri
try{
const testRunner = "JUnit";
const testResultsFiles: string[] = ["**/TEST-*.xml"];
const mergeResults = tl.getInput('mergeTestResults');
const mergeResults = constants.MERGE_TEST_RESULTS;
const platform = "any cpu";
const publishRunAttachments = tl.getInput('publishRunAttachments');
const failTaskOnFailedTests = tl.getInput('failTaskOnFailedTests');
Expand Down
10 changes: 1 addition & 9 deletions Tasks/AzureTestPlanV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"version": {
"Major": 0,
"Minor": 250,
"Patch": 4
"Patch": 6
},
"preview": true,
"demands": [],
Expand Down Expand Up @@ -102,14 +102,6 @@
"helpMarkDown": "Relative path from the repository root to the build.gradle file.",
"visibleRule": "testLanguageInput = JavaGradle"
},
{
"name": "mergeTestResults",
"type": "boolean",
"label": "Merge test results",
"defaultValue": "false",
"required": false,
"helpMarkDown": "A test run is created for each results file. Check this option to merge results into a single test run. To optimize for better performance, results will be merged into a single run if there are more than 100 result files, irrespective of this option."
},
{
"name": "publishRunAttachments",
"type": "boolean",
Expand Down
14 changes: 1 addition & 13 deletions Tasks/AzureTestPlanV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"version": {
"Major": 0,
"Minor": 250,
"Patch": 4
"Patch": 6
},
"preview": true,
"demands": [],
Expand Down Expand Up @@ -102,14 +102,6 @@
"helpMarkDown": "ms-resource:loc.input.help.gradleFilePath",
"visibleRule": "testLanguageInput = JavaGradle"
},
{
"name": "mergeTestResults",
"type": "boolean",
"label": "ms-resource:loc.input.label.mergeTestResults",
"defaultValue": "false",
"required": false,
"helpMarkDown": "ms-resource:loc.input.help.mergeTestResults"
},
{
"name": "publishRunAttachments",
"type": "boolean",
Expand Down Expand Up @@ -184,9 +176,5 @@
"chmodGradlew": "ms-resource:loc.messages.chmodGradlew",
"NoMatchingFilesFound": "ms-resource:loc.messages.NoMatchingFilesFound",
"MultipleMatchingGradlewFound": "ms-resource:loc.messages.MultipleMatchingGradlewFound"
},
"_buildConfigMapping": {
"Default": "0.238.4",
"Node20-225": "0.238.5"
}
}
4 changes: 2 additions & 2 deletions _generated/AzureTestPlanV0.versionmap.txt
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Default|0.250.4
Node20-225|0.250.5
Default|0.250.6
Node20-225|0.250.7
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@
"loc.input.help.pomFilePath": "Relative path from the repository root to the Maven POM file.",
"loc.input.label.gradleFilePath": "Gradle file path",
"loc.input.help.gradleFilePath": "Relative path from the repository root to the build.gradle file.",
"loc.input.label.mergeTestResults": "Merge test results",
"loc.input.help.mergeTestResults": "A test run is created for each results file. Check this option to merge results into a single test run. To optimize for better performance, results will be merged into a single run if there are more than 100 result files, irrespective of this option.",
"loc.input.label.publishRunAttachments": "Upload test results files",
"loc.input.help.publishRunAttachments": "Upload logs and other files containing diagnostic information collected when the tests were run.",
"loc.input.label.failTaskOnFailedTests": "Fail if there are test failures",
Expand Down
3 changes: 2 additions & 1 deletion _generated/AzureTestPlanV0/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,5 @@ export const AUTOMATED_EXECUTION = "AutomatedExecutionPhase";
export const AUTOMATED_PUBLISHING = "PublishingAutomatedResultsPhase";
export const MANUALTESTS_PUBLISHING = "ManualTestResultsPublishingPhase";
export const INSTALL_GOTESTSUM = "install gotest.tools/gotestsum@latest";
export const INSTALL_JESTJUNIT = "i jest-junit";
export const INSTALL_JESTJUNIT = "i jest-junit";
export const MERGE_TEST_RESULTS = true;
2 changes: 1 addition & 1 deletion _generated/AzureTestPlanV0/publishAutomatedTests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export async function publishAutomatedTestResult(listOfAutomatedTestPoints: stri
try{
const testRunner = "JUnit";
const testResultsFiles: string[] = ["**/TEST-*.xml"];
const mergeResults = tl.getInput('mergeTestResults');
const mergeResults = constants.MERGE_TEST_RESULTS;
const platform = "any cpu";
const publishRunAttachments = tl.getInput('publishRunAttachments');
const failTaskOnFailedTests = tl.getInput('failTaskOnFailedTests');
Expand Down
14 changes: 3 additions & 11 deletions _generated/AzureTestPlanV0/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"version": {
"Major": 0,
"Minor": 250,
"Patch": 4
"Patch": 6
},
"preview": true,
"demands": [],
Expand Down Expand Up @@ -102,14 +102,6 @@
"helpMarkDown": "Relative path from the repository root to the build.gradle file.",
"visibleRule": "testLanguageInput = JavaGradle"
},
{
"name": "mergeTestResults",
"type": "boolean",
"label": "Merge test results",
"defaultValue": "false",
"required": false,
"helpMarkDown": "A test run is created for each results file. Check this option to merge results into a single test run. To optimize for better performance, results will be merged into a single run if there are more than 100 result files, irrespective of this option."
},
{
"name": "publishRunAttachments",
"type": "boolean",
Expand Down Expand Up @@ -186,8 +178,8 @@
"MultipleMatchingGradlewFound": "Multiple gradlew files found. Selecting the first matched instance"
},
"_buildConfigMapping": {
"Default": "0.250.4",
"Default": "0.250.6",
"LocalPackages": "0.249.4",
"Node20-225": "0.250.5"
"Node20-225": "0.250.7"
}
}
14 changes: 3 additions & 11 deletions _generated/AzureTestPlanV0/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"version": {
"Major": 0,
"Minor": 250,
"Patch": 4
"Patch": 6
},
"preview": true,
"demands": [],
Expand Down Expand Up @@ -102,14 +102,6 @@
"helpMarkDown": "ms-resource:loc.input.help.gradleFilePath",
"visibleRule": "testLanguageInput = JavaGradle"
},
{
"name": "mergeTestResults",
"type": "boolean",
"label": "ms-resource:loc.input.label.mergeTestResults",
"defaultValue": "false",
"required": false,
"helpMarkDown": "ms-resource:loc.input.help.mergeTestResults"
},
{
"name": "publishRunAttachments",
"type": "boolean",
Expand Down Expand Up @@ -186,8 +178,8 @@
"MultipleMatchingGradlewFound": "ms-resource:loc.messages.MultipleMatchingGradlewFound"
},
"_buildConfigMapping": {
"Default": "0.250.4",
"Default": "0.250.6",
"LocalPackages": "0.249.4",
"Node20-225": "0.250.5"
"Node20-225": "0.250.7"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@
"loc.input.help.pomFilePath": "Relative path from the repository root to the Maven POM file.",
"loc.input.label.gradleFilePath": "Gradle file path",
"loc.input.help.gradleFilePath": "Relative path from the repository root to the build.gradle file.",
"loc.input.label.mergeTestResults": "Merge test results",
"loc.input.help.mergeTestResults": "A test run is created for each results file. Check this option to merge results into a single test run. To optimize for better performance, results will be merged into a single run if there are more than 100 result files, irrespective of this option.",
"loc.input.label.publishRunAttachments": "Upload test results files",
"loc.input.help.publishRunAttachments": "Upload logs and other files containing diagnostic information collected when the tests were run.",
"loc.input.label.failTaskOnFailedTests": "Fail if there are test failures",
Expand Down
3 changes: 2 additions & 1 deletion _generated/AzureTestPlanV0_Node20/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,4 +13,5 @@ export const AUTOMATED_EXECUTION = "AutomatedExecutionPhase";
export const AUTOMATED_PUBLISHING = "PublishingAutomatedResultsPhase";
export const MANUALTESTS_PUBLISHING = "ManualTestResultsPublishingPhase";
export const INSTALL_GOTESTSUM = "install gotest.tools/gotestsum@latest";
export const INSTALL_JESTJUNIT = "i jest-junit";
export const INSTALL_JESTJUNIT = "i jest-junit";
export const MERGE_TEST_RESULTS = true;
2 changes: 1 addition & 1 deletion _generated/AzureTestPlanV0_Node20/publishAutomatedTests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export async function publishAutomatedTestResult(listOfAutomatedTestPoints: stri
try{
const testRunner = "JUnit";
const testResultsFiles: string[] = ["**/TEST-*.xml"];
const mergeResults = tl.getInput('mergeTestResults');
const mergeResults = constants.MERGE_TEST_RESULTS;
const platform = "any cpu";
const publishRunAttachments = tl.getInput('publishRunAttachments');
const failTaskOnFailedTests = tl.getInput('failTaskOnFailedTests');
Expand Down
14 changes: 3 additions & 11 deletions _generated/AzureTestPlanV0_Node20/task.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"version": {
"Major": 0,
"Minor": 250,
"Patch": 5
"Patch": 7
},
"preview": true,
"demands": [],
Expand Down Expand Up @@ -102,14 +102,6 @@
"helpMarkDown": "Relative path from the repository root to the build.gradle file.",
"visibleRule": "testLanguageInput = JavaGradle"
},
{
"name": "mergeTestResults",
"type": "boolean",
"label": "Merge test results",
"defaultValue": "false",
"required": false,
"helpMarkDown": "A test run is created for each results file. Check this option to merge results into a single test run. To optimize for better performance, results will be merged into a single run if there are more than 100 result files, irrespective of this option."
},
{
"name": "publishRunAttachments",
"type": "boolean",
Expand Down Expand Up @@ -190,8 +182,8 @@
"MultipleMatchingGradlewFound": "Multiple gradlew files found. Selecting the first matched instance"
},
"_buildConfigMapping": {
"Default": "0.250.4",
"Default": "0.250.6",
"LocalPackages": "0.249.4",
"Node20-225": "0.250.5"
"Node20-225": "0.250.7"
}
}
14 changes: 3 additions & 11 deletions _generated/AzureTestPlanV0_Node20/task.loc.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
"version": {
"Major": 0,
"Minor": 250,
"Patch": 5
"Patch": 7
},
"preview": true,
"demands": [],
Expand Down Expand Up @@ -102,14 +102,6 @@
"helpMarkDown": "ms-resource:loc.input.help.gradleFilePath",
"visibleRule": "testLanguageInput = JavaGradle"
},
{
"name": "mergeTestResults",
"type": "boolean",
"label": "ms-resource:loc.input.label.mergeTestResults",
"defaultValue": "false",
"required": false,
"helpMarkDown": "ms-resource:loc.input.help.mergeTestResults"
},
{
"name": "publishRunAttachments",
"type": "boolean",
Expand Down Expand Up @@ -190,8 +182,8 @@
"MultipleMatchingGradlewFound": "ms-resource:loc.messages.MultipleMatchingGradlewFound"
},
"_buildConfigMapping": {
"Default": "0.250.4",
"Default": "0.250.6",
"LocalPackages": "0.249.4",
"Node20-225": "0.250.5"
"Node20-225": "0.250.7"
}
}

0 comments on commit 80d79fd

Please sign in to comment.