change return

Author: tt-cll

framework/cmd/interactive.go

@@ -30,7 +30,7 @@ func createComponentsFromForm(form *nodeSetForm) error {
 		_   *simple_node_set.Output
 		err error
 	)
-	if err := framework.DefaultNetwork(t, &sync.Once{}); err != nil {
+	if _, err := framework.DefaultNetwork(&sync.Once{}); err != nil {
 		return err
 	}
 	switch form.Network {

framework/components/clnode/clnode_test.go

@@ -49,7 +49,7 @@ func TestComponentDockerNodeWithSharedDB(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
-		err := framework.DefaultNetwork(t, &sync.Once{})
+		_, err := framework.DefaultNetwork(&sync.Once{})
 		require.NoError(t, err)
 
 		t.Run(tc.name, func(t *testing.T) {
@@ -84,7 +84,7 @@ func TestComponentDockerNodeWithDB(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
-		err := framework.DefaultNetwork(t, &sync.Once{})
+		_, err := framework.DefaultNetwork(&sync.Once{})
 		require.NoError(t, err)
 
 		t.Run(tc.name, func(t *testing.T) {

framework/components/jd/jd_test.go

@@ -1,20 +1,21 @@
 package jd_test
 
 import (
-	"github.com/smartcontractkit/chainlink-testing-framework/framework"
-	"github.com/smartcontractkit/chainlink-testing-framework/framework/components/jd"
-	"github.com/stretchr/testify/require"
 	"os"
 	"sync"
 	"testing"
+
+	"github.com/smartcontractkit/chainlink-testing-framework/framework"
+	"github.com/smartcontractkit/chainlink-testing-framework/framework/components/jd"
+	"github.com/stretchr/testify/require"
 )
 
 // here we only test that we can boot up JD
 // client examples are under "examples" dir
 // since JD is private this env var should be set locally and in CI
 // TODO: add ComponentDocker prefix to turn this on when we'll have access to ECRs
 func TestJD(t *testing.T) {
-	err := framework.DefaultNetwork(t, &sync.Once{})
+	_, err := framework.DefaultNetwork(&sync.Once{})
 	require.NoError(t, err)
 	_, err = jd.NewJD(&jd.Input{
 		Image: os.Getenv("CTF_JD_IMAGE"),

framework/components/simple_node_set/nodeset_test.go

@@ -1,14 +1,15 @@
 package simple_node_set_test
 
 import (
+	"sync"
+	"testing"
+
 	"github.com/smartcontractkit/chainlink-testing-framework/framework"
 	"github.com/smartcontractkit/chainlink-testing-framework/framework/components/blockchain"
 	"github.com/smartcontractkit/chainlink-testing-framework/framework/components/clnode"
 	"github.com/smartcontractkit/chainlink-testing-framework/framework/components/postgres"
 	ns "github.com/smartcontractkit/chainlink-testing-framework/framework/components/simple_node_set"
 	"github.com/stretchr/testify/require"
-	"sync"
-	"testing"
 )
 
 type testCase struct {
@@ -113,7 +114,7 @@ level = 'info'
 	}
 
 	for _, tc := range testCases {
-		err := framework.DefaultNetwork(t, &sync.Once{})
+		_, err := framework.DefaultNetwork(&sync.Once{})
 		require.NoError(t, err)
 
 		t.Run(tc.name, func(t *testing.T) {

framework/config.go

@@ -143,23 +143,22 @@ func Load[X any](t *testing.T) (*X, error) {
 	//		return nil, fmt.Errorf("failed to connect AWSSecretsManager: %w", err)
 	//	}
 	//}
-	err = DefaultNetwork(t, once)
+	_, err = DefaultNetwork(once)
 	require.NoError(t, err)
 	return input, nil
 }
 
-func DefaultNetwork(t *testing.T, once *sync.Once) error {
+func DefaultNetwork(once *sync.Once) (*testcontainers.DockerNetwork, error) {
 	var net *testcontainers.DockerNetwork
 	var err error
 	once.Do(func() {
 		net, err = network.New(
 			context.Background(),
 			network.WithLabels(map[string]string{"framework": "ctf"}),
 		)
-		testcontainers.CleanupNetwork(t, net)
 		DefaultNetworkName = net.Name
 	})
-	return err
+	return net, err
 }
 
 func RenderTemplate(tmpl string, data interface{}) (string, error) {