Merge pull request #313 from otiai10/develop

Fix clearlinux test case

.github/workflows/go-ci.yml

@@ -51,7 +51,7 @@ jobs:
       - name: Coverage Test
         run: go test -v -cover -race -coverprofile=coverage -covermode=atomic
       - name: Upload coverage
-        uses: codecov/codecov-action@v3
+        uses: codecov/codecov-action@v4
         with:
           file: ./coverage
-          verbose: true
+          token: ${{ secrets.CODECOV_TOKEN }}

test/runtime

@@ -128,6 +128,10 @@ function test_docker_runtimes() {
       fi
     fi
     echo "┌───────────── ${testcase}"
+    # q: Is following line correct?
+    # XXX: Since clearlinux docker image doesn't support ARM64 architecture, as of 2024-07-26a,
+    #      we need to add `--platform linux/amd64` to the `docker build` command when we build it on macos M3 chip.
+    # if [ ${testcase} == "clearlinux" ]; then DOCKER_PLATFORM_ARG="--platform linux/amd64"; fi
     if [ -n "${BUILD}" ]; then
       echo "│ [Docker] Building image..."
       docker build . ${NO_CACHE} -f ${runtime} -t gosseract/test:${testcase} ${QUIET} | sed "s/^/│ /"

test/runtimes/clearlinux.Dockerfile

@@ -12,12 +12,16 @@ RUN swupd bundle-add \
 #     This part should be replaced by curl,
 #     which is installed by default.
 RUN swupd bundle-add wget
+ENV TESSDATA_PREFIX /usr/share/tessdata
+RUN mkdir -p ${TESSDATA_PREFIX}
 RUN wget \
         -O eng.traineddata \
         https://github.com/tesseract-ocr/tessdata/blob/main/eng.traineddata?raw=true \
-    && mv eng.traineddata /usr/share/tessdata
+    && mv eng.traineddata ${TESSDATA_PREFIX}/eng.traineddata
 # }}}
 
+ENV GOPATH /go
+
 RUN mkdir -p ${GOPATH}/src/github.com/otiai10
 ADD . ${GOPATH}/src/github.com/otiai10/gosseract
 WORKDIR ${GOPATH}/src/github.com/otiai10/gosseract