From 36ff8096f35d6ab8277b013d0889181a2e8631b9 Mon Sep 17 00:00:00 2001
From: Michael McGuinness <mmcguinness@aarete.com>
Date: Thu, 22 May 2025 09:16:09 +0000
Subject: [PATCH] Merged in bugfix/dockerbuild (pull request #149)

Doc Build Update

* dockerbuild
---
 internal/test/container.go | 16 +++++++---------
 scripts/Taskfile.yml       |  1 -
 2 files changed, 7 insertions(+), 10 deletions(-)

diff --git a/internal/test/container.go b/internal/test/container.go
index 9e391789..b8edea90 100644
--- a/internal/test/container.go
+++ b/internal/test/container.go
@@ -42,14 +42,10 @@ const (
 	dockerfilePath = "build/Dockerfile"
 )
 
-type ContainerLock struct {
-	imageBuild sync.Once
-}
-
-var containerLock = &ContainerLock{}
+var imageBuild sync.Once
 
 func buildImage(t testing.TB, ctx context.Context) {
-	containerLock.imageBuild.Do(func() {
+	imageBuild.Do(func() {
 		dockerContextPath, err := os.Getwd()
 		require.NoError(t, err, "Failed to get current directory")
 
@@ -98,18 +94,20 @@ func buildImage(t testing.TB, ctx context.Context) {
 		}
 
 		buildContext, err := archive.TarWithOptions(dockerContextPath, &archive.TarOptions{
-			Compression:     archive.Uncompressed,
-			NoLchown:        true,
 			ExcludePatterns: excludePatterns,
 		})
 		require.NoError(t, err)
 		defer buildContext.Close()
 
-		_, err = cli.ImageBuild(ctx, buildContext, types.ImageBuildOptions{
+		out, err := cli.ImageBuild(ctx, buildContext, types.ImageBuildOptions{
 			Dockerfile: dockerfilePath,
 			Tags:       []string{imageTag},
 		})
 		require.NoError(t, err)
+		defer func() {
+			err := out.Body.Close()
+			require.NoError(t, err)
+		}()
 	})
 }
 
diff --git a/scripts/Taskfile.yml b/scripts/Taskfile.yml
index 65ce5c7a..3ce7a202 100644
--- a/scripts/Taskfile.yml
+++ b/scripts/Taskfile.yml
@@ -26,7 +26,6 @@ includes:
 tasks:
   fullsuite:
     - task: build:deps
-    - task: docker:build
     - task: fullsuite:test
   fullsuite:test:
     internal: true