Skip to content

Conversation

@bigcat88
Copy link
Contributor

Summary

  • Add tests for _print_npm_not_found_help function in comfy_cli/command/install.py
  • Tests cover all three OS-specific branches (Windows, macOS, Linux)
  • Tests verify fallback behavior for unknown OS

Test plan

  • All new tests pass locally
  • Tests cover previously uncovered code paths

🤖 Generated with Claude Code

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jan 24, 2026
@bigcat88 bigcat88 force-pushed the test/npm-help-coverage branch from f92f329 to 4ec2aeb Compare January 24, 2026 07:50
Tests cover all three OS-specific branches:
- Windows instructions
- macOS instructions
- Linux instructions (including fallback for unknown OS)
@bigcat88 bigcat88 force-pushed the test/npm-help-coverage branch from 4ec2aeb to a577ebe Compare January 24, 2026 07:52
@codecov
Copy link

codecov bot commented Jan 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #359      +/-   ##
==========================================
+ Coverage   51.33%   52.48%   +1.15%     
==========================================
  Files          32       32              
  Lines        3563     3563              
==========================================
+ Hits         1829     1870      +41     
+ Misses       1734     1693      -41     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bigcat88 bigcat88 merged commit d76c4a2 into main Jan 24, 2026
8 checks passed
@bigcat88 bigcat88 deleted the test/npm-help-coverage branch January 24, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants