fixup: update error message

nodejs · Oct 9, 2015 · 3b9e2be · 3b9e2be
1 parent c3672b0
commit 3b9e2be
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 23 deletions.
diff --git a/test/parallel/test-stringbytes-external-at-max.js b/test/parallel/test-stringbytes-external-at-max.js
@@ -4,27 +4,25 @@ require('../common');
 const assert = require('assert');
 
 // v8 fails silently if string length > v8::String::kMaxLength
-(function() {
-  // v8::String::kMaxLength defined in v8.h
-  const kStringMaxLength = process.binding('buffer').kStringMaxLength;
+// v8::String::kMaxLength defined in v8.h
+const kStringMaxLength = process.binding('buffer').kStringMaxLength;
 
-  try {
-    new Buffer(kStringMaxLength * 3);
-  } catch(e) {
-    assert.equal(e.message, 'Invalid array buffer length');
-    console.log(
-        '1..0 # Skipped: intensive toString tests due to memory confinements');
-    return;
-  }
+try {
+  new Buffer(kStringMaxLength * 3);
+} catch(e) {
+  assert.equal(e.message, 'Invalid typed array length');
+  console.log(
+      '1..0 # Skipped: intensive toString tests due to memory confinements');
+  return;
+}
 
-  const buf = new Buffer(kStringMaxLength);
+const buf = new Buffer(kStringMaxLength);
 
-  var maxString = buf.toString();
-  assert.equal(maxString.length, kStringMaxLength);
-  // Free the memory early instead of at the end of the next assignment
-  maxString = undefined;
+var maxString = buf.toString();
+assert.equal(maxString.length, kStringMaxLength);
+// Free the memory early instead of at the end of the next assignment
+maxString = undefined;
 
-  maxString = buf.toString('binary');
-  assert.equal(maxString.length, kStringMaxLength);
-  maxString = undefined;
-})();
+maxString = buf.toString('binary');
+assert.equal(maxString.length, kStringMaxLength);
+maxString = undefined;
diff --git a/test/parallel/test-stringbytes-external-exceed-max-by-1.js b/test/parallel/test-stringbytes-external-exceed-max-by-1.js
@@ -10,7 +10,7 @@ const kStringMaxLength = process.binding('buffer').kStringMaxLength;
 try {
   new Buffer(kStringMaxLength * 3);
 } catch(e) {
-  assert.equal(e.message, 'Invalid array buffer length');
+  assert.equal(e.message, 'Invalid typed array length');
   console.log(
       '1..0 # Skipped: intensive toString tests due to memory confinements');
   return;

diff --git a/test/parallel/test-stringbytes-external-exceed-max-by-2.js b/test/parallel/test-stringbytes-external-exceed-max-by-2.js
@@ -10,7 +10,7 @@ const kStringMaxLength = process.binding('buffer').kStringMaxLength;
 try {
   new Buffer(kStringMaxLength * 3);
 } catch(e) {
-  assert.equal(e.message, 'Invalid array buffer length');
+  assert.equal(e.message, 'Invalid typed array length');
   console.log(
       '1..0 # Skipped: intensive toString tests due to memory confinements');
   return;

diff --git a/test/parallel/test-stringbytes-external-exceed-max.js b/test/parallel/test-stringbytes-external-exceed-max.js
@@ -10,7 +10,7 @@ const kStringMaxLength = process.binding('buffer').kStringMaxLength;
 try {
   new Buffer(kStringMaxLength * 3);
 } catch(e) {
-  assert.equal(e.message, 'Invalid array buffer length');
+  assert.equal(e.message, 'Invalid typed array length');
   console.log(
       '1..0 # Skipped: intensive toString tests due to memory confinements');
   return;