Merge pull request #35 from node-oauth/feature-ci-multiple-targets

feature: ci with multiple targets

Author: HappyZombies

.github/workflows/tests.yml

@@ -39,26 +39,27 @@ jobs:
     runs-on: ubuntu-latest
     # uncomment when a linter is added
     # needs: [lintjs]
+    strategy:
+      matrix:
+        node: [12, 14, 16]
     steps:
-    - name: checkout
+    - name: Checkout ${{ matrix.node }}
       uses: actions/checkout@v2
 
-    - name: setup node
+    - name: Setup node ${{ matrix.node }}
       uses: actions/setup-node@v1
       with:
-        node-version: '12.x'
+        node-version: ${{ matrix.node }}
 
-    - name: cache dependencies
+    - name: Cache dependencies ${{ matrix.node }}
       uses: actions/cache@v1
       with:
         path: ~/.npm
-        key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
+        key: ${{ runner.os }}-node-${{ matrix.node }}-${{ hashFiles('**/package-lock.json') }}
         restore-keys: |
-          ${{ runner.os }}-node-
+          ${{ runner.os }}-node-${{ matrix.node }}
     - run: npm ci
-    - run: npm run test
-    # uncomment when coverage is added
-    # - run: npm run test:coverage
+    - run: npm run test:coverage
 
     # ----------------------------------
     # uncomment when a linter is added