From: Elias Rohrer <dev@tnull.de>
Date: Wed, 14 Aug 2024 07:22:13 +0000 (+0200)
Subject: Disallow skipping tx-sync tests in CI
X-Git-Tag: v0.0.124-beta~7^2
X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=commitdiff_plain;h=43bc78ce3da87cd0bdd90272c692ff505834f0cf;p=rust-lightning

Disallow skipping tx-sync tests in CI

Previously, we'd always skip tx-sync tests if the
`BITCOIND_EXE`/`ELECTRS_EXE` environment variables would be unset. While
this is especially fine for local testing, we still want to enforce
tests failing if somehow the `bitcoind`/`electrs` downloading or caching
in CI stops working. Here, we therefore add a `CI_ENV` variable that
indicates we're indeed running in CI, and only skip if it's unset.
---

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index d15932b5c..9ed1b5eee 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -66,7 +66,7 @@ jobs:
             echo "ELECTRS_EXE=$( pwd )/bin/electrs-${{ runner.os }}-${{ runner.arch }}" >> "$GITHUB_ENV"
       - name: Run CI script
         shell: bash # Default on Winblows is powershell
-        run: CI_MINIMIZE_DISK_USAGE=1 ./ci/ci-tests.sh
+        run: CI_ENV=1 CI_MINIMIZE_DISK_USAGE=1 ./ci/ci-tests.sh
 
   coverage:
     strategy:
diff --git a/ci/ci-tests.sh b/ci/ci-tests.sh
index 74bf427d6..43d6aeaa4 100755
--- a/ci/ci-tests.sh
+++ b/ci/ci-tests.sh
@@ -44,22 +44,24 @@ cargo check --verbose --color always --features rpc-client,rest-client,tokio
 popd
 
 if [[ "$HOST_PLATFORM" != *windows* ]]; then
-	if [ -z "$BITCOIND_EXE" ] || [ -z "$ELECTRS_EXE" ]; then
+	pushd lightning-transaction-sync
+	echo -e "\n\nChecking Transaction Sync Clients with features."
+	cargo check --verbose --color always --features esplora-blocking
+	cargo check --verbose --color always --features esplora-async
+	cargo check --verbose --color always --features esplora-async-https
+	cargo check --verbose --color always --features electrum
+
+	if [ -z "$CI_ENV" ] && [[ -z "$BITCOIND_EXE" || -z "$ELECTRS_EXE" ]]; then
 		echo -e "\n\nSkipping testing Transaction Sync Clients due to BITCOIND_EXE or ELECTRS_EXE being unset."
+		cargo check --tests
 	else
-		echo -e "\n\nBuilding and testing Transaction Sync Clients with features"
-		pushd lightning-transaction-sync
-
+		echo -e "\n\nTesting Transaction Sync Clients with features."
 		cargo test --verbose --color always --features esplora-blocking
-		cargo check --verbose --color always --features esplora-blocking
 		cargo test --verbose --color always --features esplora-async
-		cargo check --verbose --color always --features esplora-async
 		cargo test --verbose --color always --features esplora-async-https
-		cargo check --verbose --color always --features esplora-async-https
 		cargo test --verbose --color always --features electrum
-		cargo check --verbose --color always --features electrum
-		popd
 	fi
+	popd
 fi
 
 echo -e "\n\nTest futures builds"