fix(assertString): throw typeError on objects with constructor and name property

src/lib/util/assertString.js

@@ -1,4 +1,7 @@
 export default function assertString(input) {
   if (input === undefined || input === null) throw new TypeError(`Expected a string but received a ${input}`);
-  if (input.constructor.name !== 'String') throw new TypeError(`Expected a string but received a ${input.constructor.name}`);
+  if (typeof input !== 'string' && Object.prototype.toString.call(input) !== '[object String]') {
+    let inputConstructorName = input?.constructor?.name ? input.constructor.name : typeof input;
+    throw new TypeError(`Expected a string but received a ${inputConstructorName}`);
+  }
 }

test/util.test.js

@@ -54,11 +54,28 @@ describe('assertString', () => {
     assert.throws(() => { assertString([]); }, TypeError);
   });
 
+  it('Should throw an error if argument provided is an Object pretending to be a \'string\'', () => {
+    assert.throws(() => { assertString({ constructor: { name: 'string' } }); }, TypeError);
+  });
+
+  it('Should throw an error if argument provided is an Object pretending to be a \'String\'', () => {
+    assert.throws(() => { assertString({ constructor: { name: 'String' } }); }, TypeError);
+  });
+
   it('Should not throw an error if the argument is an empty string', () => {
     assert.doesNotThrow(() => { assertString(''); });
   });
 
   it('Should not throw an error if the argument is a String', () => {
     assert.doesNotThrow(() => { assertString('antidisestablishmentarianism'); });
   });
+
+  it('Should not throw an error if the argument is a boxed string', () => {
+    // eslint-disable-next-line no-new-wrappers
+    assert.doesNotThrow(() => { assertString(new String('antidisestablishmentarianism')); });
+  });
+
+  it('Should use typeof as fallback in error message when input has no constructor name', () => {
+    assert.throws(() => { assertString(Object.create(null)); }, /^TypeError: Expected a string but received a object$/);
+  });
 });