Skip to content

Commit

Permalink
parameters, not variables for devops api
Browse files Browse the repository at this point in the history
  • Loading branch information
outofambit committed Jan 7, 2025
1 parent a37fd42 commit 8cf9313
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions server/services/AzurePipelinesService.js
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ const triggerAzurePipeline = async ({ job, directory, gitSha, atVersion }) => {
}

const browser = job.testPlanRun.testPlanReport.browser.name.toLowerCase();
const variables = {
const parameters = {
callback_url: `https://${callbackUrlHostname}/api/jobs/${job.id}/test/:testRowNumber`,
status_url: `https://${callbackUrlHostname}/api/jobs/${job.id}`,
callback_header: `x-automation-secret:${job.secret}`,
Expand All @@ -98,22 +98,22 @@ const triggerAzurePipeline = async ({ job, directory, gitSha, atVersion }) => {
};

if (atKey === 'nvda') {
variables.test_pattern = '{reference/**,test-*-nvda.*}';
variables.nvda_version = atVersion?.name;
parameters.test_pattern = '{reference/**,test-*-nvda.*}';
parameters.nvda_version = atVersion?.name;
}

// Mac unsupported for now
// if (atKey === 'voiceover_macos') {
// variables.macos_version = atVersion?.name?.split('.')[0];
// parameters.macos_version = atVersion?.name?.split('.')[0];
// }

const definition = {
definition: {
id: pipelineId
},
sourceBranch: `refs/heads/main`,
variables: Object.fromEntries(
Object.entries(variables).map(([key, value]) => [key, { value }])
parameters: Object.fromEntries(
Object.entries(parameters).map(([key, value]) => [key, value])
)
};

Expand Down

0 comments on commit 8cf9313

Please sign in to comment.