Skip to content

Commit

Permalink
some stylistic changes
Browse files Browse the repository at this point in the history
  • Loading branch information
zloirock committed Jul 5, 2022
1 parent bc63ebd commit cd4c263
Show file tree
Hide file tree
Showing 21 changed files with 47 additions and 47 deletions.
2 changes: 1 addition & 1 deletion tests/pure/es.map.js
Expand Up @@ -251,7 +251,7 @@ QUnit.test('Map#size', assert => {
const getter = sizeDescriptor && sizeDescriptor.get;
const setter = sizeDescriptor && sizeDescriptor.set;
assert.same(typeof getter, 'function', 'size is getter');
assert.same(typeof setter, 'undefined', 'size isn\'t setter');
assert.same(typeof setter, 'undefined', 'size is not setter');
assert.throws(() => Map.prototype.size, TypeError);
}
});
Expand Down
2 changes: 1 addition & 1 deletion tests/pure/es.number.epsilon.js
Expand Up @@ -2,6 +2,6 @@ import EPSILON from 'core-js-pure/es/number/epsilon';

QUnit.test('Number.EPSILON', assert => {
assert.same(EPSILON, 2 ** -52, 'Is 2^-52');
assert.notSame(1, 1 + EPSILON, '1 isn\'t 1 + EPSILON');
assert.notSame(1, 1 + EPSILON, '1 is not 1 + EPSILON');
assert.same(1, 1 + EPSILON / 2, '1 is 1 + EPSILON / 2');
});
4 changes: 2 additions & 2 deletions tests/pure/es.object.is.js
Expand Up @@ -4,6 +4,6 @@ QUnit.test('Object.is', assert => {
assert.isFunction(is);
assert.true(is(1, 1), '1 is 1');
assert.true(is(NaN, NaN), '1 is 1');
assert.false(is(0, -0), '0 isn\'t -0');
assert.false(is({}, {}), '{} isn\'t {}');
assert.false(is(0, -0), '0 is not -0');
assert.false(is({}, {}), '{} is not {}');
});
2 changes: 1 addition & 1 deletion tests/pure/es.set.js
Expand Up @@ -245,7 +245,7 @@ QUnit.test('Set#size', assert => {
const getter = sizeDescriptor && sizeDescriptor.get;
const setter = sizeDescriptor && sizeDescriptor.set;
assert.same(typeof getter, 'function', 'size is getter');
assert.same(typeof setter, 'undefined', 'size isn\'t setter');
assert.same(typeof setter, 'undefined', 'size is not setter');
assert.throws(() => {
Set.prototype.size;
}, TypeError);
Expand Down
2 changes: 1 addition & 1 deletion tests/pure/es.string.replace-all.js
Expand Up @@ -29,7 +29,7 @@ QUnit.test('String#replaceAll', assert => {
assert.same(replaceAll('121314', '1', '$$'), '$2$3$4', '$$');
assert.same(replaceAll('121314', '1', '$&'), '121314', '$&');
assert.same(replaceAll('121314', '1', '$`'), '212312134', '$`');
assert.same(replaceAll('121314', '1', '$\''), '213142314344', '$\'');
assert.same(replaceAll('121314', '1', "$'"), '213142314344', "$'");

assert.throws(() => replaceAll(Symbol(), 'a', 'b'), 'throws on symbol context');
assert.throws(() => replaceAll('a', Symbol(), 'b'), 'throws on symbol argument 1');
Expand Down
4 changes: 2 additions & 2 deletions tests/pure/es.weak-map.js
Expand Up @@ -85,8 +85,8 @@ QUnit.test('WeakMap#delete', assert => {
assert.true(weakmap.has(a), 'WeakMap has values before .delete() #1');
assert.true(weakmap.has(b), 'WeakMap has values before .delete() #2');
weakmap.delete(a);
assert.false(weakmap.has(a), 'WeakMap hasn\'t value after .delete() #1');
assert.true(weakmap.has(b), 'WeakMap hasn\'t value after .delete() #2');
assert.false(weakmap.has(a), 'WeakMap has not value after .delete() #1');
assert.true(weakmap.has(b), 'WeakMap has not value after .delete() #2');
assert.notThrows(() => !weakmap.delete(1), 'return false on primitive');
const object = {};
weakmap.set(object, 42);
Expand Down
6 changes: 3 additions & 3 deletions tests/pure/es.weak-set.js
Expand Up @@ -84,8 +84,8 @@ QUnit.test('WeakSet#delete', assert => {
assert.true(weakset.has(a), 'WeakSet has values before .delete() #1');
assert.true(weakset.has(b), 'WeakSet has values before .delete() #2');
weakset.delete(a);
assert.false(weakset.has(a), 'WeakSet hasn\'t value after .delete() #1');
assert.true(weakset.has(b), 'WeakSet hasn\'t value after .delete() #2');
assert.false(weakset.has(a), 'WeakSet has not value after .delete() #1');
assert.true(weakset.has(b), 'WeakSet has not value after .delete() #2');
assert.notThrows(() => !weakset.delete(1), 'return false on primitive');
});

Expand All @@ -97,7 +97,7 @@ QUnit.test('WeakSet#has', assert => {
weakset.add(object);
assert.true(weakset.has(object), 'WeakSet has value after .add()');
weakset.delete(object);
assert.false(weakset.has(object), 'WeakSet hasn\'t value after .delete()');
assert.false(weakset.has(object), 'WeakSet has not value after .delete()');
assert.notThrows(() => !weakset.has(1), 'return false on primitive');
});

Expand Down
16 changes: 8 additions & 8 deletions tests/pure/web.url.js
Expand Up @@ -40,14 +40,14 @@ QUnit.test('URL constructor', assert => {
// assert.same(String(new URL('file://nnsc.nsf.net/bar/baz')), 'file://nnsc.nsf.net/bar/baz', 'file scheme'); // 'file:///bar/baz' in FF
// assert.same(String(new URL('file://localhost/bar/baz')), 'file:///bar/baz', 'file scheme'); // 'file://localhost/bar/baz' in Chrome

assert.throws(() => new URL(), 'TypeError: Failed to construct \'URL\': 1 argument required, but only 0 present.');
assert.throws(() => new URL(''), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('', 'about:blank'), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('abc'), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('//abc'), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('http:///www.domain.com/', 'abc'), 'TypeError: Failed to construct \'URL\': Invalid base URL');
assert.throws(() => new URL('http:///www.domain.com/', null), 'TypeError: Failed to construct \'URL\': Invalid base URL');
assert.throws(() => new URL('//abc', null), 'TypeError: Failed to construct \'URL\': Invalid base URL');
assert.throws(() => new URL(), 'TypeError: Failed to construct URL: 1 argument required, but only 0 present.');
assert.throws(() => new URL(''), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('', 'about:blank'), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('abc'), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('//abc'), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('http:///www.domain.com/', 'abc'), 'TypeError: Failed to construct URL: Invalid base URL');
assert.throws(() => new URL('http:///www.domain.com/', null), 'TypeError: Failed to construct URL: Invalid base URL');
assert.throws(() => new URL('//abc', null), 'TypeError: Failed to construct URL: Invalid base URL');
assert.throws(() => new URL('http://[20:0:0:1:0:0:0:ff'), 'incorrect IPv6');
assert.throws(() => new URL('http://[20:0:0:1:0:0:0:fg]'), 'incorrect IPv6');
// assert.throws(() => new URL('http://a%b'), 'forbidden host code point'); // no error in FF
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.aggregate-error.js
Expand Up @@ -18,7 +18,7 @@ QUnit.test('AggregateError', assert => {

assert.same(AggregateError.prototype.constructor, AggregateError, 'prototype constructor');
// eslint-disable-next-line no-prototype-builtins -- safe
assert.false(AggregateError.prototype.hasOwnProperty('cause'), 'prototype hasn\'t cause');
assert.false(AggregateError.prototype.hasOwnProperty('cause'), 'prototype has not cause');

assert.true(AggregateError([1], 1) instanceof AggregateError, 'no cause, without new');
assert.true(new AggregateError([1], 1) instanceof AggregateError, 'no cause, with new');
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.error.cause.js
Expand Up @@ -16,7 +16,7 @@ function runErrorTestCase($Error, ERROR_NAME) {

assert.same($Error.prototype.constructor, $Error, 'prototype constructor');
// eslint-disable-next-line no-prototype-builtins -- safe
assert.false($Error.prototype.hasOwnProperty('cause'), 'prototype hasn\'t cause');
assert.false($Error.prototype.hasOwnProperty('cause'), 'prototype has not cause');

assert.true($Error(1) instanceof $Error, 'no cause, without new');
assert.true(new $Error(1) instanceof $Error, 'no cause, with new');
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.map.js
Expand Up @@ -276,7 +276,7 @@ QUnit.test('Map#size', assert => {
const getter = sizeDescriptor && sizeDescriptor.get;
const setter = sizeDescriptor && sizeDescriptor.set;
assert.same(typeof getter, 'function', 'size is getter');
assert.same(typeof setter, 'undefined', 'size isn\'t setter');
assert.same(typeof setter, 'undefined', 'size is not setter');
assert.throws(() => Map.prototype.size, TypeError);
}
});
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.number.epsilon.js
Expand Up @@ -5,6 +5,6 @@ QUnit.test('Number.EPSILON', assert => {
assert.nonConfigurable(Number, 'EPSILON');
assert.nonWritable(Number, 'EPSILON');
assert.same(EPSILON, 2 ** -52, 'Is 2^-52');
assert.notSame(1, 1 + EPSILON, '1 isn\'t 1 + EPSILON');
assert.notSame(1, 1 + EPSILON, '1 is not 1 + EPSILON');
assert.same(1, 1 + EPSILON / 2, '1 is 1 + EPSILON / 2');
});
4 changes: 2 additions & 2 deletions tests/tests/es.object.is.js
Expand Up @@ -7,6 +7,6 @@ QUnit.test('Object.is', assert => {
assert.nonEnumerable(Object, 'is');
assert.true(is(1, 1), '1 is 1');
assert.true(is(NaN, NaN), '1 is 1');
assert.false(is(0, -0), '0 isn\'t -0');
assert.false(is({}, {}), '{} isn\'t {}');
assert.false(is(0, -0), '0 is not -0');
assert.false(is({}, {}), '{} is not {}');
});
6 changes: 3 additions & 3 deletions tests/tests/es.regexp.constructor.js
Expand Up @@ -13,12 +13,12 @@ if (DESCRIPTORS) {
assert.same({}.toString.call(RegExp()).slice(8, -1), 'RegExp');
assert.same({}.toString.call(new RegExp()).slice(8, -1), 'RegExp');
let regexp = /a/g;
assert.notSame(regexp, new RegExp(regexp), 'new RegExp(regexp) isn\'t regexp');
assert.notSame(regexp, new RegExp(regexp), 'new RegExp(regexp) is not regexp');
assert.same(regexp, RegExp(regexp), 'RegExp(regexp) is regexp');
regexp[Symbol.match] = false;
assert.notSame(regexp, RegExp(regexp), 'RegExp(regexp) isn\'t regexp, changed Symbol.match');
assert.notSame(regexp, RegExp(regexp), 'RegExp(regexp) is not regexp, changed Symbol.match');
const object = {};
assert.notSame(object, RegExp(object), 'RegExp(O) isn\'t O');
assert.notSame(object, RegExp(object), 'RegExp(O) is not O');
object[Symbol.match] = true;
object.constructor = RegExp;
assert.same(object, RegExp(object), 'RegExp(O) is O, changed Symbol.match');
Expand Down
6 changes: 3 additions & 3 deletions tests/tests/es.regexp.exec.js
Expand Up @@ -4,7 +4,7 @@ QUnit.test('RegExp#exec lastIndex updating', assert => {
let re = /b/;
assert.same(re.lastIndex, 0, '.lastIndex starts at 0 for non-global regexps');
re.exec('abc');
assert.same(re.lastIndex, 0, '.lastIndex isn\'t updated for non-global regexps');
assert.same(re.lastIndex, 0, '.lastIndex is not updated for non-global regexps');

re = /b/g;
assert.same(re.lastIndex, 0, '.lastIndex starts at 0 for global regexps');
Expand All @@ -13,11 +13,11 @@ QUnit.test('RegExp#exec lastIndex updating', assert => {

re = /b*/;
re.exec('a');
assert.same(re.lastIndex, 0, '.lastIndex isn\'t updated for non-global regexps if the match is empty');
assert.same(re.lastIndex, 0, '.lastIndex is not updated for non-global regexps if the match is empty');

re = /b*/g;
re.exec('a');
assert.same(re.lastIndex, 0, '.lastIndex isn\'t updated for global regexps if the match is empty');
assert.same(re.lastIndex, 0, '.lastIndex is not updated for global regexps if the match is empty');
});

QUnit.test('RegExp#exec capturing groups', assert => {
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.set.js
Expand Up @@ -261,7 +261,7 @@ QUnit.test('Set#size', assert => {
const getter = sizeDescriptor && sizeDescriptor.get;
const setter = sizeDescriptor && sizeDescriptor.set;
assert.same(typeof getter, 'function', 'size is getter');
assert.same(typeof setter, 'undefined', 'size isn\'t setter');
assert.same(typeof setter, 'undefined', 'size is not setter');
assert.throws(() => Set.prototype.size, TypeError);
}
});
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.string.replace-all.js
Expand Up @@ -31,7 +31,7 @@ QUnit.test('String#replaceAll', assert => {
assert.same('121314'.replaceAll('1', '$$'), '$2$3$4', '$$');
assert.same('121314'.replaceAll('1', '$&'), '121314', '$&');
assert.same('121314'.replaceAll('1', '$`'), '212312134', '$`');
assert.same('121314'.replaceAll('1', '$\''), '213142314344', '$\'');
assert.same('121314'.replaceAll('1', "$'"), '213142314344', "$'");

assert.throws(() => replaceAll.call(Symbol(), 'a', 'b'), 'throws on symbol context');
assert.throws(() => replaceAll.call('a', Symbol(), 'b'), 'throws on symbol argument 1');
Expand Down
2 changes: 1 addition & 1 deletion tests/tests/es.string.replace.js
Expand Up @@ -212,7 +212,7 @@ QUnit.test('RegExp#@@replace correctly handles substitutions', assert => {
// eslint-disable-next-line regexp/no-useless-dollar-replacements -- required for testing
assert.same('1234'.replace(re, '$<!!!>'), '174');
assert.same('1234'.replace(re, '$`'), '114');
assert.same('1234'.replace(re, '$\''), '144');
assert.same('1234'.replace(re, "$'"), '144');
assert.same('1234'.replace(re, '$$'), '1$4');
assert.same('1234'.replace(re, '$&'), '1234');
// eslint-disable-next-line regexp/prefer-escape-replacement-dollar-char -- required for testing
Expand Down
4 changes: 2 additions & 2 deletions tests/tests/es.weak-map.js
Expand Up @@ -85,8 +85,8 @@ QUnit.test('WeakMap#delete', assert => {
assert.true(weakmap.has(a), 'WeakMap has values before .delete() #1');
assert.true(weakmap.has(b), 'WeakMap has values before .delete() #2');
weakmap.delete(a);
assert.false(weakmap.has(a), 'WeakMap hasn\'t value after .delete() #1');
assert.true(weakmap.has(b), 'WeakMap hasn\'t value after .delete() #2');
assert.false(weakmap.has(a), 'WeakMap has not value after .delete() #1');
assert.true(weakmap.has(b), 'WeakMap has not value after .delete() #2');
assert.notThrows(() => !weakmap.delete(1), 'return false on primitive');
const object = {};
weakmap.set(object, 42);
Expand Down
6 changes: 3 additions & 3 deletions tests/tests/es.weak-set.js
Expand Up @@ -87,8 +87,8 @@ QUnit.test('WeakSet#delete', assert => {
assert.true(weakset.has(a), 'WeakSet has values before .delete() #1');
assert.true(weakset.has(b), 'WeakSet has values before .delete() #2');
weakset.delete(a);
assert.false(weakset.has(a), 'WeakSet hasn\'t value after .delete() #1');
assert.true(weakset.has(b), 'WeakSet hasn\'t value after .delete() #2');
assert.false(weakset.has(a), 'WeakSet has not value after .delete() #1');
assert.true(weakset.has(b), 'WeakSet has not value after .delete() #2');
assert.notThrows(() => !weakset.delete(1), 'return false on primitive');
});

Expand All @@ -104,7 +104,7 @@ QUnit.test('WeakSet#has', assert => {
weakset.add(object);
assert.true(weakset.has(object), 'WeakSet has value after .add()');
weakset.delete(object);
assert.false(weakset.has(object), 'WeakSet hasn\'t value after .delete()');
assert.false(weakset.has(object), 'WeakSet has not value after .delete()');
assert.notThrows(() => !weakset.has(1), 'return false on primitive');
});

Expand Down
16 changes: 8 additions & 8 deletions tests/tests/web.url.js
Expand Up @@ -39,14 +39,14 @@ QUnit.test('URL constructor', assert => {
// assert.same(String(new URL('file://nnsc.nsf.net/bar/baz')), 'file://nnsc.nsf.net/bar/baz', 'file scheme'); // 'file:///bar/baz' in FF
// assert.same(String(new URL('file://localhost/bar/baz')), 'file:///bar/baz', 'file scheme'); // 'file://localhost/bar/baz' in Chrome

assert.throws(() => new URL(), 'TypeError: Failed to construct \'URL\': 1 argument required, but only 0 present.');
assert.throws(() => new URL(''), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('', 'about:blank'), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('abc'), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('//abc'), 'TypeError: Failed to construct \'URL\': Invalid URL');
assert.throws(() => new URL('http:///www.domain.com/', 'abc'), 'TypeError: Failed to construct \'URL\': Invalid base URL');
assert.throws(() => new URL('http:///www.domain.com/', null), 'TypeError: Failed to construct \'URL\': Invalid base URL');
assert.throws(() => new URL('//abc', null), 'TypeError: Failed to construct \'URL\': Invalid base URL');
assert.throws(() => new URL(), 'TypeError: Failed to construct URL: 1 argument required, but only 0 present.');
assert.throws(() => new URL(''), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('', 'about:blank'), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('abc'), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('//abc'), 'TypeError: Failed to construct URL: Invalid URL');
assert.throws(() => new URL('http:///www.domain.com/', 'abc'), 'TypeError: Failed to construct URL: Invalid base URL');
assert.throws(() => new URL('http:///www.domain.com/', null), 'TypeError: Failed to construct URL: Invalid base URL');
assert.throws(() => new URL('//abc', null), 'TypeError: Failed to construct URL: Invalid base URL');
assert.throws(() => new URL('http://[20:0:0:1:0:0:0:ff'), 'incorrect IPv6');
assert.throws(() => new URL('http://[20:0:0:1:0:0:0:fg]'), 'incorrect IPv6');
// assert.throws(() => new URL('http://a%b'), 'forbidden host code point'); // no error in FF
Expand Down

0 comments on commit cd4c263

Please sign in to comment.