fixed mocks

Signed-off-by: Rashad Sirajudeen <rashadsirajudeen@gmail.com>

Author: rashadism

internal/commands/build.go

@@ -284,7 +284,6 @@ This option may set DOCKER_HOST environment variable for the build container if
 			cmd.Flags().MarkHidden("sparse")
 		}
 	}
-
 }
 
 func validateBuildFlags(flags *BuildFlags, cfg config.Config, inputImageRef client.InputImageReference, logger logging.Logger) error {

internal/commands/testmocks/mock_pack_client.go

@@ -14,3 +14,8 @@ func (f *FakeLifecycle) Execute(ctx context.Context, opts build.LifecycleOptions
 	f.Opts = opts
 	return nil
 }
+
+func (f *FakeLifecycle) Detect(ctx context.Context, opts build.LifecycleOptions) error {
+	f.Opts = opts
+	return nil
+}

internal/fakes/fake_lifecycle.go

@@ -3,8 +3,6 @@ package client
 import (
 	"context"
 	"fmt"
-
-	"github.com/docker/docker/api/types"
 )
 
 func (c *Client) Detect(ctx context.Context, opts BuildOptions) error {
@@ -13,8 +11,7 @@ func (c *Client) Detect(ctx context.Context, opts BuildOptions) error {
 		return err
 	}
 
-	defer c.docker.ImageRemove(context.Background(), lifecycleOpts.LifecycleImage, types.ImageRemoveOptions{Force: true})
-	defer c.docker.ImageRemove(context.Background(), lifecycleOpts.Builder.Name(), types.ImageRemoveOptions{Force: true})
+	// TODO: Cleanup
 
 	if err = c.lifecycleExecutor.Detect(ctx, *lifecycleOpts); err != nil {
 		return fmt.Errorf("executing detect: %w", err)