fix for coverage (#6872)

Co-authored-by: Andrii Chubatiuk <[email protected]>

Author: AndrewChubatiuk

.github/workflows/ci.yml

@@ -9,6 +9,7 @@ on:
     branches:
       - master
 env:
+  CYPRESS_COVERAGE: "true"
   NODE_VERSION: 18
   YARN_VERSION: 1.22.22
   REDASH_COOKIE_SECRET: "2H9gNG9obnAQ9qnR9BDTQUph6CbXKCzF"
@@ -75,6 +76,8 @@ jobs:
           docker cp tests:/app/junit.xml /tmp/test-results/unit-tests/results.xml
       - name: Upload coverage reports to Codecov
         uses: codecov/codecov-action@v4
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
       - name: Store Test Results
         uses: actions/upload-artifact@v4
         with:
@@ -165,10 +168,6 @@ jobs:
         with:
           node-version: ${{ env.NODE_VERSION }}
           cache: 'yarn'
-      - name: Enable Code Coverage Report For Master Branch
-        if: endsWith(github.ref, '/master')
-        run: |
-          echo "CODE_COVERAGE=true" >> "$GITHUB_ENV"
       - name: Install Dependencies
         run: |
           npm install --global --force yarn@$YARN_VERSION

Dockerfile

@@ -24,9 +24,6 @@ COPY --chown=redash package.json yarn.lock .yarnrc /frontend/
 COPY --chown=redash viz-lib /frontend/viz-lib
 COPY --chown=redash scripts /frontend/scripts
 
-# Controls whether to instrument code for coverage information
-ARG CODE_COVERAGE
-ENV BABEL_ENV=${CODE_COVERAGE:+test}
 RUN yarn --frozen-lockfile --network-concurrency 1;
 COPY --chown=redash client /frontend/client
 COPY --chown=redash webpack.config.js /frontend/

compose.base.yaml

@@ -5,7 +5,6 @@ services:
       args:
         FRONTEND_BUILD_MODE: ${FRONTEND_BUILD_MODE:-2}
         INSTALL_GROUPS: ${INSTALL_GROUPS:-main,all_ds,dev}
-        CODE_COVERAGE: ${CODE_COVERAGE:-"false"}
     volumes:
       - $PWD:${SERVER_MOUNT:-/app}
     command: manage version

compose.yaml

@@ -78,3 +78,4 @@ services:
       COMMIT_INFO_REMOTE: ${COMMIT_INFO_REMOTE:-""}
       CYPRESS_PROJECT_ID: ${CYPRESS_PROJECT_ID:-""}
       CYPRESS_RECORD_KEY: ${CYPRESS_RECORD_KEY:-""}
+      CYPRESS_COVERAGE: ${CYPRESS_COVERAGE:-""}