mirror of
https://github.com/angular/angular.git
synced 2026-09-14 13:54:52 +08:00
rootvector2
8e6d7f7190
`parseQueryParam` and the AngularJS-compat `parseKeyValue` accumulate query params into a plain object and check key presence with `obj.hasOwnProperty`, so a `hasOwnProperty` query key clobbers the method and the next lookup throws `TypeError`. Switch both to `Object.hasOwn`, which can't be shadowed by a query key.
(cherry picked from commit e5c37f21de)
89 lines
3.1 KiB
TypeScript
89 lines
3.1 KiB
TypeScript
/**
|
|
* @license
|
|
* Copyright Google LLC All Rights Reserved.
|
|
*
|
|
* Use of this source code is governed by an MIT-style license that can be
|
|
* found in the LICENSE file at https://angular.dev/license
|
|
*/
|
|
|
|
import {AngularJSUrlCodec} from '../src/params';
|
|
|
|
describe('AngularJSUrlCodec', () => {
|
|
const codec = new AngularJSUrlCodec();
|
|
|
|
describe('parse', () => {
|
|
it('should parse a complex URL', () => {
|
|
const result = codec.parse('http://server.com:1234/foo?bar=true#heading');
|
|
expect(result.href).toBe('http://server.com:1234/foo?bar=true#heading');
|
|
expect(result.protocol).toBe('http');
|
|
expect(result.host).toBe('server.com:1234');
|
|
expect(result.search).toBe('bar=true');
|
|
expect(result.hash).toBe('heading');
|
|
expect(result.hostname).toBe('server.com');
|
|
expect(result.port).toBe('1234');
|
|
expect(result.pathname).toBe('/foo');
|
|
});
|
|
|
|
it('should parse a URL without search', () => {
|
|
const result = codec.parse('http://server.com:1234/foo#heading');
|
|
expect(result.href).toBe('http://server.com:1234/foo#heading');
|
|
expect(result.search).toBe('');
|
|
expect(result.hash).toBe('heading');
|
|
expect(result.pathname).toBe('/foo');
|
|
});
|
|
|
|
it('should parse a URL without hash', () => {
|
|
const result = codec.parse('http://server.com:1234/foo?bar=true');
|
|
expect(result.href).toBe('http://server.com:1234/foo?bar=true');
|
|
expect(result.search).toBe('bar=true');
|
|
expect(result.hash).toBe('');
|
|
expect(result.pathname).toBe('/foo');
|
|
});
|
|
|
|
it('should parse a basic URL', () => {
|
|
const result = codec.parse('http://server.com');
|
|
expect(result.href).toBe('http://server.com/');
|
|
expect(result.protocol).toBe('http');
|
|
expect(result.host).toBe('server.com');
|
|
expect(result.search).toBe('');
|
|
expect(result.hash).toBe('');
|
|
expect(result.hostname).toBe('server.com');
|
|
expect(result.port).toBe('');
|
|
expect(result.pathname).toBe('/');
|
|
});
|
|
|
|
it('should apply a base', () => {
|
|
const withoutSlash = codec.parse('foo/bar', 'http://abc.xyz');
|
|
expect(withoutSlash.href).toBe('http://abc.xyz/foo/bar');
|
|
const withSlash = codec.parse('/foo/bar', 'http://abc.xyz/');
|
|
expect(withSlash.href).toBe('http://abc.xyz/foo/bar');
|
|
});
|
|
|
|
it('should ignore an empty base', () => {
|
|
const result = codec.parse('http://abc.xyz', '');
|
|
expect(result.href).toBe('http://abc.xyz/');
|
|
});
|
|
|
|
it('should throw an error for an invalid URL', () => {
|
|
expect(() => {
|
|
codec.parse('/foo/bar');
|
|
}).toThrowError('Invalid URL (/foo/bar) with base (undefined)');
|
|
});
|
|
|
|
it('should throw an error for an invalid base', () => {
|
|
expect(() => {
|
|
codec.parse('http://foo.bar', 'abc');
|
|
}).toThrowError('Invalid URL (http://foo.bar) with base (abc)');
|
|
});
|
|
});
|
|
|
|
describe('decodeSearch', () => {
|
|
it('should decode a search param named like an Object.prototype member', () => {
|
|
expect(codec.decodeSearch('hasOwnProperty=foo&bar=baz')).toEqual({
|
|
hasOwnProperty: 'foo',
|
|
bar: 'baz',
|
|
});
|
|
});
|
|
});
|
|
});
|