honeymoose
/
angular-cn
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

test(service-worker): ensure `SwTestHarness#parseUrl()` behaves the same on browser and Node.js (#27080) 

PR Close #27080
This commit is contained in:
George Kalpakas 2019-03-20 23:29:14 +02:00 committed by Matias Niemelä
parent b3dda0ebc1
commit 415de9a291
1 changed files with 15 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
packages/service-worker/worker/testing/scope.ts
View File

@ -95,8 +95,13 @@ export class SwTestHarness implements ServiceWorkerGlobalScope, Adapter, Context
} as any;
static envIsSupported(): boolean {
return (typeof URL === 'function') ||
(typeof require === 'function' && typeof require('url')['parse'] === 'function');
if (typeof URL === 'function') {
return true;
}
// In older Node.js versions, the `URL` global does not exist. We can use `url` instead.
const url = (typeof require === 'function') && require('url');
return url && (typeof url.parse === 'function') && (typeof url.resolve === 'function');
}
time: number;
@ -176,13 +181,14 @@ export class SwTestHarness implements ServiceWorkerGlobalScope, Adapter, Context
}
parseUrl(url: string, relativeTo?: string): {origin: string, path: string} {
if (typeof URL === 'function') {
const obj = new URL(url, relativeTo);
return {origin: obj.origin, path: obj.pathname};
} else {
const obj = require('url').parse(url);
return {origin: `${obj.protocol}//${obj.host}`, path: obj.pathname};
}
const parsedUrl: URL = (typeof URL === 'function') ?
new URL(url, relativeTo) :
require('url').parse(require('url').resolve(relativeTo || '', url));
return {
origin: parsedUrl.origin || `${parsedUrl.protocol}//${parsedUrl.host}`,
path: parsedUrl.pathname,
};
}
async skipWaiting(): Promise<void> { this.skippedWaiting = true; }