Disable sync test on ubuntu CI runners

They don't seem to have DNS or network configured during the tests. Also make capitalisation of step names consistent.
ZcashFoundation · Oct 19, 2020 · 5dc0d46 · 5dc0d46
1 parent 2208702
commit 5dc0d46
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -22,9 +22,13 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: fetch
-      - name: install LLVM on Windows
+      - name: Install LLVM on Windows
         if: matrix.os == 'windows-latest'
         run: choco install llvm -y
+      - name: Skip network tests as needed
+        # Some runners don't have network or DNS configured during test steps
+        if: matrix.os == 'ubuntu-latest'
+        run: echo "ZEBRA_SKIP_NETWORK_TESTS=1" >> $GITHUB_ENV
       - name: Run tests
         env:
           RUST_BACKTRACE: full
@@ -67,7 +71,7 @@ jobs:
         with:
           toolchain: stable
           override: true
-      - name: install LLVM on Windows
+      - name: Install LLVM on Windows
         if: matrix.os == 'windows-latest'
         run: choco install llvm -y
       - name: cargo fetch

diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -20,6 +20,10 @@ jobs:
         with:
           toolchain: stable
           override: true
+      - name: Skip network tests as needed
+        # Some runners don't have network or DNS configured during test steps
+        if: matrix.os == 'ubuntu-latest'
+        run: echo "ZEBRA_SKIP_NETWORK_TESTS=1" >> $GITHUB_ENV
       - name: Run cargo-tarpaulin
         uses: actions-rs/[email protected]
         with: