[Fix] node 0.8: an object arg to a TA only throws a RangeError when…

… it is an ArrayBuffer of an incompatible byte length
inspect-js · Mar 2, 2023 · d5108f6 · d5108f6
1 parent 133732e
commit d5108f6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/index.js b/index.js
@@ -35,7 +35,7 @@ module.exports = $byteLength || $abSlice
 			try {
 				return (new $Float32Array(obj)).buffer === obj && !isTypedArray(obj);
 			} catch (e) {
-				return false;
+				return typeof obj === 'object' && e.name === 'RangeError';
 			}
 		}
 		: function isArrayBuffer(obj) { // eslint-disable-line no-unused-vars

diff --git a/test/index.js b/test/index.js
@@ -21,9 +21,11 @@ test('isArrayBuffer', function (t) {
 
 	t.test('actual ArrayBuffer instances', { skip: typeof ArrayBuffer === 'undefined' }, function (st) {
 		var ab = new ArrayBuffer();
-
 		st.equal(isArrayBuffer(ab), true, inspect(ab) + ' is an ArrayBuffer');
 
+		var ab42 = new ArrayBuffer(42);
+		st.equal(isArrayBuffer(ab42), true, inspect(ab42) + ' is an ArrayBuffer');
+
 		st.end();
 	});