test(utilities): refactor tests

Author: remarkablemark

test/utilities.test.js

@@ -5,91 +5,89 @@ const {
   isCustomComponent
 } = require('../lib/utilities');
 
-describe('utilities', () => {
-  describe('invertObject', () => {
-    it.each([undefined, null, 'string', 0, 1, () => {}])(
-      `throws an error if the first argument is %p`,
-      input => {
-        expect(() => {
-          invertObject(input);
-        }).toThrow(TypeError);
-      }
-    );
+describe('invertObject', () => {
+  it.each([undefined, null, '', 'test', 0, 1, true, false, () => {}])(
+    'throws error for value: %p',
+    value => {
+      expect(() => {
+        invertObject(value);
+      }).toThrow(TypeError);
+    }
+  );
 
-    it('swaps key with value', () => {
-      expect(
-        invertObject({
-          foo: 'bar',
-          baz: 'qux'
-        })
-      ).toEqual({
-        bar: 'foo',
-        qux: 'baz'
-      });
+  it('swaps key with value', () => {
+    expect(
+      invertObject({
+        foo: 'bar',
+        baz: 'qux'
+      })
+    ).toEqual({
+      bar: 'foo',
+      qux: 'baz'
     });
+  });
 
-    it('swaps key with value if value is string', () => {
+  it('swaps key with value if value is string', () => {
+    expect(
+      invertObject({
+        $: 'dollar',
+        _: 'underscore',
+        num: 1,
+        u: undefined,
+        n: null
+      })
+    ).toEqual({
+      dollar: '$',
+      underscore: '_'
+    });
+  });
+
+  describe('options', () => {
+    it('applies override if provided', () => {
       expect(
-        invertObject({
-          $: 'dollar',
-          _: 'underscore',
-          num: 1,
-          u: undefined,
-          n: null
+        invertObject({ foo: 'bar', baz: 'qux' }, key => {
+          if (key === 'foo') {
+            return ['key', 'value'];
+          }
         })
-      ).toEqual({
-        dollar: '$',
-        underscore: '_'
-      });
+      ).toEqual({ key: 'value', qux: 'baz' });
     });
 
-    describe('options', () => {
-      it('applies override if provided', () => {
-        expect(
-          invertObject({ foo: 'bar', baz: 'qux' }, key => {
-            if (key === 'foo') {
-              return ['key', 'value'];
-            }
-          })
-        ).toEqual({ key: 'value', qux: 'baz' });
-      });
-
-      it('does not apply override if invalid', () => {
-        expect(
-          invertObject({ foo: 'bar', baz: 'qux' }, key => {
-            if (key === 'foo') {
-              return ['key'];
-            } else if (key === 'baz') {
-              return { key: 'value' };
-            }
-          })
-        ).toEqual({ bar: 'foo', qux: 'baz' });
-      });
+    it('does not apply override if invalid', () => {
+      expect(
+        invertObject({ foo: 'bar', baz: 'qux' }, key => {
+          if (key === 'foo') {
+            return ['key'];
+          } else if (key === 'baz') {
+            return { key: 'value' };
+          }
+        })
+      ).toEqual({ bar: 'foo', qux: 'baz' });
     });
   });
+});
 
-  describe('isCustomComponent', () => {
-    it('returns true if the tag contains a hyphen and is not in the whitelist', () => {
-      expect(isCustomComponent('my-custom-element')).toBe(true);
-    });
+describe('isCustomComponent', () => {
+  it('returns true if the tag contains a hyphen and is not in the whitelist', () => {
+    expect(isCustomComponent('my-custom-element')).toBe(true);
+  });
 
-    it('returns false if the tag is in the whitelist', () => {
-      expect(isCustomComponent('annotation-xml')).toBe(false);
-      expect(isCustomComponent('color-profile')).toBe(false);
-      expect(isCustomComponent('font-face')).toBe(false);
-    });
+  it('returns false if the tag is in the whitelist', () => {
+    expect(isCustomComponent('annotation-xml')).toBe(false);
+    expect(isCustomComponent('color-profile')).toBe(false);
+    expect(isCustomComponent('font-face')).toBe(false);
+  });
 
-    it('returns true if the props contains an `is` key', () => {
-      expect(isCustomComponent('button', { is: 'custom-button' })).toBe(true);
-    });
+  it('returns true if the props contains an `is` key', () => {
+    expect(isCustomComponent('button', { is: 'custom-button' })).toBe(true);
   });
+});
 
-  describe('PRESERVE_CUSTOM_ATTRIBUTES', () => {
-    const isReactGreaterThan15 =
-      parseInt(React.version.match(/^\d./)[0], 10) >= 16;
+describe('PRESERVE_CUSTOM_ATTRIBUTES', () => {
+  const isReactGreaterThan15 =
+    parseInt(React.version.match(/^\d./)[0], 10) >= 16;
 
-    it(`is ${isReactGreaterThan15} when React.version="${React.version}"`, () => {
-      expect(PRESERVE_CUSTOM_ATTRIBUTES).toBe(isReactGreaterThan15);
-    });
+  it(`is ${isReactGreaterThan15} when React.version="${React.version}"`, () => {
+    expect(PRESERVE_CUSTOM_ATTRIBUTES).toBe(isReactGreaterThan15);
   });
 });