Commit 8ea9658
committed
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.1 parent 438eac7 commit 8ea9658
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