ci: add codecov for coverage

arusakov · Oct 2, 2024 · a2fc660 · a2fc660
1 parent d22afb5
commit a2fc660
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -19,4 +19,10 @@ jobs:
         env:
           REDIS_HOST: redis
           REDIS_PORT: 6379
+      - uses: codecov/codecov-action@v4
+        with:
+          fail_ci_if_error: true
+          files: ./coverage/lcov.info
+          token: ${{ secrets.CODECOV_TOKEN }}
+          verbose: true # optional (default = false)
 
diff --git a/package.json b/package.json
@@ -23,7 +23,7 @@
     "build": "tsc -p .",
     "compile": "yarn build --noEmit",
     "test:all": "yarn test ./test/*.test.ts",
-    "test:coverage": "c8 --reporter=text --reporter=text-summary yarn test:all",
+    "test:coverage": "c8 --reporter=lcovonly --reporter=text yarn test:all",
     "test": "node --test --test-concurrency=1 --require=ts-node/register"
   },
   "engines": {

diff --git a/test/utils.ts b/test/utils.ts
@@ -4,11 +4,10 @@ import { WSDiscovery } from "../src"
 import { ID } from "../src/constants"
 
 export const createRedis = () => {
-
   return new Redis({
     lazyConnect: true,
     host: process.env.REDIS_HOST,
-    port: Number(process.env.REDIS_PORT),
+    port: Number(process.env.REDIS_PORT) || undefined,
   })
 }