Skip to content

Commit a715a4d

Browse files
committed
fix some assert messages
1 parent eb092c7 commit a715a4d

21 files changed

+32
-32
lines changed

tests/unit-global/es.array.sort.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ QUnit.test('Array#sort', assert => {
1010

1111
assert.deepEqual([1, 3, 2].sort(), [1, 2, 3], '#1');
1212
assert.deepEqual([1, 3, 2, 11].sort(), [1, 11, 2, 3], '#2');
13-
assert.deepEqual([1, -1, 3, NaN, 2, 0, 11, -0].sort(), [-1, 0, -0, 1, 11, 2, 3, NaN], '#1');
13+
assert.deepEqual([1, -1, 3, NaN, 2, 0, 11, -0].sort(), [-1, 0, -0, 1, 11, 2, 3, NaN], '#3');
1414

1515
let array = Array(5);
1616
array[0] = 1;

tests/unit-global/es.array.to-sorted.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ QUnit.test('Array#toSorted', assert => {
1414

1515
assert.deepEqual([1, 3, 2].toSorted(), [1, 2, 3], '#1');
1616
assert.deepEqual([1, 3, 2, 11].toSorted(), [1, 11, 2, 3], '#2');
17-
assert.deepEqual([1, -1, 3, NaN, 2, 0, 11, -0].toSorted(), [-1, 0, -0, 1, 11, 2, 3, NaN], '#1');
17+
assert.deepEqual([1, -1, 3, NaN, 2, 0, 11, -0].toSorted(), [-1, 0, -0, 1, 11, 2, 3, NaN], '#3');
1818

1919
array = Array(5);
2020
array[0] = 1;

tests/unit-global/es.math.trunc.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ QUnit.test('Math.trunc', assert => {
1515
assert.same(trunc(null), 0, 'null -> 0');
1616
assert.same(trunc({}), NaN, '{} -> NaN');
1717
assert.same(trunc([]), 0, '[] -> 0');
18-
assert.same(trunc(1.01), 1, '1.01 -> 0');
19-
assert.same(trunc(1.99), 1, '1.99 -> 0');
18+
assert.same(trunc(1.01), 1, '1.01 -> 1');
19+
assert.same(trunc(1.99), 1, '1.99 -> 1');
2020
assert.same(trunc(-1), -1, '-1 -> -1');
2121
assert.same(trunc(-1.99), -1, '-1.99 -> -1');
2222
assert.same(trunc(-555.555), -555, '-555.555 -> -555');

tests/unit-global/es.object.is-extensible.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ QUnit.test('Object.isExtensible', assert => {
1010
const primitives = [42, 'string', false, null, undefined];
1111
for (const value of primitives) {
1212
assert.notThrows(() => isExtensible(value) || true, `accept ${ value }`);
13-
assert.false(isExtensible(value), `returns true on ${ value }`);
13+
assert.false(isExtensible(value), `returns false on ${ value }`);
1414
}
1515
assert.true(isExtensible({}));
1616
if (NATIVE) assert.false(isExtensible(preventExtensions({})));

tests/unit-global/es.object.is.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ QUnit.test('Object.is', assert => {
66
assert.looksNative(is);
77
assert.nonEnumerable(Object, 'is');
88
assert.true(is(1, 1), '1 is 1');
9-
assert.true(is(NaN, NaN), '1 is 1');
9+
assert.true(is(NaN, NaN), 'NaN is NaN');
1010
assert.false(is(0, -0), '0 is not -0');
1111
assert.false(is({}, {}), '{} is not {}');
1212
});

tests/unit-global/es.object.lookup-setter.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ if (DESCRIPTORS) {
1414
const object = {};
1515
function setter() { /* empty */ }
1616
object.__defineSetter__('key', setter);
17-
assert.same(object.__lookupSetter__('key'), setter, 'own getter');
18-
assert.same(create(object).__lookupSetter__('key'), setter, 'proto getter');
17+
assert.same(object.__lookupSetter__('key'), setter, 'own setter');
18+
assert.same(create(object).__lookupSetter__('key'), setter, 'proto setter');
1919
assert.same(create(object).__lookupSetter__('foo'), undefined, 'empty proto');
2020
if (STRICT) {
2121
assert.throws(() => __lookupSetter__.call(null, 1, () => { /* empty */ }), TypeError, 'Throws on null as `this`');

tests/unit-global/es.string.split.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -358,14 +358,14 @@ const run = assert => {
358358
assert.same(split.length, 1, 'S15.5.4.14_A2_T16 #2');
359359
assert.same(split[0], '', 'S15.5.4.14_A2_T16 #3');
360360
split = Object(' ').split('');
361-
assert.same(split.constructor, Array, 'S15.5.4.14_A2_T18 #1');
362-
assert.same(split.length, 1, 'S15.5.4.14_A2_T18 #2');
363-
assert.same(split[0], ' ', 'S15.5.4.14_A2_T18 #3');
361+
assert.same(split.constructor, Array, 'S15.5.4.14_A2_T17 #1');
362+
assert.same(split.length, 1, 'S15.5.4.14_A2_T17 #2');
363+
assert.same(split[0], ' ', 'S15.5.4.14_A2_T17 #3');
364364
split = Object(' ').split(' ');
365-
assert.same(split.constructor, Array, 'S15.5.4.14_A2_T19 #1');
366-
assert.same(split.length, 2, 'S15.5.4.14_A2_T19 #2');
367-
assert.same(split[0], '', 'S15.5.4.14_A2_T19 #3');
368-
assert.same(split[1], '', 'S15.5.4.14_A2_T19 #4');
365+
assert.same(split.constructor, Array, 'S15.5.4.14_A2_T18 #1');
366+
assert.same(split.length, 2, 'S15.5.4.14_A2_T18 #2');
367+
assert.same(split[0], '', 'S15.5.4.14_A2_T18 #3');
368+
assert.same(split[1], '', 'S15.5.4.14_A2_T18 #4');
369369
split = ''.split('x');
370370
assert.same(split.constructor, Array, 'S15.5.4.14_A2_T19 #1');
371371
assert.same(split.length, 1, 'S15.5.4.14_A2_T19 #2');

tests/unit-global/es.typed-array.slice.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ if (DESCRIPTORS) QUnit.test('%TypedArrayPrototype%.slice', assert => {
55
for (const { name, TypedArray } of TYPED_ARRAYS) {
66
const { slice } = TypedArray.prototype;
77
assert.isFunction(slice, `${ name }::slice is function`);
8-
assert.arity(slice, 2, `${ name }::slice arity is 0`);
8+
assert.arity(slice, 2, `${ name }::slice arity is 2`);
99
assert.name(slice, 'slice', `${ name }::slice name is 'slice'`);
1010
assert.looksNative(slice, `${ name }::slice looks native`);
1111
const array = new TypedArray([1, 2, 3, 4, 5]);

tests/unit-global/es.typed-array.with.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ if (DESCRIPTORS) QUnit.test('%TypedArrayPrototype%.with', assert => {
77
const { with: withAt } = TypedArray.prototype;
88

99
assert.isFunction(withAt, `${ name }::with is function`);
10-
assert.arity(withAt, 2, `${ name }::with arity is 0`);
10+
assert.arity(withAt, 2, `${ name }::with arity is 2`);
1111
// assert.name(withAt, 'with', `${ name }::with name is 'with'`);
1212
assert.looksNative(withAt, `${ name }::with looks native`);
1313

tests/unit-global/esnext.iterator.sliding.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ QUnit.test('Iterator#sliding', assert => {
2424
assert.true(result instanceof Iterator, 'returns iterator');
2525
assert.deepEqual(result.next(), { done: false, value: [1, 2, 3] }, '.next with active inner iterator result');
2626
assert.deepEqual(result.return(), { done: true, value: undefined }, '.return with active inner iterator result');
27-
assert.deepEqual(result.next(), { done: true, value: undefined }, '.return with active inner iterator result on closed iterator');
27+
assert.deepEqual(result.next(), { done: true, value: undefined }, '.next on closed iterator');
2828

2929
if (STRICT) {
3030
assert.throws(() => sliding.call('', 1), TypeError, 'iterable non-object this');

0 commit comments

Comments
 (0)