From cd4c26347a1846c74568e3d91a650c49dd0c7735 Mon Sep 17 00:00:00 2001 From: Denis Pushkarev Date: Tue, 5 Jul 2022 14:24:24 +0700 Subject: [PATCH] some stylistic changes --- tests/pure/es.map.js | 2 +- tests/pure/es.number.epsilon.js | 2 +- tests/pure/es.object.is.js | 4 ++-- tests/pure/es.set.js | 2 +- tests/pure/es.string.replace-all.js | 2 +- tests/pure/es.weak-map.js | 4 ++-- tests/pure/es.weak-set.js | 6 +++--- tests/pure/web.url.js | 16 ++++++++-------- tests/tests/es.aggregate-error.js | 2 +- tests/tests/es.error.cause.js | 2 +- tests/tests/es.map.js | 2 +- tests/tests/es.number.epsilon.js | 2 +- tests/tests/es.object.is.js | 4 ++-- tests/tests/es.regexp.constructor.js | 6 +++--- tests/tests/es.regexp.exec.js | 6 +++--- tests/tests/es.set.js | 2 +- tests/tests/es.string.replace-all.js | 2 +- tests/tests/es.string.replace.js | 2 +- tests/tests/es.weak-map.js | 4 ++-- tests/tests/es.weak-set.js | 6 +++--- tests/tests/web.url.js | 16 ++++++++-------- 21 files changed, 47 insertions(+), 47 deletions(-) diff --git a/tests/pure/es.map.js b/tests/pure/es.map.js index 28e7b977e450..a69c7a394664 100644 --- a/tests/pure/es.map.js +++ b/tests/pure/es.map.js @@ -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); } }); diff --git a/tests/pure/es.number.epsilon.js b/tests/pure/es.number.epsilon.js index 49a93fa5bbb4..804dcccecfe2 100644 --- a/tests/pure/es.number.epsilon.js +++ b/tests/pure/es.number.epsilon.js @@ -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'); }); diff --git a/tests/pure/es.object.is.js b/tests/pure/es.object.is.js index e99f17e9db77..805396b53b06 100644 --- a/tests/pure/es.object.is.js +++ b/tests/pure/es.object.is.js @@ -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 {}'); }); diff --git a/tests/pure/es.set.js b/tests/pure/es.set.js index cb57d3be4cf1..bbd6a2f52ea3 100644 --- a/tests/pure/es.set.js +++ b/tests/pure/es.set.js @@ -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); diff --git a/tests/pure/es.string.replace-all.js b/tests/pure/es.string.replace-all.js index bf784af9eeaf..9eca17623850 100644 --- a/tests/pure/es.string.replace-all.js +++ b/tests/pure/es.string.replace-all.js @@ -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'); diff --git a/tests/pure/es.weak-map.js b/tests/pure/es.weak-map.js index e36dad211133..0cdb693b8c8a 100644 --- a/tests/pure/es.weak-map.js +++ b/tests/pure/es.weak-map.js @@ -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); diff --git a/tests/pure/es.weak-set.js b/tests/pure/es.weak-set.js index 9fdb1e27b743..0cec42683ac1 100644 --- a/tests/pure/es.weak-set.js +++ b/tests/pure/es.weak-set.js @@ -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'); }); @@ -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'); }); diff --git a/tests/pure/web.url.js b/tests/pure/web.url.js index 038cc6c9368e..f7b0ebb786f8 100644 --- a/tests/pure/web.url.js +++ b/tests/pure/web.url.js @@ -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 diff --git a/tests/tests/es.aggregate-error.js b/tests/tests/es.aggregate-error.js index 122b89298fdf..8ba2c1ec12bb 100644 --- a/tests/tests/es.aggregate-error.js +++ b/tests/tests/es.aggregate-error.js @@ -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'); diff --git a/tests/tests/es.error.cause.js b/tests/tests/es.error.cause.js index 4461f68e2478..bf18293cddf3 100644 --- a/tests/tests/es.error.cause.js +++ b/tests/tests/es.error.cause.js @@ -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'); diff --git a/tests/tests/es.map.js b/tests/tests/es.map.js index b25085898d20..62271ea12cdb 100644 --- a/tests/tests/es.map.js +++ b/tests/tests/es.map.js @@ -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); } }); diff --git a/tests/tests/es.number.epsilon.js b/tests/tests/es.number.epsilon.js index 990bd6174c8c..b3b95115a271 100644 --- a/tests/tests/es.number.epsilon.js +++ b/tests/tests/es.number.epsilon.js @@ -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'); }); diff --git a/tests/tests/es.object.is.js b/tests/tests/es.object.is.js index 4c7dac4034f2..ef8f3daecaf9 100644 --- a/tests/tests/es.object.is.js +++ b/tests/tests/es.object.is.js @@ -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 {}'); }); diff --git a/tests/tests/es.regexp.constructor.js b/tests/tests/es.regexp.constructor.js index 474a67d1e52d..b897354e48e1 100644 --- a/tests/tests/es.regexp.constructor.js +++ b/tests/tests/es.regexp.constructor.js @@ -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'); diff --git a/tests/tests/es.regexp.exec.js b/tests/tests/es.regexp.exec.js index 1cff3012b21c..579f6cf0a010 100644 --- a/tests/tests/es.regexp.exec.js +++ b/tests/tests/es.regexp.exec.js @@ -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'); @@ -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 => { diff --git a/tests/tests/es.set.js b/tests/tests/es.set.js index 13c134996be2..3ed063d957e6 100644 --- a/tests/tests/es.set.js +++ b/tests/tests/es.set.js @@ -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); } }); diff --git a/tests/tests/es.string.replace-all.js b/tests/tests/es.string.replace-all.js index effe5678d024..cac712b41935 100644 --- a/tests/tests/es.string.replace-all.js +++ b/tests/tests/es.string.replace-all.js @@ -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'); diff --git a/tests/tests/es.string.replace.js b/tests/tests/es.string.replace.js index 375f89322fd8..99ab8277e1a8 100644 --- a/tests/tests/es.string.replace.js +++ b/tests/tests/es.string.replace.js @@ -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 diff --git a/tests/tests/es.weak-map.js b/tests/tests/es.weak-map.js index bff74d80ebd7..bab19dad76ae 100644 --- a/tests/tests/es.weak-map.js +++ b/tests/tests/es.weak-map.js @@ -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); diff --git a/tests/tests/es.weak-set.js b/tests/tests/es.weak-set.js index 11f8af432fbb..14d3f310053b 100644 --- a/tests/tests/es.weak-set.js +++ b/tests/tests/es.weak-set.js @@ -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'); }); @@ -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'); }); diff --git a/tests/tests/web.url.js b/tests/tests/web.url.js index 05934b0d1080..16f0a6bdbb86 100644 --- a/tests/tests/web.url.js +++ b/tests/tests/web.url.js @@ -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