Add SelectByName parameter #148
Conversation
Vahila
requested review from
davidbuzinski,
mcafaro and
sameagen-MW
and removed request for
davidbuzinski,
mcafaro and
sameagen-MW
| "defaultValue": "", | ||
| "groupName": "filterTests", | ||
| "required": false, | ||
| "helpMarkDown": "Test name used to select test suite elements. To create a test suite, the task uses only the test elements with the specified name." |
There was a problem hiding this comment.
Thanks for adding this text, Vahila.
@davidbuzinski: Where would such text show up? In the UI?
There was a problem hiding this comment.
It shows up if you try to edit the yaml in the pipeline editor at dev.azure.com. There is a list of all the input fields, and each field has a info icon that you can hover over to see this help text.
| set -e | ||
| grep -q StartupTest test-results/matlab/selectbyname.xml | ||
| grep -q simpleTest test-results/matlab/selectbyname.xml | ||
| ! grep -q FirstTest test-results/matlab/selectbytag.xml |
There was a problem hiding this comment.
Should this be ! grep -q FirstTest test-results/matlab/selectbytag.xml?
| "defaultValue": "", | ||
| "groupName": "filterTests", | ||
| "required": false, | ||
| "helpMarkDown": "Test name used to select test suite elements. To create a test suite, the task uses only the test elements with the specified name." |
There was a problem hiding this comment.
It shows up if you try to edit the yaml in the pipeline editor at dev.azure.com. There is a list of all the input fields, and each field has a info icon that you can hover over to see this help text.
| if (!input || !input.trim()) { | ||
| return "{}"; | ||
| } | ||
| const items = input.split(/\s+/).filter(Boolean).map((s) => `'${s}'`); |
There was a problem hiding this comment.
The .filter(Boolean) took me a second. It looks like this is to remove extra spaces (ex: myTest myTest2). We could consider a test that includes extra spaces in the input.
Also, is it possible for test names to have spaces in them? How would a user specify such a test?
| } | ||
|
|
||
| // Function to convert space separated names to cell array of character vectors | ||
| export function getSelectByNameAsCellArray(input?: string): string { |
There was a problem hiding this comment.
Hopefully this is just temporary until scriptgen gets updated to properly accept SelectByName as a space separated string?
While unlikely, single-quotes in the strings need to be escaped or it may result in a malformed cell array.
| SourceFolder: taskLib.getInput("sourceFolder"), | ||
| SelectByFolder: taskLib.getInput("selectByFolder"), | ||
| SelectByTag: taskLib.getInput("selectByTag"), | ||
| SelectByName: taskLib.getInput("SelectByName"), |
There was a problem hiding this comment.
| SelectByName: taskLib.getInput("SelectByName"), | |
| SelectByName: taskLib.getInput("selectByName"), |
| `'CoberturaModelCoverage','${options.CoberturaModelCoverage || ""}',` + | ||
| `'SelectByTag','${options.SelectByTag || ""}',` + | ||
| `'SelectByFolder','${options.SelectByFolder || ""}',` + | ||
| `'SelectByName',${getSelectByNameAsCellArray(options.SelectByName)},` + |
There was a problem hiding this comment.
Is there a plan to go back and fix scriptgen so this can be:
'SelectByName','${options.SelectByName || ""}', ?
| if (!input || !input.trim()) { | ||
| return "{}"; | ||
| } | ||
| const items = input.split(/\s+/).filter(Boolean).map((s) => `'${s}'`); |
There was a problem hiding this comment.
Single quotes and curly braces need to be escaped
4 participants
Added new 'SelectByName' parameter. This would help in supporting Parallel Strategy in Azure.
Note: Genscript changes to support SelectByName needs to be available before these can be merged.