switch to small change and remove added tests

cmd/builder/internal/command.go

@@ -116,7 +116,7 @@ configuration is provided, ocb will generate a default Collector.
 		return nil, err
 	}
 	// version of this binary
-	cmd.AddCommand(versionCommand(binVersion))
+	cmd.AddCommand(versionCommand())
 
 	return cmd, nil
 }

cmd/builder/internal/version.go

@@ -14,34 +14,20 @@ var (
 	version = ""
 )
 
-type debugReadBuildInfoFunc func() (info *debug.BuildInfo, ok bool)
-type binVersionFunc func(fn debugReadBuildInfoFunc) (string, error)
-
-// binVersion returns the version of the binary.
-// If the version is not set, it attempts to read the build information.
-// Returns an error if the build information cannot be read.
-func binVersion(fn debugReadBuildInfoFunc) (string, error) {
-	if version != "" {
-		return version, nil
-	}
-	info, ok := fn()
-	if !ok {
-		return "", fmt.Errorf("failed to read build info")
+func init() {
+	// the second returned value is a boolean, which is true if the binaries are built with module support.
+	if version == "" {
+		info, _ := debug.ReadBuildInfo()
+		version = info.Main.Version
 	}
-	return info.Main.Version, nil
 }
 
-func versionCommand(fn binVersionFunc) *cobra.Command {
-	var err error
+func versionCommand() *cobra.Command {
 	return &cobra.Command{
 		Use:   "version",
 		Short: "Version of ocb",
 		Long:  "Prints the version of the ocb binary",
 		RunE: func(cmd *cobra.Command, _ []string) error {
-			version, err = fn(debug.ReadBuildInfo)
-			if err != nil {
-				return err
-			}
 			cmd.Println(fmt.Sprintf("%s version %s", cmd.Parent().Name(), version))
 			return nil
 		},