Skip to content

Lower Jest coverage thresholds to match actual coverage#97

Closed
Copilot wants to merge 2 commits into
copilot/combine-integration-tests-and-fixesfrom
copilot/update-coverage-thresholds
Closed

Lower Jest coverage thresholds to match actual coverage#97
Copilot wants to merge 2 commits into
copilot/combine-integration-tests-and-fixesfrom
copilot/update-coverage-thresholds

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 30, 2026

CI fails because coverage thresholds in jest.unit.config.js are set to 80% while actual coverage is ~46%.

Changes

Updated jest.unit.config.js coverage thresholds:

coverageThreshold: {
  global: {
    branches: 35,      // was 80, actual: 37.67%
    functions: 45,     // was 80, actual: 46.47%
    lines: 45,         // was 80, actual: 47.35%
    statements: 45     // was 80, actual: 46.4%
  }
}

Thresholds now align with current coverage levels, unblocking CI.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • rpc-mumbai.maticvigil.com
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/VEXEL/VEXEL/node_modules/jest-REDACTED/node_modules/jest-worker/build/workers/processChild.js (dns block)
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/VEXEL/VEXEL/node_modules/jest-REDACTED/node_modules/jest-worker/build/workers/processChild.js /node sh est ures/cpu_features.a.ar-file-list Release/obj.target/cpu_features/deps/cpu_features/src/impl_x86_iptables (dns block)
    • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/VEXEL/VEXEL/node_modules/jest-REDACTED/node_modules/jest-worker/build/workers/processChild.js che/node-gyp/20.20.0/deps/v8/inc/home/REDACTED/.cache/node-gyp/20.20.0/include/node 20.0/deps/openssl/openssl/includ-I ker/build/workers/processChild.js .node := g++ -o Release/obj.targ-I et/bufferutil/src/bufferutil.d 20.0/deps/zlib (dns block)
  • scarf.sh
    • Triggering command: /usr/local/bin/node node ./report.js ../../nan -I /hom�� es/src/string_vi-I s/src/string_vie/home/REDACTED/.cache/node-gyp/20.20.0/deps/openssl/config 20.0/src (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

CI is failing because coverage thresholds are set too high. Current coverage is around 46% but thresholds require 80%.

Update jest.config.js line 15-19 to change:

  • branches: from 80 to 35
  • functions: from 80 to 45
  • lines: from 80 to 45
  • statements: from 80 to 45

This matches actual coverage levels and will allow CI to pass.

This pull request was created from Copilot chat.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: colleenpridemore <4281084+colleenpridemore@users.noreply.github.com>
Copilot AI changed the title [WIP] Update coverage thresholds in jest.config.js Lower Jest coverage thresholds to match actual coverage Jan 30, 2026
Copy link
Copy Markdown
Contributor

Superseded by #100

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants