Merge pull request #1 from alleyinteractive/test-cleanup

Test cleanup

.travis.yml

@@ -0,0 +1,8 @@
+language: node_js
+cache: yarn
+node_js:
+  - node
+  - "12"
+  - "10"
+install:
+  - YARN_IGNORE_ENGINES=true yarn

package.json

@@ -16,7 +16,7 @@
   "scripts": {
     "lint": "eslint .",
     "test": "jest",
-    "test:watch": "jest --watch"
+    "test:watch": "jest --watchAll"
   },
   "devDependencies": {
     "@babel/core": "^7.8.4",

src/getComponentClassnames.spec.js

@@ -21,19 +21,19 @@ describe('Get a hashed classname from a component', () => {
       },
     ],
   ])(
-    'Gets the `%s` classname from `%s`',
+    'Gets the `%s` component',
     (componentName, expected) => {
       expect(getComponentClassnames(componentName)).toEqual(expected);
     }
   );
 });
 
-describe('Fail to find a nonexistent classname', () => {
+describe('Fail to find a nonexistent component', () => {
   test.each([
     ['mock-three'],
     ['foo-bar'],
   ])(
-    'The `%s` classname does not exist in `%s`',
+    'Ignores non-existent component `%s`',
     (componentName) => {
       expect(getComponentClassnames(componentName)).toMatchObject({});
     }

src/getComponentSelectors.spec.js

@@ -1,7 +1,7 @@
 /* eslint-disable max-len */
 import getComponentSelectors from './getComponentSelectors';
 
-describe('Get the formatted classnames for a component', () => {
+describe('Get the class selectors for a component', () => {
   test.each([
     ['mock-one', {
       alignleft: '.mock-one__alignleft___1cXIA',
@@ -15,7 +15,7 @@ describe('Get the formatted classnames for a component', () => {
       sharedName: '.mock-two__shared-name___Kf9x0',
     }],
   ])(
-    'Gets formatted classnames for %s',
+    'Gets class selectors from `%s`',
     (componentName, expected) => {
       expect(getComponentSelectors(componentName)).toEqual(expected);
     }

src/getElementFromComponent.spec.js

@@ -14,8 +14,8 @@ const elements = {
   },
 };
 
-describe('Test getting an element from a component', () => {
-  test('Getting elements from the available components', () => {
+describe('Test getting elements from a component', () => {
+  test('Get an element from a component', () => {
     const mockOneAlignLeft = getElementFromComponent('alignleft', 'mock-one');
     expect(mockOneAlignLeft).toEqual(elements.mockOne.alignleft);
     expect(mockOneAlignLeft).not.toEqual(elements.mockTwo.alignleft);
@@ -30,7 +30,7 @@ describe('Test getting an element from a component', () => {
     expect(firstShared).toEqual(elements.mockTwo.sharedFirst);
   });
 
-  test('Getting all elements from a component', () => {
+  test('Get all elements from a component', () => {
     const allShared = getElementFromComponent('shared-name', 'mock-two', true);
     expect(allShared).toEqual(elements.mockTwo.shared);
     expect(allShared).toBeInstanceOf(NodeList);

src/getSelectorFromComponent.spec.js

@@ -1,14 +1,14 @@
 /* eslint-disable max-len */
 import getSelectorFromComponent from './getSelectorFromComponent';
 
-describe('Get a hashed classname from a component', () => {
+describe('Get a hashed class selector from a component', () => {
   test.each([
     ['alignleft', 'mock-one', '.mock-one__alignleft___1cXIA'],
     ['alignleft', 'mock-two', '.mock-two__alignleft___Lww3s'],
     ['title', 'mock-one', '.mock-one__title___2kGA7 _typography__header-main___2IME8'],
     ['shared-name', 'mock-two', '.mock-two__shared-name___Kf9x0'],
   ])(
-    'Gets the `%s` classname from `%s`',
+    'Gets the `%s` selector from `%s`',
     (className, componentName, expected) => {
       expect(getSelectorFromComponent(className, componentName)).toEqual(expected);
     }

src/util/kebabToCamelCase.spec.js

@@ -5,7 +5,7 @@ describe('camelCase a kebab-case string', () => {
     ['mock-one', 'mockOne'],
     ['menu-item-link', 'menuItemLink'],
   ])(
-    'Gets the `%s` classname from `%s`',
+    'camelCases `%s` as `%s`',
     (componentName, expected) => {
       expect(kebabToCamelCase(componentName)).toEqual(expected);
     }
@@ -18,7 +18,7 @@ describe('Ignore non-kebab-case strings', () => {
     ['fooBar'],
     ['alignleft'],
   ])(
-    'The `%s` classname does not exist in `%s`',
+    'Ignores `%s`',
     (componentName) => {
       expect(kebabToCamelCase(componentName)).toEqual(componentName);
     }