node icon indicating copy to clipboard operation
node copied to clipboard

test: simplify `test-cli-node-options-docs`

Open aduh95 opened this issue 2 weeks ago • 2 comments

The current version is hard to reason about with non-const variables, and use of === false. Also we need a change for https://github.com/nodejs/node/pull/60906 to correctly detect HAVE_SQLITE in the code.

aduh95 avatar Dec 09 '25 14:12 aduh95

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 88.53%. Comparing base (0ffa607) to head (888ab08). :warning: Report is 17 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #61006      +/-   ##
==========================================
- Coverage   92.23%   88.53%   -3.71%     
==========================================
  Files         343      703     +360     
  Lines      139073   208432   +69359     
  Branches    22251    40195   +17944     
==========================================
+ Hits       128277   184534   +56257     
- Misses      10569    15905    +5336     
- Partials      227     7993    +7766     

see 481 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Dec 09 '25 15:12 codecov[bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/70457/

nodejs-github-bot avatar Dec 09 '25 19:12 nodejs-github-bot

Landed in 029ebac19d1c45069439c4e4ac5ffb5a944827d2

nodejs-github-bot avatar Dec 11 '25 14:12 nodejs-github-bot