Make it work better in the browser (#27)

Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>

index.js

@@ -1,11 +1,21 @@
 'use strict';
-const {inspect} = require('util');
 
 class NonError extends Error {
 	constructor(message) {
-		super(inspect(message));
+		super(NonError._prepareSuperMessage(message));
 		this.name = 'NonError';
-		Error.captureStackTrace(this, NonError);
+
+		if (Error.captureStackTrace) {
+			Error.captureStackTrace(this, NonError);
+		}
+	}
+
+	static _prepareSuperMessage(message) {
+		try {
+			return JSON.stringify(message);
+		} catch (_) {
+			return String(message);
+		}
 	}
 }
 

test.js

@@ -1,6 +1,13 @@
 import test from 'ava';
 import {serializeError, deserializeError} from '.';
 
+function deserializeNonError(t, value) {
+	const deserialized = deserializeError(value);
+	t.true(deserialized instanceof Error);
+	t.is(deserialized.constructor.name, 'NonError');
+	t.is(deserialized.message, JSON.stringify(value));
+}
+
 test('main', t => {
 	const serialized = serializeError(new Error('foo'));
 	const keys = Object.keys(serialized);
@@ -115,27 +122,29 @@ test('should handle top-level null values', t => {
 });
 
 test('should deserialize null', t => {
-	const deserialized = deserializeError(null);
-	t.true(deserialized instanceof Error);
-	t.is(deserialized.message, 'null');
+	deserializeNonError(t, null);
 });
 
 test('should deserialize number', t => {
-	const deserialized = deserializeError(1);
-	t.true(deserialized instanceof Error);
-	t.is(deserialized.message, '1');
+	deserializeNonError(t, 1);
 });
 
-test('should deserialize error', t => {
-	const deserialized = deserializeError(new Error('test'));
-	t.true(deserialized instanceof Error);
-	t.is(deserialized.message, 'test');
+test('should deserialize boolean', t => {
+	deserializeNonError(t, true);
+});
+
+test('should deserialize string', t => {
+	deserializeNonError(t, '123');
 });
 
 test('should deserialize array', t => {
-	const deserialized = deserializeError([1]);
+	deserializeNonError(t, [1]);
+});
+
+test('should deserialize error', t => {
+	const deserialized = deserializeError(new Error('test'));
 	t.true(deserialized instanceof Error);
-	t.is(deserialized.message, '[ 1 ]');
+	t.is(deserialized.message, 'test');
 });
 
 test('should deserialize and preserve existing properties', t => {