Commit 086f5bb
authored
CI: run unit tests on multiple Nodejs versions (#220)
* CI: run unit tests on multiple Nodejs versions
Signed-off-by: Levko Kravets <levko.ne@gmail.com>
* Temporarily exclude Nodejs@14
Signed-off-by: Levko Kravets <levko.ne@gmail.com>
* Collect reports for all Node versions separately
Signed-off-by: Levko Kravets <levko.ne@gmail.com>
---------
Signed-off-by: Levko Kravets <levko.ne@gmail.com>1 parent a4e4aa7 commit 086f5bb
2 files changed
Lines changed: 32 additions & 21 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -32,30 +32,39 @@ jobs: | |||
| 32 | 32 | ||
| 33 | 33 | unit-test: | |
| 34 | 34 | runs-on: ubuntu-latest | |
| 35 | + strategy: | ||
| 36 | + matrix: | ||
| 37 | + # only LTS versions starting from the lowest we support | ||
| 38 | + # TODO: Include Nodejs@14 | ||
| 39 | + node-version: ['16', '18', '20'] | ||
| 35 | 40 | env: | |
| 36 | 41 | cache-name: cache-node-modules | |
| 42 | + NYC_REPORT_DIR: coverage_unit_node${{ matrix.node-version }} | ||
| 37 | 43 | ||
| 38 | 44 | steps: | |
| 45 | + - uses: actions/setup-node@v4 | ||
| 46 | + with: | ||
| 47 | + node-version: ${{ matrix.node-version }} | ||
| 39 | 48 | - uses: actions/checkout@v3 | |
| 40 | 49 | - name: Cache node modules | |
| 41 | 50 | uses: actions/cache@v3 | |
| 42 | 51 | with: | |
| 43 | 52 | path: ~/.npm | |
| 44 | - key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }} | ||
| 53 | + key: ${{ runner.os }}-${{ matrix.node-version }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }} | ||
| 45 | 54 | restore-keys: | | |
| 46 | - ${{ runner.os }}-build-${{ env.cache-name }}- | ||
| 47 | - ${{ runner.os }}-build- | ||
| 48 | - ${{ runner.os }}- | ||
| 55 | + ${{ runner.os }}-${{ matrix.node-version }}-build-${{ env.cache-name }}- | ||
| 56 | + ${{ runner.os }}-${{ matrix.node-version }}-build- | ||
| 57 | + ${{ runner.os }}-${{ matrix.node-version }}- | ||
| 49 | 58 | - name: Run unit tests | |
| 50 | 59 | run: | | |
| 51 | 60 | npm ci | |
| 52 | 61 | npm run test | |
| 53 | - - run: tar -cvf coverage_unit.tar coverage_unit | ||
| 62 | + - run: tar -cvf ${{ env.NYC_REPORT_DIR }}.tar ${{ env.NYC_REPORT_DIR }} | ||
| 54 | 63 | - name: Store coverage report | |
| 55 | - uses: actions/upload-artifact@v3 | ||
| 64 | + uses: actions/upload-artifact@v4 | ||
| 56 | 65 | with: | |
| 57 | - name: coverage_unit | ||
| 58 | - path: coverage_unit.tar | ||
| 66 | + name: ${{ env.NYC_REPORT_DIR }} | ||
| 67 | + path: ${{ env.NYC_REPORT_DIR }}.tar | ||
| 59 | 68 | retention-days: 1 | |
| 60 | 69 | ||
| 61 | 70 | e2e-test: | |
@@ -67,6 +76,7 @@ jobs: | |||
| 67 | 76 | E2E_ACCESS_TOKEN: ${{ secrets.DATABRICKS_TOKEN }} | |
| 68 | 77 | E2E_TABLE_SUFFIX: ${{github.sha}} | |
| 69 | 78 | cache-name: cache-node-modules | |
| 79 | + NYC_REPORT_DIR: coverage_e2e | ||
| 70 | 80 | ||
| 71 | 81 | steps: | |
| 72 | 82 | - uses: actions/checkout@v3 | |
@@ -83,12 +93,12 @@ jobs: | |||
| 83 | 93 | run: | | |
| 84 | 94 | npm ci | |
| 85 | 95 | NODE_OPTIONS="--max-old-space-size=4096" npm run e2e | |
| 86 | - - run: tar -cvf coverage_e2e.tar coverage_e2e | ||
| 96 | + - run: tar -cvf ${{ env.NYC_REPORT_DIR }}.tar ${{ env.NYC_REPORT_DIR }} | ||
| 87 | 97 | - name: Store coverage report | |
| 88 | - uses: actions/upload-artifact@v3 | ||
| 98 | + uses: actions/upload-artifact@v4 | ||
| 89 | 99 | with: | |
| 90 | - name: coverage_e2e | ||
| 91 | - path: coverage_e2e.tar | ||
| 100 | + name: ${{ env.NYC_REPORT_DIR }} | ||
| 101 | + path: ${{ env.NYC_REPORT_DIR }}.tar | ||
| 92 | 102 | retention-days: 1 | |
| 93 | 103 | ||
| 94 | 104 | coverage: | |
@@ -108,14 +118,15 @@ jobs: | |||
| 108 | 118 | ${{ runner.os }}-build-${{ env.cache-name }}- | |
| 109 | 119 | ${{ runner.os }}-build- | |
| 110 | 120 | ${{ runner.os }}- | |
| 111 | - - uses: actions/download-artifact@v3 | ||
| 112 | - with: | ||
| 113 | - name: coverage_unit | ||
| 114 | - - uses: actions/download-artifact@v3 | ||
| 121 | + - uses: actions/download-artifact@v4 | ||
| 115 | 122 | with: | |
| 116 | - name: coverage_e2e | ||
| 117 | - - run: tar -xvf coverage_unit.tar && rm coverage_unit.tar | ||
| 118 | - - run: tar -xvf coverage_e2e.tar && rm coverage_e2e.tar | ||
| 123 | + pattern: coverage_* | ||
| 124 | + merge-multiple: true | ||
| 125 | + - name: Unpack coverage reports | ||
| 126 | + run: | | ||
| 127 | + ls -1 coverage_*.tar | xargs -I '{}' -- tar -xvf '{}' | ||
| 128 | + rm coverage_*.tar | ||
| 129 | + - run: ls -la | ||
| 119 | 130 | - name: Coverage | |
| 120 | 131 | uses: codecov/codecov-action@v3 | |
| 121 | 132 | with: | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -13,8 +13,8 @@ | |||
| 13 | 13 | }, | |
| 14 | 14 | "scripts": { | |
| 15 | 15 | "prepare": "npm run build", | |
| 16 | - "e2e": "nyc --reporter=lcov --report-dir=coverage_e2e mocha --config tests/e2e/.mocharc.js", | ||
| 17 | - "test": "nyc --reporter=lcov --report-dir=coverage_unit mocha --config tests/unit/.mocharc.js", | ||
| 16 | + "e2e": "nyc --reporter=lcov --report-dir=${NYC_REPORT_DIR:-coverage_e2e} mocha --config tests/e2e/.mocharc.js", | ||
| 17 | + "test": "nyc --reporter=lcov --report-dir=${NYC_REPORT_DIR:-coverage_unit} mocha --config tests/unit/.mocharc.js", | ||
| 18 | 18 | "update-version": "node bin/update-version.js && prettier --write ./lib/version.ts", | |
| 19 | 19 | "build": "npm run update-version && tsc", | |
| 20 | 20 | "watch": "tsc -w", | |
0 commit comments