fix: disable race check for go tests

Author: riccardopinosio

.github/workflows/release.yaml

@@ -33,7 +33,7 @@ jobs:
         run: |
           go mod download
       - name: Build and run tests
-        run: ./scripts/run-unit-tests.sh ALL
+        run: ./scripts/run-unit-tests.sh
       - name: Install goveralls
         run: go install github.com/mattn/goveralls@latest
       - name: Send coverage

.github/workflows/test.yaml

@@ -23,13 +23,13 @@ jobs:
         run: |
           go mod download
       - name: Build and run tests
-        run: ./scripts/run-unit-tests.sh ALL
+        run: ./scripts/run-unit-tests.sh
       - name: Install goveralls
         run: go install github.com/mattn/goveralls@latest
       - name: Send coverage
         env:
           COVERALLS_TOKEN: ${{ secrets.COVERALLS_TOKEN }}
-        run: goveralls -coverprofile=./testTarget/unit/cover.out -service=github
+        run: goveralls -coverprofile=./testTarget/unit/cover.final.out -service=github
       - uses: actions/upload-artifact@v4
         with:
           name: libtokenizers.a

hugot_go_test.go

@@ -1,3 +1,5 @@
+//go:build GO || ALL
+
 package hugot
 
 import (

scripts/run-unit-tests-container.sh

@@ -6,16 +6,25 @@ cd /build && \
 mkdir -p /test/unit && \
 go run ./testData/downloadModels.go
 
-echo HUGOT_BUILD_TAG is "$HUGOT_BUILD_TAG"
+gotestsum --junitfile=/test/unit/unit-onnx.xml \
+    --jsonfile=/test/unit/unit-onnx.json \
+    -- tags=ORT -coverprofile=/test/unit/cover-onnx.out -coverpkg ./... -timeout 60m -race -covermode=atomic ./...
 
-if [[ -n $HUGOT_BUILD_TAG ]]; then 
-    echo "running with -tags=ALL"
-    gotestsum --junitfile=/test/unit/unit.xml --jsonfile=/test/unit/unit.json -- -tags=ALL -coverprofile=/test/unit/cover.out.pre -coverpkg ./... -timeout 60m -race -covermode=atomic ./...
-else 
-    echo "running without build tags"
-    gotestsum --junitfile=/test/unit/unit.xml --jsonfile=/test/unit/unit.json -- -coverprofile=/test/unit/cover.out.pre  ./... -timeout 60m -race -covermode=atomic ./...
-fi
+gotestsum --junitfile=/test/unit/unit-xla.xml \
+    --jsonfile=/test/unit/unit-xla.json \
+    -- -tags=XLA -coverprofile=/test/unit/cover-xla.out -coverpkg ./... -timeout 60m -race -covermode=atomic ./...
 
-grep -v "downloadModels.go" /test/unit/cover.out.pre > /test/unit/cover.out && rm  /test/unit/cover.out.pre
+gotestsum --junitfile=/test/unit/unit-go.xml \
+    --jsonfile=/test/unit/unit-go.json \
+    -- -tags=GO -coverprofile=/test/unit/cover-go.out -coverpkg ./... -timeout 60m -covermode=atomic ./...
 
-echo Done.
+# Merge coverage files manually
+head -n 1 /test/unit/cover-onnx.out > /test/unit/cover.out
+tail -n +2 /test/unit/cover-onnx.out >> /test/unit/cover.out
+tail -n +2 /test/unit/cover-xla.out >> /test/unit/cover.out
+tail -n +2 /test/unit/cover-go.out >> /test/unit/cover.out
+
+# Optionally, remove downloadModels.go lines
+grep -v "downloadModels.go" /test/unit/cover.out > /test/unit/cover.final.out
+
+echo Done.

scripts/run-unit-tests.sh

@@ -15,20 +15,6 @@ mkdir -p "$test_folder"
 host_uid=$(id -u "$USER")
 export host_uid
 
-if [ $# -eq 0 ]; then
-    echo "running without build tags"
-    export HUGOT_BUILD_TAG=""
-else
-    first_arg=$1
-    if [[ $first_arg == "ALL" ]]; then
-        echo "running with build tag ALL"
-        export HUGOT_BUILD_TAG="ALL"
-    else
-        echo "build tag $first_arg is not supported"
-        exit 1
-    fi
-fi
-
 # build with compose
 docker compose -f "$src_dir/compose-test.yaml" build hugot && \
 docker compose -f "$src_dir/compose-test.yaml" build hugot-test