Repository: buildpack/pack (go)

Issue:
pack inspect-builder <non-existent-tag> does not error
No error when I try to inspect a non-existant or unreachable builder:
```
± zm |master ✓| → pack inspect-builder cloudfoundry/cnb:0.0.40-bionic
Inspecting builder: cloudfoundry/cnb:0.0.40-bionic

REMOTE:
(not present)

LOCAL:
(not present)

```

Pull request description:
inspect-builder throws error with missing remote and local builder
Signed-off-by: ktpv <ktpv@users.noreply.github.com>

Patch:
diff --git a/cmd/pack/main.go b/cmd/pack/main.go
index f6d38a70..209993f4 100644
--- a/cmd/pack/main.go
+++ b/cmd/pack/main.go
@@ -54,8 +54,6 @@ func main() {
 	rootCmd.PersistentFlags().Bool("timestamps", false, "Enable timestamps in output")
 	rootCmd.PersistentFlags().BoolP("quiet", "q", false, "Show less output")
 	rootCmd.PersistentFlags().BoolP("verbose", "v", false, "Show more output")
-	rootCmd.Flags().Bool("version", false, "Show current 'pack' version")
-
 	commands.AddHelpFlag(rootCmd, "pack")
 
 	rootCmd.AddCommand(commands.Build(logger, cfg, &packClient))
@@ -76,9 +74,6 @@ func main() {
 
 	rootCmd.AddCommand(commands.CompletionCommand(logger))
 
-	rootCmd.Version = cmd.Version
-	rootCmd.SetVersionTemplate(`{{.Version}}{{"\n"}}`)
-
 	if err := rootCmd.Execute(); err != nil {
 		if commands.IsSoftError(err) {
 			os.Exit(2)
diff --git a/internal/commands/inspect_builder.go b/internal/commands/inspect_builder.go
index c8e12a9a..5d19c0ec 100644
--- a/internal/commands/inspect_builder.go
+++ b/internal/commands/inspect_builder.go
@@ -36,28 +36,33 @@ func InspectBuilder(logger logging.Logger, cfg config.Config, client PackClient)
 				imageName = args[0]
 			}
 
+			presentRemote, remoteOutput, remoteWarnings, remoteErr := inspectBuilderOutput(client, cfg, imageName, false)
+			presentLocal, localOutput, localWarnings, localErr := inspectBuilderOutput(client, cfg, imageName, true)
+
+			if !presentRemote && !presentLocal {
+				return errors.New(fmt.Sprintf("Unable to find builder '%s' locally or remotely.\n", imageName))
+			}
+
 			if imageName == cfg.DefaultBuilder {
 				logger.Infof("Inspecting default builder: %s\n", style.Symbol(imageName))
 			} else {
 				logger.Infof("Inspecting builder: %s\n", style.Symbol(imageName))
 			}
 
-			remoteOutput, warnings, err := inspectBuilderOutput(client, cfg, imageName, false)
-			if err != nil {
-				logger.Error(err.Error())
+			if remoteErr != nil {
+				logger.Error(remoteErr.Error())
 			} else {
 				logger.Infof("\nREMOTE:\n%s\n", remoteOutput)
-				for _, w := range warnings {
+				for _, w := range remoteWarnings {
 					logger.Warn(w)
 				}
 			}
 
-			localOutput, warnings, err := inspectBuilderOutput(client, cfg, imageName, true)
-			if err != nil {
-				logger.Error(err.Error())
+			if localErr != nil {
+				logger.Error(localErr.Error())
 			} else {
 				logger.Infof("\nLOCAL:\n%s\n", localOutput)
-				for _, w := range warnings {
+				for _, w := range localWarnings {
 					logger.Warn(w)
 				}
 			}
@@ -69,7 +74,7 @@ func InspectBuilder(logger logging.Logger, cfg config.Config, client PackClient)
 	return cmd
 }
 
-func inspectBuilderOutput(client PackClient, cfg config.Config, imageName string, local bool) (output string, warning []string, err error) {
+func inspectBuilderOutput(client PackClient, cfg config.Config, imageName string, local bool) (present bool, output string, warning []string, err error) {
 	source := "remote"
 	if local {
 		source = "local"
@@ -77,20 +82,20 @@ func inspectBuilderOutput(client PackClient, cfg config.Config, imageName string
 
 	info, err := client.InspectBuilder(imageName, local)
 	if err != nil {
-		return "", nil, errors.Wrapf(err, "inspecting %s image '%s'", source, imageName)
+		return true, "", nil, errors.Wrapf(err, "inspecting %s image '%s'", source, imageName)
 	}
 
 	if info == nil {
-		return "(not present)", nil, nil
+		return false, "(not present)", nil, nil
 	}
 
 	var buf bytes.Buffer
 	warnings, err := generateBuilderOutput(&buf, imageName, cfg, *info)
 	if err != nil {
-		return "", nil, errors.Wrapf(err, "writing output for %s image '%s'", source, imageName)
+		return true, "", nil, errors.Wrapf(err, "writing output for %s image '%s'", source, imageName)
 	}
 
-	return buf.String(), warnings, nil
+	return true, buf.String(), warnings, nil
 }
 
 func generateBuilderOutput(writer io.Writer, imageName string, cfg config.Config, info pack.BuilderInfo) (warnings []string, err error) {


Test patch:
diff --git a/internal/commands/inspect_builder_test.go b/internal/commands/inspect_builder_test.go
index aefc9bc4..44826efe 100644
--- a/internal/commands/inspect_builder_test.go
+++ b/internal/commands/inspect_builder_test.go
@@ -38,8 +38,147 @@ func testInspectBuilderCommand(t *testing.T, when spec.G, it spec.S) {
 		mockController *gomock.Controller
 		mockClient     *testmocks.MockPackClient
 		cfg            config.Config
-	)
+		buildpack1Info = dist.BuildpackInfo{ID: "test.bp.one", Version: "1.0.0"}
+		buildpack2Info = dist.BuildpackInfo{ID: "test.bp.two", Version: "2.0.0"}
+		buildpacks     = []builder.BuildpackMetadata{
+			{BuildpackInfo: buildpack1Info, Latest: true},
+			{BuildpackInfo: buildpack2Info, Latest: false},
+		}
+		remoteInfo = &pack.BuilderInfo{
+			Description:     "Some remote description",
+			Stack:           "test.stack.id",
+			Mixins:          []string{"mixin1", "mixin2", "build:mixin3", "build:mixin4"},
+			RunImage:        "some/run-image",
+			RunImageMirrors: []string{"first/default", "second/default"},
+			Buildpacks:      buildpacks,
+			Order: dist.Order{
+				{Group: []dist.BuildpackRef{
+					{BuildpackInfo: buildpack1Info, Optional: true},
+					{BuildpackInfo: dist.BuildpackInfo{ID: buildpack2Info.ID}},
+				}}},
+			Lifecycle: builder.LifecycleDescriptor{
+				Info: builder.LifecycleInfo{
+					Version: &builder.Version{
+						Version: *semver.MustParse("6.7.8"),
+					},
+				},
+				API: builder.LifecycleAPI{
+					BuildpackVersion: api.MustParse("5.6"),
+					PlatformVersion:  api.MustParse("7.8"),
+				},
+			},
+			CreatedBy: builder.CreatorMetadata{
+				Name:    "Pack CLI",
+				Version: "1.2.3",
+			},
+		}
+		localInfo = &pack.BuilderInfo{
+			Description:     "Some local description",
+			Stack:           "test.stack.id",
+			Mixins:          []string{"mixin1", "mixin2", "build:mixin3", "build:mixin4"},
+			RunImage:        "some/run-image",
+			RunImageMirrors: []string{"first/local-default", "second/local-default"},
+			Buildpacks:      buildpacks,
+			Order: dist.Order{
+				{Group: []dist.BuildpackRef{{BuildpackInfo: buildpack1Info}}},
+				{Group: []dist.BuildpackRef{{BuildpackInfo: dist.BuildpackInfo{ID: buildpack2Info.ID}, Optional: true}}},
+			},
+			Lifecycle: builder.LifecycleDescriptor{
+				Info: builder.LifecycleInfo{
+					Version: &builder.Version{
+						Version: *semver.MustParse("4.5.6"),
+					},
+				},
+				API: builder.LifecycleAPI{
+					BuildpackVersion: api.MustParse("1.2"),
+					PlatformVersion:  api.MustParse("3.4"),
+				},
+			},
+			CreatedBy: builder.CreatorMetadata{
+				Name:    "Pack CLI",
+				Version: "4.5.6",
+			},
+		}
+		remoteOutput = `
+REMOTE:
+
+Description: Some remote description
 
+Created By:
+  Name: Pack CLI
+  Version: 1.2.3
+
+Stack:
+  ID: test.stack.id
+  Mixins:
+    mixin1
+    mixin2
+    build:mixin3
+    build:mixin4
+
+Lifecycle:
+  Version: 6.7.8
+  Buildpack API: 5.6
+  Platform API: 7.8
+
+Run Images:
+  first/local     (user-configured)
+  second/local    (user-configured)
+  some/run-image
+  first/default
+  second/default
+
+Buildpacks:
+  ID                 VERSION
+  test.bp.one        1.0.0
+  test.bp.two        2.0.0
+
+Detection Order:
+  Group #1:
+    test.bp.one@1.0.0    (optional)
+    test.bp.two          
+`
+		localOutput = `
+LOCAL:
+
+Description: Some local description
+
+Created By:
+  Name: Pack CLI
+  Version: 4.5.6
+
+Stack:
+  ID: test.stack.id
+  Mixins:
+    mixin1
+    mixin2
+    build:mixin3
+    build:mixin4
+
+Lifecycle:
+  Version: 4.5.6
+  Buildpack API: 1.2
+  Platform API: 3.4
+
+Run Images:
+  first/local     (user-configured)
+  second/local    (user-configured)
+  some/run-image
+  first/local-default
+  second/local-default
+
+Buildpacks:
+  ID                 VERSION
+  test.bp.one        1.0.0
+  test.bp.two        2.0.0
+
+Detection Order:
+  Group #1:
+    test.bp.one@1.0.0    
+  Group #2:
+    test.bp.two    (optional)
+`
+	)
 	it.Before(func() {
 		cfg = config.Config{
 			DefaultBuilder: "default/builder",
@@ -59,15 +198,38 @@ func testInspectBuilderCommand(t *testing.T, when spec.G, it spec.S) {
 	})
 
 	when("#Get", func() {
-		when("image cannot be found", func() {
-			it("logs 'Not present'", func() {
+		when("remote builder image cannot be found", func() {
+			it("warns 'remote image not present'", func() {
 				mockClient.EXPECT().InspectBuilder("some/image", false).Return(nil, nil)
+				mockClient.EXPECT().InspectBuilder("some/image", true).Return(localInfo, nil)
+				command.SetArgs([]string{"some/image"})
+				h.AssertNil(t, command.Execute())
+				h.AssertContains(t, outBuf.String(), `Inspecting builder: 'some/image'`)
+				h.AssertContains(t, outBuf.String(), "REMOTE:\n(not present)\n\n")
+				h.AssertContains(t, outBuf.String(), localOutput)
+			})
+		})
+
+		when("local builder image cannot be found", func() {
+			it("warns 'local image not present'", func() {
+				mockClient.EXPECT().InspectBuilder("some/image", false).Return(remoteInfo, nil)
 				mockClient.EXPECT().InspectBuilder("some/image", true).Return(nil, nil)
 
 				command.SetArgs([]string{"some/image"})
 				h.AssertNil(t, command.Execute())
+				h.AssertContains(t, outBuf.String(), `Inspecting builder: 'some/image'`)
+				h.AssertContains(t, outBuf.String(), "LOCAL:\n(not present)\n")
+				h.AssertContains(t, outBuf.String(), remoteOutput)
+			})
+		})
+
+		when("image cannot be found", func() {
+			it("logs 'errors when no image is found'", func() {
+				mockClient.EXPECT().InspectBuilder("some/image", false).Return(nil, nil)
+				mockClient.EXPECT().InspectBuilder("some/image", true).Return(nil, nil)
 
-				h.AssertContains(t, outBuf.String(), "REMOTE:\n(not present)\n\nLOCAL:\n(not present)\n")
+				command.SetArgs([]string{"some/image"})
+				h.AssertError(t, command.Execute(), `Unable to find builder 'some/image' locally or remotely.`)
 			})
 		})
 
@@ -140,70 +302,6 @@ func testInspectBuilderCommand(t *testing.T, when spec.G, it spec.S) {
 		})
 
 		when("is successful", func() {
-			var (
-				buildpack1Info = dist.BuildpackInfo{ID: "test.bp.one", Version: "1.0.0"}
-				buildpack2Info = dist.BuildpackInfo{ID: "test.bp.two", Version: "2.0.0"}
-				buildpacks     = []builder.BuildpackMetadata{
-					{BuildpackInfo: buildpack1Info, Latest: true},
-					{BuildpackInfo: buildpack2Info, Latest: false},
-				}
-				remoteInfo = &pack.BuilderInfo{
-					Description:     "Some remote description",
-					Stack:           "test.stack.id",
-					Mixins:          []string{"mixin1", "mixin2", "build:mixin3", "build:mixin4"},
-					RunImage:        "some/run-image",
-					RunImageMirrors: []string{"first/default", "second/default"},
-					Buildpacks:      buildpacks,
-					Order: dist.Order{
-						{Group: []dist.BuildpackRef{
-							{BuildpackInfo: buildpack1Info, Optional: true},
-							{BuildpackInfo: dist.BuildpackInfo{ID: buildpack2Info.ID}},
-						}}},
-					Lifecycle: builder.LifecycleDescriptor{
-						Info: builder.LifecycleInfo{
-							Version: &builder.Version{
-								Version: *semver.MustParse("6.7.8"),
-							},
-						},
-						API: builder.LifecycleAPI{
-							BuildpackVersion: api.MustParse("5.6"),
-							PlatformVersion:  api.MustParse("7.8"),
-						},
-					},
-					CreatedBy: builder.CreatorMetadata{
-						Name:    "Pack CLI",
-						Version: "1.2.3",
-					},
-				}
-				localInfo = &pack.BuilderInfo{
-					Description:     "Some local description",
-					Stack:           "test.stack.id",
-					Mixins:          []string{"mixin1", "mixin2", "build:mixin3", "build:mixin4"},
-					RunImage:        "some/run-image",
-					RunImageMirrors: []string{"first/local-default", "second/local-default"},
-					Buildpacks:      buildpacks,
-					Order: dist.Order{
-						{Group: []dist.BuildpackRef{{BuildpackInfo: buildpack1Info}}},
-						{Group: []dist.BuildpackRef{{BuildpackInfo: dist.BuildpackInfo{ID: buildpack2Info.ID}, Optional: true}}},
-					},
-					Lifecycle: builder.LifecycleDescriptor{
-						Info: builder.LifecycleInfo{
-							Version: &builder.Version{
-								Version: *semver.MustParse("4.5.6"),
-							},
-						},
-						API: builder.LifecycleAPI{
-							BuildpackVersion: api.MustParse("1.2"),
-							PlatformVersion:  api.MustParse("3.4"),
-						},
-					},
-					CreatedBy: builder.CreatorMetadata{
-						Name:    "Pack CLI",
-						Version: "4.5.6",
-					},
-				}
-			)
-
 			when("using the default builder", func() {
 				it.Before(func() {
 					cfg.DefaultBuilder = "some/image"
@@ -215,86 +313,8 @@ func testInspectBuilderCommand(t *testing.T, when spec.G, it spec.S) {
 				it("inspects the default builder", func() {
 					h.AssertNil(t, command.Execute())
 					h.AssertContains(t, outBuf.String(), "Inspecting default builder: 'default/builder'")
-					h.AssertContains(t, outBuf.String(), `
-REMOTE:
-
-Description: Some remote description
-
-Created By:
-  Name: Pack CLI
-  Version: 1.2.3
-
-Stack:
-  ID: test.stack.id
-  Mixins:
-    mixin1
-    mixin2
-    build:mixin3
-    build:mixin4
-
-Lifecycle:
-  Version: 6.7.8
-  Buildpack API: 5.6
-  Platform API: 7.8
-
-Run Images:
-  first/local     (user-configured)
-  second/local    (user-configured)
-  some/run-image
-  first/default
-  second/default
-
-Buildpacks:
-  ID                 VERSION
-  test.bp.one        1.0.0
-  test.bp.two        2.0.0
-
-Detection Order:
-  Group #1:
-    test.bp.one@1.0.0    (optional)
-    test.bp.two          
-`)
-
-					h.AssertContains(t, outBuf.String(), `
-LOCAL:
-
-Description: Some local description
-
-Created By:
-  Name: Pack CLI
-  Version: 4.5.6
-
-Stack:
-  ID: test.stack.id
-  Mixins:
-    mixin1
-    mixin2
-    build:mixin3
-    build:mixin4
-
-Lifecycle:
-  Version: 4.5.6
-  Buildpack API: 1.2
-  Platform API: 3.4
-
-Run Images:
-  first/local     (user-configured)
-  second/local    (user-configured)
-  some/run-image
-  first/local-default
-  second/local-default
-
-Buildpacks:
-  ID                 VERSION
-  test.bp.one        1.0.0
-  test.bp.two        2.0.0
-
-Detection Order:
-  Group #1:
-    test.bp.one@1.0.0    
-  Group #2:
-    test.bp.two    (optional)
-`)
+					h.AssertContains(t, outBuf.String(), remoteOutput)
+					h.AssertContains(t, outBuf.String(), localOutput)
 				})
 			})
 
@@ -308,86 +328,8 @@ Detection Order:
 				it("displays builder information for local and remote", func() {
 					h.AssertNil(t, command.Execute())
 					h.AssertContains(t, outBuf.String(), "Inspecting builder: 'some/image'")
-					h.AssertContains(t, outBuf.String(), `
-REMOTE:
-
-Description: Some remote description
-
-Created By:
-  Name: Pack CLI
-  Version: 1.2.3
-
-Stack:
-  ID: test.stack.id
-  Mixins:
-    mixin1
-    mixin2
-    build:mixin3
-    build:mixin4
-
-Lifecycle:
-  Version: 6.7.8
-  Buildpack API: 5.6
-  Platform API: 7.8
-
-Run Images:
-  first/local     (user-configured)
-  second/local    (user-configured)
-  some/run-image
-  first/default
-  second/default
-
-Buildpacks:
-  ID                 VERSION
-  test.bp.one        1.0.0
-  test.bp.two        2.0.0
-
-Detection Order:
-  Group #1:
-    test.bp.one@1.0.0    (optional)
-    test.bp.two          
-`)
-
-					h.AssertContains(t, outBuf.String(), `
-LOCAL:
-
-Description: Some local description
-
-Created By:
-  Name: Pack CLI
-  Version: 4.5.6
-
-Stack:
-  ID: test.stack.id
-  Mixins:
-    mixin1
-    mixin2
-    build:mixin3
-    build:mixin4
-
-Lifecycle:
-  Version: 4.5.6
-  Buildpack API: 1.2
-  Platform API: 3.4
-
-Run Images:
-  first/local     (user-configured)
-  second/local    (user-configured)
-  some/run-image
-  first/local-default
-  second/local-default
-
-Buildpacks:
-  ID                 VERSION
-  test.bp.one        1.0.0
-  test.bp.two        2.0.0
-
-Detection Order:
-  Group #1:
-    test.bp.one@1.0.0    
-  Group #2:
-    test.bp.two    (optional)
-`)
+					h.AssertContains(t, outBuf.String(), remoteOutput)
+					h.AssertContains(t, outBuf.String(), localOutput)
 				})
 			})
 		})