add some missed error types to assert.throws

Author: zloirock

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

@@ -120,8 +120,8 @@ QUnit.test('Array#sort', assert => {
   assert.same([obj1, obj2].sort()[0], obj1, 'stable sort for equal string representations');
 
   assert.notThrows(() => [1, 2, 3].sort(undefined).length === 3, 'works with undefined');
-  assert.throws(() => [1, 2, 3].sort(null), 'throws on null');
-  assert.throws(() => [1, 2, 3].sort({}), 'throws on {}');
+  assert.throws(() => [1, 2, 3].sort(null), TypeError, 'throws on null');
+  assert.throws(() => [1, 2, 3].sort({}), TypeError, 'throws on {}');
 
   if (typeof Symbol == 'function' && !Symbol.sham) {
     assert.throws(() => [Symbol(1), Symbol(2)].sort(), 'w/o cmp throws on symbols');

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

@@ -117,8 +117,8 @@ QUnit.test('Array#toSorted', assert => {
   assert.same(result, 'DGBEFHACIJK', 'stable #2');
 
   assert.notThrows(() => [1, 2, 3].toSorted(undefined).length === 3, 'works with undefined');
-  assert.throws(() => [1, 2, 3].toSorted(null), 'throws on null');
-  assert.throws(() => [1, 2, 3].toSorted({}), 'throws on {}');
+  assert.throws(() => [1, 2, 3].toSorted(null), TypeError, 'throws on null');
+  assert.throws(() => [1, 2, 3].toSorted({}), TypeError, 'throws on {}');
 
   if (typeof Symbol == 'function' && !Symbol.sham) {
     assert.throws(() => [Symbol(1), Symbol(2)].toSorted(), 'w/o cmp throws on symbols');

tests/unit-global/es.reflect.construct.js

@@ -23,6 +23,6 @@ QUnit.test('Reflect.construct', assert => {
   assert.notThrows(() => getPrototypeOf(construct(B, [])) === Object.prototype);
   assert.notThrows(() => typeof construct(Date, []).getTime() == 'number', 'works with native constructors with 2 arguments');
   // eslint-disable-next-line prefer-arrow-callback -- testing
-  assert.throws(() => construct(function () { /* empty */ }), 'throws when the second argument is not an object');
+  assert.throws(() => construct(function () { /* empty */ }), TypeError, 'throws when the second argument is not an object');
 });
 

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

@@ -58,8 +58,8 @@ if (DESCRIPTORS) QUnit.test('%TypedArrayPrototype%.sort', assert => {
 
     assert.throws(() => sort.call([0], () => { /* empty */ }), "isn't generic");
     assert.notThrows(() => new TypedArray([1, 2, 3]).sort(undefined).length === 3, 'works with undefined');
-    assert.throws(() => new TypedArray([1, 2, 3]).sort(null), 'throws on null');
-    assert.throws(() => new TypedArray([1, 2, 3]).sort({}), 'throws on {}');
+    assert.throws(() => new TypedArray([1, 2, 3]).sort(null), TypeError, 'throws on null');
+    assert.throws(() => new TypedArray([1, 2, 3]).sort({}), TypeError, 'throws on {}');
     if (STRICT) {
       assert.throws(() => sort.call(null), TypeError, 'ToObject(this)');
       assert.throws(() => sort.call(undefined), TypeError, 'ToObject(this)');

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

@@ -61,8 +61,8 @@ if (DESCRIPTORS) QUnit.test('%TypedArrayPrototype%.toSorted', assert => {
     }
 
     assert.notThrows(() => new TypedArray([1, 2, 3]).toSorted(undefined).length === 3, 'works with undefined');
-    assert.throws(() => new TypedArray([1, 2, 3]).toSorted(null), 'throws on null');
-    assert.throws(() => new TypedArray([1, 2, 3]).toSorted({}), 'throws on {}');
+    assert.throws(() => new TypedArray([1, 2, 3]).toSorted(null), TypeError, 'throws on null');
+    assert.throws(() => new TypedArray([1, 2, 3]).toSorted({}), TypeError, 'throws on {}');
 
     assert.throws(() => toSorted.call(null), TypeError, "isn't generic #1");
     assert.throws(() => toSorted.call(undefined), TypeError, "isn't generic #2");

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

@@ -5,8 +5,8 @@ import sort from 'core-js-pure/es/array/sort';
 QUnit.test('Array#sort', assert => {
   assert.isFunction(sort);
   assert.notThrows(() => sort([1, 2, 3], undefined), 'works with undefined');
-  assert.throws(() => sort([1, 2, 3], null), 'throws on null');
-  assert.throws(() => sort([1, 2, 3], {}), 'throws on {}');
+  assert.throws(() => sort([1, 2, 3], null), TypeError, 'throws on null');
+  assert.throws(() => sort([1, 2, 3], {}), TypeError, 'throws on {}');
 
   assert.deepEqual(sort([1, 3, 2]), [1, 2, 3], '#1');
   assert.deepEqual(sort([1, 3, 2, 11]), [1, 11, 2, 3], '#2');

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

@@ -113,8 +113,8 @@ QUnit.test('Array#toSorted', assert => {
   assert.same(result, 'DGBEFHACIJK', 'stable #2');
 
   assert.notThrows(() => toSorted([1, 2, 3], undefined).length === 3, 'works with undefined');
-  assert.throws(() => toSorted([1, 2, 3], null), 'throws on null');
-  assert.throws(() => toSorted([1, 2, 3], {}), 'throws on {}');
+  assert.throws(() => toSorted([1, 2, 3], null), TypeError, 'throws on null');
+  assert.throws(() => toSorted([1, 2, 3], {}), TypeError, 'throws on {}');
 
   if (typeof Symbol == 'function' && !Symbol.sham) {
     assert.throws(() => toSorted([Symbol(1), Symbol(2)]), 'w/o cmp throws on symbols');

tests/unit-pure/es.reflect.construct.js

@@ -24,5 +24,5 @@ QUnit.test('Reflect.construct', assert => {
   assert.notThrows(() => getPrototypeOf(construct(B, [])) === Object.prototype);
   assert.notThrows(() => typeof construct(Date, []).getTime() == 'number', 'works with native constructors with 2 arguments');
   // eslint-disable-next-line prefer-arrow-callback -- testing
-  assert.throws(() => construct(function () { /* empty */ }), 'throws when the second argument is not an object');
+  assert.throws(() => construct(function () { /* empty */ }), TypeError, 'throws when the second argument is not an object');
 });