Skip to content

test: add integration test for comfy node pack#392

Merged
bigcat88 merged 1 commit intomainfrom
test/pack-integration
Mar 27, 2026
Merged

test: add integration test for comfy node pack#392
bigcat88 merged 1 commit intomainfrom
test/pack-integration

Conversation

@bigcat88
Copy link
Copy Markdown
Contributor

End-to-end integration test for comfy node pack. Creates a real git repo with a pyproject.toml, tracked files, a .comfyignore, and an includes directory, then invokes the CLI and inspects the resulting zip.

Verifies five behaviors in one test: git-tracked files are included, .comfyignore patterns exclude files even when git-tracked, untracked files are excluded, [tool.comfy] includes directories are force-included even when not git-tracked, and the zip file itself is not recursively included.

The only thing mocked is the tracking consent prompt. Everything else runs through the real pipeline: CLI entry point, pyproject.toml parsing via extract_node_configuration, git ls-files, .comfyignore via pathspec, and zip_files.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 27, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #392      +/-   ##
==========================================
+ Coverage   74.22%   74.66%   +0.43%     
==========================================
  Files          33       33              
  Lines        3915     3915              
==========================================
+ Hits         2906     2923      +17     
+ Misses       1009      992      -17     

see 3 files 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 f95823d into main Mar 27, 2026
8 checks passed
@bigcat88 bigcat88 deleted the test/pack-integration branch March 27, 2026 19:01
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.

1 participant