ci: refactor lint CI task and fix lint issues

.github/workflows/ci.yml

@@ -29,26 +29,27 @@ jobs:
         run: npm ci
       - name: CI Node Engine Check
         run: npm run ci:checkNodeEngine
-  # check-lint:
-  #   name: Lint
-  #   timeout-minutes: 15
-  #   runs-on: ubuntu-18.04
-  #   steps:
-  #     - uses: actions/checkout@v2
-  #     - name: Use Node.js ${{ env.NODE_VERSION }}
-  #       uses: actions/setup-node@v1
-  #       with:
-  #         node-version: ${{ env.node-version }}
-  #     - name: Cache Node.js modules
-  #       uses: actions/cache@v2
-  #       with:
-  #         path: ~/.npm
-  #         key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-${{ hashFiles('**/package-lock.json') }}
-  #         restore-keys: |
-  #           ${{ runner.os }}-node-${{ env.NODE_VERSION }}-
-  #     - name: Install dependencies
-  #       run: npm ci
-  #     - run: npm run lint
+  check-jest:
+    name: Lint
+    timeout-minutes: 15
+    runs-on: ubuntu-18.04
+    steps:
+      - uses: actions/checkout@v2
+      - name: Use Node.js ${{ env.NODE_VERSION }}
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ env.node-version }}
+      - name: Cache Node.js modules
+        uses: actions/cache@v2
+        with:
+          path: ~/.npm
+          key: ${{ runner.os }}-node-${{ env.NODE_VERSION }}-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-node-${{ env.NODE_VERSION }}-
+      - name: Install dependencies
+        run: npm ci
+      - name: Test with Jest
+        run: npm test
   check-circular:
      name: Circular Dependencies
      timeout-minutes: 5

jsconfig.json

@@ -0,0 +1,8 @@
+{
+  "typeAcquisition": {
+      "include": [
+          "jest"
+      ]
+  },
+  "exclude": ["node_modules"]
+}

package.json

@@ -88,6 +88,7 @@
     "@semantic-release/github": "7.2.3",
     "@semantic-release/npm": "7.1.3",
     "@semantic-release/release-notes-generator": "9.0.3",
+    "@types/jest": "^27.4.0",
     "all-node-versions": "8.0.0",
     "babel-eslint": "10.1.0",
     "babel-loader": "8.2.3",

release.config.js

@@ -26,7 +26,7 @@ async function config() {
   // Get branch
   const branch = ref.split('/').pop();
   console.log(`Running on branch: ${branch}`);
-  
+
   // Set changelog file
   const changelogFile = `./changelogs/CHANGELOG_${branch}.md`;
   console.log(`Changelog file output to: ${changelogFile}`);
@@ -86,7 +86,7 @@ async function config() {
       ['@semantic-release/github', {
         successComment: getReleaseComment(),
         labels: ['type:ci'],
-        releasedLabels: ['state:released<%= nextRelease.channel ? `-\${nextRelease.channel}` : "" %>']
+        releasedLabels: ['state:released<%= nextRelease.channel ? `-${nextRelease.channel}` : "" %>']
       }],
     ],
   };

src/components/Markdown/Markdown.react.js

@@ -6,7 +6,7 @@
  * the root directory of this source tree.
  */
 import CodeSnippet    from 'components/CodeSnippet/CodeSnippet.react';
-import marked         from 'marked';
+import { marked }         from 'marked';
 import PropTypes      from 'lib/PropTypes';
 import React          from 'react';
 import ReactDOMServer from 'react-dom/server';

src/components/Pill/Pill.react.js

@@ -7,15 +7,15 @@
  */
 import React  from 'react';
 import styles from 'components/Pill/Pill.scss';
-import Icon from "components/Icon/Icon.react";
+import Icon from 'components/Icon/Icon.react';
 
 //TODO: refactor, may want to move onClick outside or need to make onClick able to handle link/button a11y
 let Pill = ({ value, onClick, fileDownloadLink, followClick = false, shrinkablePill = false }) => (
   <span
     className={[
       styles.pill,
       !followClick && onClick ? styles.action : void 0
-    ].join(" ")}
+    ].join(' ')}
     onClick={!followClick && onClick ? onClick : null}
   >
     <span className={!followClick && fileDownloadLink ? styles.content : shrinkablePill ? styles.pillText : ''}>{value}</span>

src/components/PushPreview/PushPreview.react.js

@@ -5,8 +5,6 @@
  * This source code is licensed under the license found in the LICENSE file in
  * the root directory of this source tree.
  */
-import PropTypes          from 'lib/PropTypes';
-import ParseApp           from 'lib/ParseApp';
 import React              from 'react';
 import SegmentSelect      from 'components/SegmentSelect/SegmentSelect.react';
 import styles             from 'components/PushPreview/PushPreview.scss';

src/components/Sidebar/Pin.react.js

@@ -1,7 +1,7 @@
-import React from "react";
+import React from 'react';
 
-import Icon from "components/Icon/Icon.react";
-import styles from "components/Sidebar/Sidebar.scss";
+import Icon from 'components/Icon/Icon.react';
+import styles from 'components/Sidebar/Sidebar.scss';
 
 const Pin = ({ onClick }) => (
   <div className={styles.pinContainer} onClick={onClick}>

src/dashboard/Data/Browser/Browser.react.js

@@ -401,7 +401,7 @@ class Browser extends DashboardView {
           if (name === 'objectId' || this.state.isUnique && name !== this.state.uniqueField) {
             return;
           }
-          if (!!required) {
+          if (required) {
             requiredCols.push(name);
           }
           if (className === '_User' && (name === 'username' || name === 'password')) {
@@ -418,7 +418,7 @@ class Browser extends DashboardView {
       for (let idx = 0; idx < requiredCols.length; idx++) {
         const name = requiredCols[idx];
         if (!obj.get(name)) {
-          this.showNote("Please enter all required fields", true);
+          this.showNote('Please enter all required fields', true);
           this.setState({
             markRequiredFieldRow: -1
           });
@@ -460,7 +460,7 @@ class Browser extends DashboardView {
               if (msg) {
                 msg = msg[0].toUpperCase() + msg.substr(1);
               }
-              obj.set(attr, prev);
+              obj.revert();
               this.setState({ data: this.state.data });
               this.showNote(msg, true);
             }
@@ -505,7 +505,7 @@ class Browser extends DashboardView {
           if (name === 'objectId' || this.state.isUnique && name !== this.state.uniqueField) {
             return;
           }
-          if (!!required) {
+          if (required) {
             requiredCols.push(name);
           }
           if (className === '_User' && (name === 'username' || name === 'password')) {
@@ -522,7 +522,7 @@ class Browser extends DashboardView {
       for (let idx = 0; idx < requiredCols.length; idx++) {
         const name = requiredCols[idx];
         if (!obj.get(name)) {
-          this.showNote("Please enter all required fields", true);
+          this.showNote('Please enter all required fields', true);
           this.setState({
             markRequiredFieldRow: rowIndex
           });

src/dashboard/Data/Browser/BrowserToolbar.react.js

@@ -326,7 +326,7 @@ let BrowserToolbar = ({
       {editCloneRows && editCloneRows.length > 0 && (
         <BrowserMenu title="Clone" icon="clone-icon">
           <MenuItem
-            text={"Cancel all pending rows"}
+            text={'Cancel all pending rows'}
             onClick={onCancelPendingEditRows}
           />
         </BrowserMenu>

src/dashboard/Data/Browser/DataBrowser.react.js

@@ -257,6 +257,7 @@ export default class DataBrowser extends React.Component {
   getNextVisibleColumnIndex(distance = 1, min = 0, max = 0) {
     if (distance === 0) { return this.state.current.col; }
     let newIndex = this.state.current.col + distance;
+    // eslint-disable-next-line no-constant-condition
     while (true) {
       if (this.state.order[newIndex]?.visible) { return newIndex; }
       if (newIndex <= min) { return min; }

src/dashboard/Data/Browser/ExportSelectedRowsDialog.react.js

@@ -27,7 +27,7 @@ export default class ExportSelectedRowsDialog extends React.Component {
       <Modal
         type={Modal.Types.INFO}
         icon='warn-outline'
-        title={this.props.selection['*'] ? 'Export all rows?' : (selectionLength === 1 ? `Export 1 selected row?` : `Export ${selectionLength} selected rows?`)}
+        title={this.props.selection['*'] ? 'Export all rows?' : (selectionLength === 1 ? 'Export 1 selected row?' : `Export ${selectionLength} selected rows?`)}
         subtitle={this.props.selection['*'] ? 'Note: Exporting is limited to the first 10,000 rows.' : ''}
         disabled={!this.valid()}
         confirmText={'Yes export'}

src/lib/tests/Button.test.js

@@ -14,7 +14,7 @@ const Button = require('../../components/Button/Button.react').default;
 describe('Button', () => {
   it('has a default state', () => {
     const component = renderer.create(<Button value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable');
     expect(component.children[0].type).toBe('span');
     expect(component.children[0].children[0]).toBe('A button');
@@ -23,43 +23,43 @@ describe('Button', () => {
   it('can be primary', () => {
     const component = renderer.create(<Button primary={true} value='A button' />).toJSON();
 
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable primary');
   });
 
   it('can be colored', () => {
     const component = renderer.create(<Button color='red' value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable red');
   });
 
   it('can be colored and primary', () => {
     const component = renderer.create(<Button color='red' primary={true} value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable primary red');
   });
 
   it('can be disabled', () => {
     const component = renderer.create(<Button color='red' disabled={true} value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable disabled');
   });
 
   it('special-cases white disabled buttons', () => {
     const component = renderer.create(<Button color='white' disabled={true} value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable disabled white');
   });
 
   it('can indidate progress', () => {
     const component = renderer.create(<Button progress={true} value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.className).toBe('button unselectable progress');
   });
 
   it('can override width', () => {
     const component = renderer.create(<Button width='300px' value='A button' />).toJSON();
-    expect(component.type).toBe('a');
+    expect(component.type).toBe('button');
     expect(component.props.style.width).toBe('300px');
   });
 });