skip some reconstruction

.github/workflows/integration-tests.yml

@@ -97,7 +97,7 @@ jobs:
           go-version: ${{ inputs.go-version }}
 
       - name: run reconstruction tests
-        run: make test-integration TAGS=reconstruction
+        run: make test-integration SHORT=true TAGS=reconstruction
 
   sync_tests:
     name: Integration Tests Sync
@@ -111,5 +111,5 @@ jobs:
         with:
           go-version: ${{ inputs.go-version }}
 
-      - name: run reconstruction tests
+      - name: run sync tests
         run: make test-integration TAGS=sync

Makefile

@@ -4,6 +4,7 @@ DIR_FULLPATH=$(shell pwd)
 versioningPath := "github.com/celestiaorg/celestia-node/nodebuilder/node"
 LDFLAGS=-ldflags="-X '$(versioningPath).buildTime=$(shell date)' -X '$(versioningPath).lastCommit=$(shell git rev-parse HEAD)' -X '$(versioningPath).semanticVersion=$(shell git describe --tags --dirty=-dev 2>/dev/null || git rev-parse --abbrev-ref HEAD)'"
 TAGS=integration
+SHORT=
 ifeq (${PREFIX},)
 	PREFIX := /usr/local
 endif
@@ -14,6 +15,11 @@ else
 	VERBOSE =
 	LOG_AND_FILTER =
 endif
+ifeq ($(SHORT),true)
+	INTEGRATION_RUN_LENGTH = -short
+else
+	INTEGRATION_RUN_LENGTH =
+endif
 ## help: Get more info on make commands.
 help: Makefile
 	@echo " Choose a command run in "$(PROJECTNAME)":"
@@ -124,8 +130,8 @@ test-unit-race:
 
 ## test-integration: Running /integration tests located in nodebuilder/tests
 test-integration:
-	@echo "--> Running integrations tests -tags=$(TAGS)"
-	@go test -tags=$(TAGS) ./nodebuilder/tests
+	@echo "--> Running integrations tests $(VERBOSE) -tags=$(TAGS) $(INTEGRATION_RUN_LENGTH)"
+	@go test $(VERBOSE) -tags=$(TAGS) $(INTEGRATION_RUN_LENGTH) ./nodebuilder/tests
 .PHONY: test-integration
 
 ## test-integration-race: Running integration tests with data race detector located in node/tests

nodebuilder/tests/reconstruct_test.go

@@ -86,6 +86,10 @@ Test-Case: Full Node reconstructs blocks from each other, after unsuccessfully s
 block from LN subnetworks. Analog to TestShareAvailable_DisconnectedFullNodes.
 */
 func TestFullReconstructFromFulls(t *testing.T) {
+	if testing.Short() {
+		t.Skip()
+	}
+
 	light.DefaultSampleAmount = 10 // s
 	const (
 		blocks = 10
@@ -252,6 +256,10 @@ Steps:
 9. Check that the FN can retrieve shares from 1 to 20 blocks
 */
 func TestFullReconstructFromLights(t *testing.T) {
+	if testing.Short() {
+		t.Skip()
+	}
+
 	eds.RetrieveQuadrantTimeout = time.Millisecond * 100
 	light.DefaultSampleAmount = 20
 	const (

nodebuilder/tests/sync_test.go

@@ -18,7 +18,7 @@ import (
 const (
 	numBlocks = 20
 	bsize     = 16
-	btime     = time.Millisecond * 300
+	sbtime    = time.Millisecond * 300
 )
 
 /*
@@ -47,7 +47,7 @@ func TestSyncAgainstBridge_NonEmptyChain(t *testing.T) {
 	ctx, cancel := context.WithTimeout(context.Background(), swamp.DefaultTestTimeout)
 	t.Cleanup(cancel)
 
-	sw := swamp.NewSwamp(t, swamp.WithBlockTime(btime))
+	sw := swamp.NewSwamp(t, swamp.WithBlockTime(sbtime))
 	// wait for core network to fill 20 blocks
 	fillDn := swamp.FillBlocks(ctx, sw.ClientContext, sw.Accounts, bsize, numBlocks)
 	sw.WaitTillHeight(ctx, numBlocks)
@@ -140,7 +140,7 @@ func TestSyncAgainstBridge_EmptyChain(t *testing.T) {
 	ctx, cancel := context.WithTimeout(context.Background(), swamp.DefaultTestTimeout)
 	t.Cleanup(cancel)
 
-	sw := swamp.NewSwamp(t, swamp.WithBlockTime(btime))
+	sw := swamp.NewSwamp(t, swamp.WithBlockTime(sbtime))
 	sw.WaitTillHeight(ctx, numBlocks)
 
 	// create bridge node and set it as the bootstrapper for the suite