Upstream lukeed/klona state of the touched files before ba2e73a80c

src/full.js

@@ -10,7 +10,7 @@ export function klona(x) {
 
 	var i=0, k, list, tmp, str=Object.prototype.toString.call(x);
 
-	if (str === '[object Object]' || (x && (x.constructor === Object || x.constructor == null))) {
+	if (str === '[object Object]') {
 		tmp = Object.create(x.__proto__ || null);
 	} else if (str === '[object Array]') {
 		tmp = Array(x.length);

src/index.js

@@ -3,7 +3,7 @@ export function klona(x) {
 
 	var k, tmp, str=Object.prototype.toString.call(x);
 
-	if (str === '[object Object]' || (x && (x.constructor === Object || x.constructor == null))) {
+	if (str === '[object Object]') {
 		if (x.constructor !== Object && typeof x.constructor === 'function') {
 			tmp = new x.constructor();
 			for (k in x) {

src/json.js

@@ -7,7 +7,7 @@ export function klona(val) {
 		return out;
 	}
 
-	if (Object.prototype.toString.call(val) === '[object Object]' || (val && (val.constructor === Object || val.constructor == null))) {
+	if (Object.prototype.toString.call(val) === '[object Object]') {
 		out = {}; // null
 		for (k in val) {
 			if (k === '__proto__') {

src/lite.js

@@ -3,7 +3,7 @@ export function klona(x) {
 
 	var k, tmp, str=Object.prototype.toString.call(x);
 
-	if (str === '[object Object]' || (x && (x.constructor === Object || x.constructor == null))) {
+	if (str === '[object Object]') {
 		if (x.constructor !== Object && typeof x.constructor === 'function') {
 			tmp = new x.constructor();
 			for (k in x) {

test/full.js

@@ -13,7 +13,6 @@ suites.RegExps(klona);
 
 suites.Objects(klona);
 suites.Arrays(klona);
-suites.ToStringTag(klona);
 
 suites.Functions(klona);
 suites.Pollutions(klona);

test/index.js

@@ -13,7 +13,6 @@ suites.RegExps(klona);
 
 suites.Objects(klona);
 suites.Arrays(klona);
-suites.ToStringTag(klona);
 
 suites.Functions(klona);
 suites.Pollutions(klona);

test/json.js

@@ -10,6 +10,5 @@ suites.Nully(klona);
 
 suites.Objects(klona);
 suites.Arrays(klona);
-suites.ToStringTag(klona);
 
 suites.Pollutions(klona);

test/lite.js

@@ -13,7 +13,6 @@ suites.RegExps(klona);
 
 suites.Objects(klona);
 suites.Arrays(klona);
-suites.ToStringTag(klona);
 
 suites.Functions(klona);
 suites.Pollutions(klona);

test/suites/index.js

@@ -11,7 +11,6 @@ export { default as Sets } from './set';
 export { default as TypedArrays } from './typedarray';
 export { default as Descriptors } from './descriptor';
 export { default as Symbols } from './symbol';
-export { default as ToStringTag } from './tostringtag';
 
 export { default as Dates } from './date';
 export { default as RegExps } from './regexp';

test/suites/tostringtag.js

@@ -1,42 +0,0 @@
-import { suite } from 'uvu';
-import * as assert from 'assert';
-
-export default function (klona) {
-	const Spoofed = suite('Symbol.toStringTag spoofing');
-
-	// A `Symbol.toStringTag` klona special-cases (eg "Map") used to misroute a
-	// plain object into that branch and crash.
-	// @see https://github.com/lukeed/klona/issues/54
-	Spoofed('recognized tag :: plain object is cloned, not misrouted', () => {
-		const input = { a: 1, b: { c: 2 }, [Symbol.toStringTag]: 'Map' };
-		const output = klona(input);
-
-		assert.notEqual(output, input, 'must return a new object, not the input');
-		assert.equal(output.a, 1);
-		assert.deepEqual(output.b, { c: 2 });
-
-		output.a = 999;
-		output.b.c = 888;
-		assert.equal(input.a, 1, 'mutating output.a must not affect input');
-		assert.equal(input.b.c, 2, 'mutating output.b must not affect input');
-	});
-
-	// An unrecognized `Symbol.toStringTag` used to fall through every branch
-	// and be returned by reference instead of cloned.
-	// @see https://github.com/lukeed/klona/issues/54
-	Spoofed('unrecognized tag :: plain object is cloned, not returned by reference', () => {
-		const input = { a: 1, b: { c: 2 }, [Symbol.toStringTag]: 'Foobar' };
-		const output = klona(input);
-
-		assert.notEqual(output, input, 'must return a new object, not the input');
-		assert.equal(output.a, 1);
-		assert.deepEqual(output.b, { c: 2 });
-
-		output.a = 999;
-		output.b.c = 888;
-		assert.equal(input.a, 1, 'mutating output.a must not affect input');
-		assert.equal(input.b.c, 2, 'mutating output.b must not affect input');
-	});
-
-	Spoofed.run();
-}