[WIP] Improve integration tests

Use github.com/rogpeppe/go-internal/testscript and github.com/kubernetes-sigs/e2e-framework to test the builder run command with a kind kubernetes cluster New integration tests should now be easier to add, e.g. for hyperledger-labs#228 Signed-off-by: James Taylor <[email protected]>
jt-nti · Jan 14, 2025 · e126535 · e126535
1 parent 3e24250
commit e126535
Show file tree

Hide file tree

Showing 17 changed files with 673 additions and 312 deletions.
diff --git a/.github/workflows/go.yml b/.github/workflows/go.yml
@@ -44,7 +44,6 @@ jobs:
       run: go test -v ./...
       env:
         FABRIC_K8S_BUILDER_DEBUG: 'true'
-        INCLUDE_KIND_TESTS: ${{ matrix.os == 'ubuntu-latest' && 'true' || 'false' }}
 
     - name: Package
       run: |

diff --git a/cmd/build.go b/cmd/build.go
@@ -0,0 +1,56 @@
+// SPDX-License-Identifier: Apache-2.0
+
+package cmd
+
+import (
+	"context"
+	"os"
+	"strconv"
+
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/builder"
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/log"
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/util"
+)
+
+func Build() int {
+	const (
+		expectedArgsLength            = 4
+		chaincodeSourceDirectoryArg   = 1
+		chaincodeMetadataDirectoryArg = 2
+		buildOutputDirectoryArg       = 3
+	)
+
+	debug, _ := strconv.ParseBool(util.GetOptionalEnv(util.DebugVariable, "false"))
+	ctx := log.NewCmdContext(context.Background(), debug)
+	logger := log.New(ctx)
+
+	if len(os.Args) != expectedArgsLength {
+		logger.Println(
+			"Expected CHAINCODE_SOURCE_DIR, CHAINCODE_METADATA_DIR and BUILD_OUTPUT_DIR arguments",
+		)
+
+		return 1
+	}
+
+	chaincodeSourceDirectory := os.Args[chaincodeSourceDirectoryArg]
+	chaincodeMetadataDirectory := os.Args[chaincodeMetadataDirectoryArg]
+	buildOutputDirectory := os.Args[buildOutputDirectoryArg]
+
+	logger.Debugf("Chaincode source directory: %s", chaincodeSourceDirectory)
+	logger.Debugf("Chaincode metadata directory: %s", chaincodeMetadataDirectory)
+	logger.Debugf("Build output directory: %s", buildOutputDirectory)
+
+	build := &builder.Build{
+		ChaincodeSourceDirectory:   chaincodeSourceDirectory,
+		ChaincodeMetadataDirectory: chaincodeMetadataDirectory,
+		BuildOutputDirectory:       buildOutputDirectory,
+	}
+
+	if err := build.Run(ctx); err != nil {
+		logger.Printf("Error building chaincode: %+v", err)
+
+		return 1
+	}
+
+	return 0
+}
diff --git a/cmd/build/main.go b/cmd/build/main.go
@@ -3,49 +3,11 @@
 package main
 
 import (
-	"context"
 	"os"
 
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/builder"
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/log"
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/util"
-)
-
-const (
-	expectedArgsLength            = 4
-	chaincodeSourceDirectoryArg   = 1
-	chaincodeMetadataDirectoryArg = 2
-	buildOutputDirectoryArg       = 3
+	"github.com/hyperledger-labs/fabric-builder-k8s/cmd"
 )
 
 func main() {
-	debug := util.GetOptionalEnv(util.DebugVariable, "false")
-	ctx := log.NewCmdContext(context.Background(), debug == "true")
-	logger := log.New(ctx)
-
-	if len(os.Args) != expectedArgsLength {
-		logger.Println(
-			"Expected CHAINCODE_SOURCE_DIR, CHAINCODE_METADATA_DIR and BUILD_OUTPUT_DIR arguments",
-		)
-		os.Exit(1)
-	}
-
-	chaincodeSourceDirectory := os.Args[chaincodeSourceDirectoryArg]
-	chaincodeMetadataDirectory := os.Args[chaincodeMetadataDirectoryArg]
-	buildOutputDirectory := os.Args[buildOutputDirectoryArg]
-
-	logger.Debugf("Chaincode source directory: %s", chaincodeSourceDirectory)
-	logger.Debugf("Chaincode metadata directory: %s", chaincodeMetadataDirectory)
-	logger.Debugf("Build output directory: %s", buildOutputDirectory)
-
-	build := &builder.Build{
-		ChaincodeSourceDirectory:   chaincodeSourceDirectory,
-		ChaincodeMetadataDirectory: chaincodeMetadataDirectory,
-		BuildOutputDirectory:       buildOutputDirectory,
-	}
-
-	if err := build.Run(ctx); err != nil {
-		logger.Printf("Error building chaincode: %+v", err)
-		os.Exit(1)
-	}
+	os.Exit(cmd.Build())
 }
diff --git a/cmd/detect.go b/cmd/detect.go
@@ -0,0 +1,54 @@
+// SPDX-License-Identifier: Apache-2.0
+
+package cmd
+
+import (
+	"context"
+	"errors"
+	"os"
+	"strconv"
+
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/builder"
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/log"
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/util"
+)
+
+func Detect() int {
+	const (
+		expectedArgsLength            = 3
+		chaincodeSourceDirectoryArg   = 1
+		chaincodeMetadataDirectoryArg = 2
+	)
+
+	debug, _ := strconv.ParseBool(util.GetOptionalEnv(util.DebugVariable, "false"))
+	ctx := log.NewCmdContext(context.Background(), debug)
+	logger := log.New(ctx)
+
+	if len(os.Args) != expectedArgsLength {
+		logger.Println("Expected CHAINCODE_SOURCE_DIR and CHAINCODE_METADATA_DIR arguments")
+
+		return 1
+	}
+
+	chaincodeSourceDirectory := os.Args[chaincodeSourceDirectoryArg]
+	chaincodeMetadataDirectory := os.Args[chaincodeMetadataDirectoryArg]
+
+	logger.Debugf("Chaincode source directory: %s", chaincodeSourceDirectory)
+	logger.Debugf("Chaincode metadata directory: %s", chaincodeMetadataDirectory)
+
+	detect := &builder.Detect{
+		ChaincodeSourceDirectory:   chaincodeSourceDirectory,
+		ChaincodeMetadataDirectory: chaincodeMetadataDirectory,
+	}
+
+	if err := detect.Run(ctx); err != nil {
+		if !errors.Is(err, builder.ErrUnsupportedChaincodeType) {
+			// don't spam the peer log if it's just chaincode we don't recognise
+			logger.Printf("Error detecting chaincode: %+v", err)
+		}
+
+		return 1
+	}
+
+	return 0
+}
diff --git a/cmd/detect/main.go b/cmd/detect/main.go
@@ -3,48 +3,11 @@
 package main
 
 import (
-	"context"
-	"errors"
 	"os"
 
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/builder"
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/log"
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/util"
-)
-
-const (
-	expectedArgsLength            = 3
-	chaincodeSourceDirectoryArg   = 1
-	chaincodeMetadataDirectoryArg = 2
+	"github.com/hyperledger-labs/fabric-builder-k8s/cmd"
 )
 
 func main() {
-	debug := util.GetOptionalEnv(util.DebugVariable, "false")
-	ctx := log.NewCmdContext(context.Background(), debug == "true")
-	logger := log.New(ctx)
-
-	if len(os.Args) != expectedArgsLength {
-		logger.Println("Expected CHAINCODE_SOURCE_DIR and CHAINCODE_METADATA_DIR arguments")
-		os.Exit(1)
-	}
-
-	chaincodeSourceDirectory := os.Args[chaincodeSourceDirectoryArg]
-	chaincodeMetadataDirectory := os.Args[chaincodeMetadataDirectoryArg]
-
-	logger.Debugf("Chaincode source directory: %s", chaincodeSourceDirectory)
-	logger.Debugf("Chaincode metadata directory: %s", chaincodeMetadataDirectory)
-
-	detect := &builder.Detect{
-		ChaincodeSourceDirectory:   chaincodeSourceDirectory,
-		ChaincodeMetadataDirectory: chaincodeMetadataDirectory,
-	}
-
-	if err := detect.Run(ctx); err != nil {
-		if !errors.Is(err, builder.ErrUnsupportedChaincodeType) {
-			// don't spam the peer log if it's just chaincode we don't recognise
-			logger.Printf("Error detecting chaincode: %+v", err)
-		}
-
-		os.Exit(1)
-	}
+	os.Exit(cmd.Detect())
 }
diff --git a/cmd/release.go b/cmd/release.go
@@ -0,0 +1,50 @@
+// SPDX-License-Identifier: Apache-2.0
+
+package cmd
+
+import (
+	"context"
+	"os"
+	"strconv"
+
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/builder"
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/log"
+	"github.com/hyperledger-labs/fabric-builder-k8s/internal/util"
+)
+
+func Release() int {
+	const (
+		expectedArgsLength        = 3
+		buildOutputDirectoryArg   = 1
+		releaseOutputDirectoryArg = 2
+	)
+
+	debug, _ := strconv.ParseBool(util.GetOptionalEnv(util.DebugVariable, "false"))
+	ctx := log.NewCmdContext(context.Background(), debug)
+	logger := log.New(ctx)
+
+	if len(os.Args) != expectedArgsLength {
+		logger.Println("Expected BUILD_OUTPUT_DIR and RELEASE_OUTPUT_DIR arguments")
+
+		return 1
+	}
+
+	buildOutputDirectory := os.Args[buildOutputDirectoryArg]
+	releaseOutputDirectory := os.Args[releaseOutputDirectoryArg]
+
+	logger.Debugf("Build output directory: %s", buildOutputDirectory)
+	logger.Debugf("Release output directory: %s", releaseOutputDirectory)
+
+	release := &builder.Release{
+		BuildOutputDirectory:   buildOutputDirectory,
+		ReleaseOutputDirectory: releaseOutputDirectory,
+	}
+
+	if err := release.Run(ctx); err != nil {
+		logger.Printf("Error releasing chaincode: %+v", err)
+
+		return 1
+	}
+
+	return 0
+}
diff --git a/cmd/release/main.go b/cmd/release/main.go
@@ -3,43 +3,11 @@
 package main
 
 import (
-	"context"
 	"os"
 
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/builder"
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/log"
-	"github.com/hyperledger-labs/fabric-builder-k8s/internal/util"
-)
-
-const (
-	expectedArgsLength        = 3
-	buildOutputDirectoryArg   = 1
-	releaseOutputDirectoryArg = 2
+	"github.com/hyperledger-labs/fabric-builder-k8s/cmd"
 )
 
 func main() {
-	debug := util.GetOptionalEnv(util.DebugVariable, "false")
-	ctx := log.NewCmdContext(context.Background(), debug == "true")
-	logger := log.New(ctx)
-
-	if len(os.Args) != expectedArgsLength {
-		logger.Println("Expected BUILD_OUTPUT_DIR and RELEASE_OUTPUT_DIR arguments")
-		os.Exit(1)
-	}
-
-	buildOutputDirectory := os.Args[buildOutputDirectoryArg]
-	releaseOutputDirectory := os.Args[releaseOutputDirectoryArg]
-
-	logger.Debugf("Build output directory: %s", buildOutputDirectory)
-	logger.Debugf("Release output directory: %s", releaseOutputDirectory)
-
-	release := &builder.Release{
-		BuildOutputDirectory:   buildOutputDirectory,
-		ReleaseOutputDirectory: releaseOutputDirectory,
-	}
-
-	if err := release.Run(ctx); err != nil {
-		logger.Printf("Error releasing chaincode: %+v", err)
-		os.Exit(1)
-	}
+	os.Exit(cmd.Release())
 }