fix: github actions (#4014)

Description --- Fix integration tests on github actions. They were running but not reporting. Fix the coverage setup. How Has This Been Tested? --- I've tested non-critical tests only.
tari-project · Apr 11, 2022 · a03392e · a03392e
1 parent 4812261
commit a03392e
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 13 deletions.
diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -3,6 +3,10 @@ on:
   push:
     branches:
       - development
+
+env:
+  toolchain: nightly-2021-11-20
+
 jobs:
   coverage:
     name: test
@@ -19,17 +23,30 @@ jobs:
         run: |
           sudo apt-get update && \
           sudo apt-get -y install \
-          build-essential \
-          libgtk-3-dev \
-          libwebkit2gtk-4.0-dev \
-          libsoup2.4-dev \
-          curl \
-          wget \
-          libappindicator3-dev \
-          patchelf \
-          librsvg2-dev \
-          libprotobuf-dev \
-          protobuf-compiler
+            openssl \
+            libssl-dev \
+            pkg-config \
+            libsqlite3-dev \
+            clang-10 \
+            clang \
+            git \
+            cmake \
+            libc++-dev \
+            libc++abi-dev \
+            libprotobuf-dev \
+            protobuf-compiler \
+            libncurses5-dev \
+            libncursesw5-dev \
+            zip \
+            build-essential \
+            libgtk-3-dev \
+            libwebkit2gtk-4.0-dev \
+            libsoup2.4-dev \
+            curl \
+            wget \
+            libappindicator3-dev \
+            patchelf \
+            librsvg2-dev \
       - name: test key manager wasm
         run: |
           npm install -g wasm-pack
@@ -50,7 +67,6 @@ jobs:
           command: test
           args: --all-features
       - name: generate coverage report
-        usesid: coverage
         run: |
           grcov . -s . --binary-path ./target/debug -t lcov --branch --ignore-not-existing -o ./target/report.lcov
       - name: Coveralls upload

diff --git a/.github/workflows/long_running.yml b/.github/workflows/long_running.yml
@@ -1,5 +1,5 @@
 # Runs weekly (saturday noon)
-name: Non critical integration tests
+name: Long running integration tests
 on:
   schedule:
     - cron: "0 12 * * 6"
@@ -80,9 +80,11 @@ jobs:
       - name: Run integration tests
         run: cd integration_tests && mkdir -p cucumber_output && node_modules/.bin/cucumber-js --profile "long-running" --tags "not @wallet-ffi"  --format json:cucumber_output/tests.cucumber --exit --retry 2 --retry-tag-filter "@flaky and not @broken"
       - name: Generate report
+        if: always()
         run: cd integration_tests && node ./generate_report.js
       - name: Store test results
         uses: actions/upload-artifact@v3
+        if: always()
         with:
           name: test results
           path: |

diff --git a/.github/workflows/non_critical_integration_tests.yml b/.github/workflows/non_critical_integration_tests.yml
@@ -80,9 +80,11 @@ jobs:
       - name: Run integration tests
         run: cd integration_tests && mkdir -p cucumber_output && node_modules/.bin/cucumber-js --profile "non-critical" --tags "not @wallet-ffi"  --format json:cucumber_output/tests.cucumber --exit --retry 2 --retry-tag-filter "@flaky and not @broken"
       - name: Generate report
+        if: always()
         run: cd integration_tests && node ./generate_report.js
       - name: Store test results
         uses: actions/upload-artifact@v3
+        if: always()
         with:
           name: test results
           path: |