ci: add mac os to matrix tests

Signed-off-by: Boris Glimcher <[email protected]>

Author: glimchb

.github/workflows/test.yml

@@ -10,7 +10,7 @@ jobs:
       fail-fast: false
       matrix:
         go: [ '1.19', '1.20', '1.21' ]
-        os: [windows-latest, ubuntu-latest]
+        os: [windows-latest, macos-latest, ubuntu-latest]
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v3
@@ -22,6 +22,12 @@ jobs:
           version: v1.15.0
           install-only: true
 
+      - name: Setup docker (missing on MacOS)
+        if: contains(runner.os, 'macos')
+        run: |
+          brew install docker
+          colima start
+
       - run: |
           go version
           go env

magefiles/test.go

@@ -90,14 +90,13 @@ func testImpl(impl string) error {
 		return errors.New("unknown `gokv.Store` implementation")
 	}
 
-	// TODO: until docker images for windows appear, skip those test for windows
-	if dockerCmd != "" && runtime.GOOS == "windows" {
-		return nil
-	}
-
 	// For some implementations there's no way to test with a Docker container yet.
 	// For them we skip the Docker stuff but still execute the tests, which can skip on connection error and we can see the skips in the test results.
 	if dockerCmd != "" {
+		// TODO: until docker images for windows and darwin appear, skip those tests
+		if runtime.GOOS == "windows" || runtime.GOOS == "darwin" {
+			return nil
+		}
 		// Start Docker container
 		var out string
 		out, err = script.Exec(dockerCmd).String()