✅ [ci] Add CI integration test and boyscout fixes.

Signed-off-by: Oliver Tale-Yazdi <oliver@perun.network>

.github/workflows/ci.yml

@@ -37,7 +37,10 @@ jobs:
         go build -o op ./cmd/operator
         go build -o cl ./cmd/client
 
-    - name: Test
+    - name: Unit Tests
       run: go test -timeout 60s ./...
+
+    - name: Integration Test
+      run: go test -tags=integration -timeout 600s -v ./client -run TestWalkthroughs
       env:
         ERDSTALL_GANACHE_CMD: docker run --rm --net=host trufflesuite/ganache-cli

eth/client.go

@@ -171,6 +171,9 @@ func (cl *Client) SubscribeEpochs(ctx context.Context, params tee.Parameters, si
 	for {
 		select {
 		case block := <-sub.Blocks():
+			if block == nil {
+				return nil
+			}
 			blockSink <- block.NumberU64()
 			if newEpoch := params.DepositEpoch(block.NumberU64()); newEpoch > oldEpoch {
 				sink <- newEpoch

operator/rpcoperator.go

@@ -3,6 +3,7 @@
 package operator
 
 import (
+	"math/big"
 	"sync"
 
 	"github.com/ethereum/go-ethereum/common"
@@ -49,6 +50,7 @@ func NewRPCOperator(enclave tee.Enclave) *RPCOperator {
 func (o *RPCOperator) Send(tx tee.Transaction) error {
 	o.mtx.Lock()
 	defer o.mtx.Unlock()
+	log.Infof("Sending %d WEI 0x%s…->0x%s…", (*big.Int)(tx.Amount).Uint64(), tx.Sender.Hex()[:5], tx.Recipient.Hex()[:5])
 	return o.enclave.ProcessTXs(&tx)
 }