From 9d2a568a901fcbb101d86aa92581f10fddcb11b7 Mon Sep 17 00:00:00 2001 From: chris48s Date: Sun, 11 Feb 2024 13:03:37 +0000 Subject: [PATCH] tolerate missing short_version in [visualstudioappcenter] (#9951) --- ...-studio-app-center-releases-version.service.js | 5 ++++- ...l-studio-app-center-releases-version.tester.js | 15 +++++++++++++++ 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/services/visual-studio-app-center/visual-studio-app-center-releases-version.service.js b/services/visual-studio-app-center/visual-studio-app-center-releases-version.service.js index 18c7fcfee5152..fe25c0bb7aff3 100644 --- a/services/visual-studio-app-center/visual-studio-app-center-releases-version.service.js +++ b/services/visual-studio-app-center/visual-studio-app-center-releases-version.service.js @@ -8,7 +8,7 @@ import { const schema = Joi.object({ version: Joi.string().required(), - short_version: Joi.string().required(), + short_version: Joi.string().required().allow(''), }).required() export default class VisualStudioAppCenterReleasesVersion extends BaseVisualStudioAppCenterService { @@ -53,6 +53,9 @@ export default class VisualStudioAppCenterReleasesVersion extends BaseVisualStud token, schema, }) + if (!shortVersion) { + return renderVersionBadge({ version }) + } return renderVersionBadge({ version: `${shortVersion} (${version})`, }) diff --git a/services/visual-studio-app-center/visual-studio-app-center-releases-version.tester.js b/services/visual-studio-app-center/visual-studio-app-center-releases-version.tester.js index e1afba88ff6b0..b6564bfbfc09b 100644 --- a/services/visual-studio-app-center/visual-studio-app-center-releases-version.tester.js +++ b/services/visual-studio-app-center/visual-studio-app-center-releases-version.tester.js @@ -29,6 +29,21 @@ t.create('Invalid user, invalid project, valid API token') message: 'project not found', }) +t.create('Missing Short Version') + .get('/nock/nock/nock.json') + .intercept(nock => + nock('https://api.appcenter.ms/v0.1/apps/') + .get('/nock/nock/releases/latest') + .reply(200, { + version: '1.0', + short_version: '', + }), + ) + .expectBadge({ + label: 'release', + message: 'v1.0', + }) + t.create('Invalid API Token').get('/invalid/invalid/invalid.json').expectBadge({ label: 'release', message: 'invalid token',