Merge pull request #2236 from samalba/ci-gotest

ci: run go unit tests from the dagger plan

Makefile

@@ -29,8 +29,8 @@ install: # Install a dev dagger binary
 	go install -ldflags '-X go.dagger.io/dagger/version.Revision=$(GIT_REVISION)' ./cmd/dagger
 
 .PHONY: test
-test: # Run all tests
-	go test -race -v ./...
+test: dagger # Run all tests
+	./cmd/dagger/dagger do test
 
 .PHONY: golint
 golint: dagger # Go lint

ci.cue

@@ -73,13 +73,7 @@ dagger.#Plan & {
 			source:  _source
 			package: "./..."
 
-			// FIXME: doesn't work with CGO_ENABLED=0
-			// command: flags: "-race": true
-
-			env: {
-				// FIXME: removing this complains about lack of gcc
-				CGO_ENABLED: "0"
-			}
+			command: flags: "-race": true
 		}
 
 		lint: {

pkg/universe.dagger.io/go/image.cue

@@ -13,7 +13,11 @@ _#DefaultVersion: "1.18"
 
 	packages: [pkgName=string]: version: string | *""
 	// FIXME Remove once golang image include 1.18 *or* go compiler is smart with -buildvcs
-	packages: git: _
+	packages: {
+		git: _
+		// For GCC and other possible build dependencies
+		"alpine-sdk": _
+	}
 
 	// FIXME Basically a copy of alpine.#Build with a different image
 	// Should we create a special definition?