fix(swamp): simplify and correct FillBlocks and increase amount of pr…

…eallocated accounts

FillBlocks was generating much more blocks than expected. Calling of testnode.FillBlock once with block amount of accounts fixes it.
Also, the amount of preallocated accounts was increased, to ensure enough blocks can be generated.
Additionally, the blocktime was increased to avoid over production of blocks.

nodebuilder/tests/reconstruct_test.go

@@ -39,7 +39,7 @@ func TestFullReconstructFromBridge(t *testing.T) {
 	const (
 		blocks = 20
 		bsize  = 16
-		btime  = time.Millisecond * 100
+		btime  = time.Millisecond * 300
 	)
 
 	ctx, cancel := context.WithTimeout(context.Background(), 5*time.Minute)
@@ -97,7 +97,7 @@ func TestFullReconstructFromLights(t *testing.T) {
 	share.DefaultSampleAmount = 20
 	const (
 		blocks = 20
-		btime  = time.Millisecond * 100
+		btime  = time.Millisecond * 300
 		bsize  = 16
 		lnodes = 69
 	)

nodebuilder/tests/swamp/swamp.go

@@ -67,7 +67,7 @@ func NewSwamp(t *testing.T, options ...Option) *Swamp {
 	ctx := context.Background()
 
 	// we create an arbitray number of funded accounts
-	accounts := make([]string, 10)
+	accounts := make([]string, 100)
 	for i := 0; i < 10; i++ {
 		accounts = append(accounts, tmrand.Str(9))
 	}

nodebuilder/tests/swamp/swamp_tx.go

@@ -8,6 +8,7 @@ import (
 	"github.com/tendermint/tendermint/pkg/consts"
 
 	"github.com/celestiaorg/celestia-app/testutil/testnode"
+
 	"github.com/celestiaorg/celestia-app/x/payment"
 	paytypes "github.com/celestiaorg/celestia-app/x/payment/types"
 )
@@ -26,19 +27,11 @@ func (s *Swamp) SubmitData(ctx context.Context, data []byte) error {
 	return nil
 }
 
-func (s *Swamp) FillBlocks(ctx context.Context, bsize, blocks int) error {
+func (s *Swamp) FillBlocks(_ context.Context, bsize, blocks int) error {
 	btime := s.comps.CoreCfg.Consensus.CreateEmptyBlocksInterval
 	timer := time.NewTimer(btime)
 	defer timer.Stop()
-	for i := 0; i < blocks; i++ {
-		_, err := testnode.FillBlock(s.ClientContext, bsize, s.accounts)
-		if err != nil {
-			return err
-		}
-		err = testnode.WaitForNextBlock(s.ClientContext)
-		if err != nil {
-			return err
-		}
-	}
-	return nil
+
+	_, err := testnode.FillBlock(s.ClientContext, bsize, s.accounts[:blocks+1])
+	return err
 }