zsp/everything-claude-code
1
0
Fork 0
mirror of https://github.com/affaan-m/everything-claude-code.git synced 2026-05-20 07:43:07 +08:00
Code Issues Packages Projects Releases Wiki Activity

test(install-targets): add positive rules assertion to claude-project foreign-path test

Browse Source 
Addresses CodeRabbit review: the negative-only assertions could have
passed on an empty plan. Add a positive assertion that the non-foreign
'rules' path is still planned under .claude/rules/ecc so regression to
zero ops would fail loudly.
This commit is contained in:
Mhd Ghaith Al Abtah
2026-05-19 19:50:48 +04:00
committed by Affaan Mustafa
parent 7004a66243
commit b2c2616ab4
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/lib/install-targets.test.js
View File

@@ -943,6 +943,13 @@ function runTests() {
], ],
}); });
assert.ok(
plan.operations.some(operation => (
normalizedRelativePath(operation.sourceRelativePath) === 'rules'
&& operation.destinationPath === path.join(projectRoot, '.claude', 'rules', 'ecc')
)),
'Should still include non-foreign rules path (guards against empty-plan regression)'
);
assert.ok( assert.ok(
!plan.operations.some(operation => ( !plan.operations.some(operation => (
normalizedRelativePath(operation.sourceRelativePath) === '.cursor' normalizedRelativePath(operation.sourceRelativePath) === '.cursor'