Commit b103dfc
Fix integration test to properly demonstrate package exclusion bug
The test was extracting from the entire workspace, which caused both
modules to be treated as input packages. This meant their ModDirs were
added to wantedRoots even without the fix, so the bug never manifested.
Changed test.py to extract only mainmodule packages using a specific
command pattern. This ensures:
- Only mainmodule is in the initial input packages
- configmodule is visited as a dependency
- Without the fix, configmodule's ModDir is NOT in wantedRoots
- The relative path check produces ".." and incorrectly excludes it
Test results:
- WITHOUT fix: 2 files extracted (configmodule missing) ❌
- WITH fix: 4 files extracted (all present) ✅
This addresses the reviewer's feedback that the test was passing
without any code changes.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>1 parent 438eac7 commit b103dfc
1 file changed
+3
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
0 commit comments