remove url check to support on-prem ADO

This commit is contained in:
Omer Farooq
2020-12-30 13:11:35 +13:00
parent 8e37daeb5b
commit 3f7bfe7b11
3 changed files with 13 additions and 8 deletions
@@ -47,13 +47,7 @@ export class AzureUrlReader implements UrlReader {
constructor(
private readonly options: AzureIntegrationConfig,
private readonly deps: { treeResponseFactory: ReadTreeResponseFactory },
) {
if (options.host !== 'dev.azure.com') {
throw Error(
`Azure integration currently only supports 'dev.azure.com', tried to use host '${options.host}'`,
);
}
}
) {}
async read(url: string): Promise<Buffer> {
const builtUrl = getAzureFileFetchUrl(url);
@@ -54,6 +54,18 @@ describe('azure core', () => {
result:
'https://dev.azure.com/org-name/project-name/_apis/git/repositories/repo-name/items?path=my-template.yaml',
},
{
url:
'https://api.com/org-name/project-name/_git/repo-name?path=my-template.yaml',
result:
'https://api.com/org-name/project-name/_apis/git/repositories/repo-name/items?path=my-template.yaml',
},
{
url:
'https://api.com/org-name/project-name/_git/repo-name?path=my-template.yaml&version=GBmaster',
result:
'https://api.com/org-name/project-name/_apis/git/repositories/repo-name/items?path=my-template.yaml&version=master',
},
])('should handle happy path %#', async ({ url, result }) => {
expect(getAzureFileFetchUrl(url)).toBe(result);
});
-1
View File
@@ -43,7 +43,6 @@ export function getAzureFileFetchUrl(url: string): string {
const ref = parsedUrl.searchParams.get('version')?.substr(2);
if (
parsedUrl.hostname !== 'dev.azure.com' ||
empty !== '' ||
userOrOrg === '' ||
project === '' ||