Skip to content

Commit f446230

Browse files
DA Client remove interface: Part 2.5, create e2e test to validate that a blob is posted in DA layer. (#2920)
Create e2e test to validate that a blob is posted into DA. This will verify that once the final refactor is done it still continue working with the new client implementation. <!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> --------- Co-authored-by: tac0turtle <marko@baricevic.me>
1 parent 7808c5a commit f446230

File tree

7 files changed

+397
-20
lines changed

7 files changed

+397
-20
lines changed

.github/workflows/test.yml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,17 @@ jobs:
7676
uses: actions/setup-go@v6
7777
with:
7878
go-version-file: ./go.mod
79+
- name: Set up Docker Buildx
80+
uses: docker/setup-buildx-action@v3
81+
- name: Build evstack:local-dev (cached)
82+
uses: docker/build-push-action@v6
83+
with:
84+
context: .
85+
file: apps/testapp/Dockerfile
86+
load: true
87+
tags: evstack:local-dev
88+
cache-from: type=gha
89+
cache-to: type=gha,mode=max
7990
- name: E2E Tests
8091
run: make test-e2e
8192

scripts/build.mk

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ build-da:
6060
## docker-build: Build Docker image for local testing
6161
docker-build:
6262
@echo "--> Building Docker image for local testing"
63-
@docker build -t evstack:local-dev .
63+
@docker build -t evstack:local-dev -f apps/testapp/Dockerfile .
6464
@echo "--> Docker image built: evstack:local-dev"
6565
@echo "--> Checking if image exists locally..."
6666
@docker images evstack:local-dev

scripts/test.mk

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ test-integration:
2222
.PHONY: test-integration
2323

2424
## test-e2e: Running e2e tests
25-
test-e2e: build build-da build-evm
25+
test-e2e: build build-da build-evm docker-build-if-local
2626
@echo "--> Running e2e tests"
2727
@cd test/e2e && go test -mod=readonly -failfast -timeout=15m -tags='e2e evm' ./... --binary=../../build/testapp --evm-binary=../../build/evm
2828
.PHONY: test-e2e
@@ -69,8 +69,12 @@ test-docker-upgrade-e2e:
6969
## docker-build-if-local: Build Docker image if using local repository
7070
docker-build-if-local:
7171
@if [ -z "$(EV_NODE_IMAGE_REPO)" ] || [ "$(EV_NODE_IMAGE_REPO)" = "evstack" ]; then \
72-
echo "--> Local repository detected, building Docker image..."; \
73-
$(MAKE) docker-build; \
72+
if docker image inspect evstack:local-dev >/dev/null 2>&1; then \
73+
echo "--> Found local Docker image: evstack:local-dev (skipping build)"; \
74+
else \
75+
echo "--> Local repository detected, building Docker image..."; \
76+
$(MAKE) docker-build; \
77+
fi; \
7478
else \
7579
echo "--> Using remote repository: $(EV_NODE_IMAGE_REPO)"; \
7680
fi

test/e2e/da_client_integration_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
//go:build e2e
2+
13
package e2e
24

35
import (

0 commit comments

Comments
 (0)