chore(worflow): add test workflow (#125)

.github/workflows/tests.yml

@@ -0,0 +1,35 @@
+name: CI
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  tests:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node: ['12.x']
+    name: Node ${{ matrix.node }}
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v2
+      with:
+        fetch-depth: 0
+        persist-credentials: false
+        ref: ${{ github.event.pull_request.head.sha }}
+    - run: |
+        git remote set-branches --add origin main
+        git fetch
+    - name: Setup Node
+      uses: actions/setup-node@v1
+      with:
+        node-version: ${{ matrix.node }}
+    - name: Install Dependencies
+      run: yarn
+      env:
+        NODE_ENV: development
+    - name: Run Tests
+      run: yarn run test

examples/add-progress-bar/index.js

@@ -18,7 +18,7 @@ const AddProgressBar = () => (
               render={({ step, steps }) => (
                 <div>
                   <Line
-                    percent={(steps.indexOf(step) + 1) / steps.length * 100}
+                    percent={((steps.indexOf(step) + 1) / steps.length) * 100}
                     className="pad-b"
                   />
                   <TransitionGroup>

examples/skip-a-step/index.js

@@ -30,7 +30,7 @@ const SkipAStep = () => (
               render={({ step, steps }) => (
                 <div>
                   <Line
-                    percent={(steps.indexOf(step) + 1) / steps.length * 100}
+                    percent={((steps.indexOf(step) + 1) / steps.length) * 100}
                     className="pad-b"
                   />
                   <TransitionGroup>

package.json

@@ -63,12 +63,12 @@
     "jest": "^21.2.1",
     "node-sass": "^4.5.0",
     "prettier": "^1.6.1",
+    "rc-progress": "^2.2.5",
     "react": "^16.0.0",
     "react-dom": "^16.0.0",
     "react-router-dom": "^4.0.0",
     "react-test-renderer": "^16.0.0",
     "react-transition-group": "^2.2.1",
-    "rc-progress": "^2.2.5",
     "rimraf": "^2.5.2",
     "sass-loader": "^6.0.2",
     "style-loader": "^0.13.2",

src/components/Steps.js

@@ -45,6 +45,8 @@ Steps.defaultProps = {
 };
 
 Steps.contextTypes = {
+  // disabling due to lost context
+  // eslint-disable-next-line react/forbid-prop-types
   wizard: PropTypes.object,
 };
 

src/components/Wizard.js

@@ -17,6 +17,8 @@ import PropTypes from 'prop-types';
 import { createMemoryHistory } from 'history';
 import renderCallback from '../utils/renderCallback';
 
+// TODO: fix below
+/* eslint-disable no-undef */
 class Wizard extends Component {
   state = {
     step: {
@@ -77,8 +79,8 @@ class Wizard extends Component {
 
   pathToStep = pathname => {
     const id = pathname.replace(this.basename, '');
-    const [step] = this.state.steps.filter(
-      s => (this.props.exactMatch ? s.id === id : id.startsWith(s.id))
+    const [step] = this.state.steps.filter(s =>
+      this.props.exactMatch ? s.id === id : id.startsWith(s.id)
     );
 
     return step || this.state.step;
@@ -121,10 +123,14 @@ class Wizard extends Component {
 Wizard.propTypes = {
   basename: PropTypes.string,
   history: PropTypes.shape({
+    // disabling due to lost context
+    // eslint-disable-next-line react/forbid-prop-types
     entries: PropTypes.array,
     go: PropTypes.func,
     goBack: PropTypes.func,
     listen: PropTypes.func,
+    // disabling due to lost context
+    // eslint-disable-next-line react/forbid-prop-types
     location: PropTypes.object,
     push: PropTypes.func,
     replace: PropTypes.func,
@@ -142,6 +148,8 @@ Wizard.defaultProps = {
 };
 
 Wizard.childContextTypes = {
+  // disabling due to lost context
+  // eslint-disable-next-line react/forbid-prop-types
   wizard: PropTypes.object,
 };
 

src/components/createWizardComponent.js

@@ -19,6 +19,8 @@ const createWizardComponent = name => {
   const WizardComponent = (props, { wizard: { init, ...wizard } }) => renderCallback(props, wizard);
 
   WizardComponent.contextTypes = {
+    // disabling due to lost context
+    // eslint-disable-next-line react/forbid-prop-types
     wizard: PropTypes.object,
   };
 

src/withWizard.js

@@ -24,6 +24,8 @@ const withWizard = Component => {
     });
 
   WithWizard.contextTypes = {
+    // disabling due to lost context
+    // eslint-disable-next-line react/forbid-prop-types
     wizard: PropTypes.object,
   };
 

src/wizardShape.js

@@ -17,6 +17,8 @@ import PropTypes from 'prop-types';
 export default PropTypes.shape({
   go: PropTypes.func.isRequired,
   set: PropTypes.func.isRequired,
+  // disabling due to lost context
+  // eslint-disable-next-line react/forbid-prop-types
   history: PropTypes.object.isRequired,
   next: PropTypes.func.isRequired,
   previous: PropTypes.func.isRequired,