fix some assert messages

Author: zloirock

tests/unit-global/es.data-view.js

@@ -29,7 +29,7 @@ QUnit.test('DataView', assert => {
     assert.same(dataview.byteLength, 0, '#byteLength, passed buffer and byteOffset with buffer length');
     // TypeError in IE + FF bug - TypeError instead of RangeError
     assert.throws(() => new DataView(new ArrayBuffer(8), -1), RangeError, 'If offset < 0, throw a RangeError exception');
-    assert.throws(() => new DataView(new ArrayBuffer(8), 16), RangeError, 'If newByteLength < 0, throw a RangeError exception');
+    assert.throws(() => new DataView(new ArrayBuffer(8), 16), RangeError, 'If offset > bufferByteLength, throw a RangeError exception');
     assert.throws(() => new DataView(new ArrayBuffer(24), 8, 24), RangeError, 'If offset+newByteLength > bufferByteLength, throw a RangeError exception');
     assert.throws(() => new DataView(new ArrayBuffer(8), 0, -1), RangeError, 'negative byteLength throws RangeError');
     // Android ~ 4.0
@@ -38,7 +38,7 @@ QUnit.test('DataView', assert => {
   } else {
     // FF bug - TypeError instead of RangeError
     assert.throws(() => new DataView(new ArrayBuffer(8), -1), 'If offset < 0, throw a RangeError exception');
-    assert.throws(() => new DataView(new ArrayBuffer(8), 16), 'If newByteLength < 0, throw a RangeError exception');
+    assert.throws(() => new DataView(new ArrayBuffer(8), 16), 'If offset > bufferByteLength, throw a RangeError exception');
     assert.throws(() => new DataView(new ArrayBuffer(24), 8, 24), 'If offset+newByteLength > bufferByteLength, throw a RangeError exception');
     assert.throws(() => new DataView(new ArrayBuffer(8), 0, -1), 'negative byteLength throws');
   }

tests/unit-global/es.iterator.concat.js

@@ -43,12 +43,12 @@ QUnit.test('Iterator.concat', assert => {
 
   iterator = concat(createIterable([1, 2, 3]));
   assert.deepEqual(iterator.return(), { done: true, value: undefined }, '.return with no active inner iterator result');
-  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.return with no active inner iterator result on closed iterator');
+  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.next on closed iterator');
 
   iterator = concat(createIterable([1, 2, 3]));
   assert.deepEqual(iterator.next(), { done: false, value: 1 }, '.next with active inner iterator result');
   assert.deepEqual(iterator.return(), { done: true, value: undefined }, '.return with active inner iterator result');
-  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.return with active inner iterator result on closed iterator');
+  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.next on closed iterator after .return with active inner iterator');
 
   let called = false;
   iterator = concat(createIterable([1, 2, 3], {

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

@@ -62,7 +62,7 @@ const run = assert => {
   assert.arrayEqual('tesst'.split(/(s)*?/), ['t', undefined, 'e', undefined, 's', undefined, 's', undefined, 't'], "'tesst'.split(/(s)*?/) results in ['t', undefined, 'e', undefined, 's', undefined, 's', undefined, 't']");
   assert.arrayEqual('tesst'.split(/(s*)/), ['t', '', 'e', 'ss', 't'], "'tesst'.split(/(s*)/) results in ['t', '', 'e', 'ss', 't']");
   assert.arrayEqual('tesst'.split(/(s*?)/), ['t', '', 'e', '', 's', '', 's', '', 't'], "'tesst'.split(/(s*?)/) results in ['t', '', 'e', '', 's', '', 's', '', 't']");
-  assert.arrayEqual('tesst'.split(/s*/), ['t', 'e', 't'], "'tesst'.split(/(?:s)*/) results in ['t', 'e', 't']");
+  assert.arrayEqual('tesst'.split(/s*/), ['t', 'e', 't'], "'tesst'.split(/s*/) results in ['t', 'e', 't']");
   assert.arrayEqual('tesst'.split(/(?=s+)/), ['te', 's', 'st'], "'tesst'.split(/(?=s+)/) results in ['te', 's', 'st']");
   assert.arrayEqual('test'.split('t'), ['', 'es', ''], "'test'.split('t') results in ['', 'es', '']");
   assert.arrayEqual('test'.split('es'), ['t', 't'], "'test'.split('es') results in ['t', 't']");

tests/unit-global/esnext.async-iterator.from.js

@@ -41,7 +41,7 @@ QUnit.test('AsyncIterator.from', assert => {
   }).then(() => {
     assert.avoid();
   }, error => {
-    assert.same(error, 42, 'rejection on a callback error');
+    assert.same(error, 42, 'rejection on a `.next()` promise rejection');
     assert.true(closableIterator.closed, 'closes sync iterator on promise rejection');
   });
 });

tests/unit-pure/es.iterator.concat.js

@@ -42,12 +42,12 @@ QUnit.test('Iterator.concat', assert => {
 
   iterator = concat(createIterable([1, 2, 3]));
   assert.deepEqual(iterator.return(), { done: true, value: undefined }, '.return with no active inner iterator result');
-  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.return with no active inner iterator result on closed iterator');
+  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.next on closed iterator');
 
   iterator = concat(createIterable([1, 2, 3]));
   assert.deepEqual(iterator.next(), { done: false, value: 1 }, '.next with active inner iterator result');
   assert.deepEqual(iterator.return(), { done: true, value: undefined }, '.return with active inner iterator result');
-  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.return with active inner iterator result on closed iterator');
+  assert.deepEqual(iterator.next(), { done: true, value: undefined }, '.next on closed iterator after .return with active inner iterator');
 
   let called = false;
   iterator = concat(createIterable([1, 2, 3], {

tests/unit-pure/esnext.async-iterator.from.js

@@ -44,7 +44,7 @@ QUnit.test('AsyncIterator.from', assert => {
   }).then(() => {
     assert.avoid();
   }, error => {
-    assert.same(error, 42, 'rejection on a callback error');
+    assert.same(error, 42, 'rejection on a `.next()` promise rejection');
     assert.true(closableIterator.closed, 'closes sync iterator on promise rejection');
   });
 });