Skip to content

Commit 2fd9aaa

Browse files
mnriemCopilot
andcommitted
Cover owned generic dev links during force reinstall
Exercise successful force reinstall for owned command and skill dev symlinks alongside regular outputs, complementing cross-extension retargeting regressions. Assisted-by: GitHub Copilot (model: GPT-6 Sol, autonomous) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent ca3d27e commit 2fd9aaa

1 file changed

Lines changed: 11 additions & 3 deletions

File tree

‎tests/integrations/test_integration_generic.py‎

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1324,17 +1324,25 @@ def test_generic_extension_does_not_overwrite_existing_command_or_skill(
13241324

13251325

13261326
@pytest.mark.parametrize("skills", [False, True])
1327+
@pytest.mark.parametrize("dev_symlink", [False, True])
13271328
def test_generic_extension_force_reinstall_only_replaces_owned_artifacts(
1328-
tmp_path, generic_extension, skills,
1329+
tmp_path, generic_extension, skills, dev_symlink,
13291330
):
13301331
project = generic_project(tmp_path, skills=skills)
13311332
manager = ExtensionManager(project)
1332-
manager.install_from_directory(generic_extension, "1.0.0")
1333+
manager.install_from_directory(generic_extension, "1.0.0", link_commands=dev_symlink)
13331334
artifact = project / ".custom/commands" / (
13341335
"speckit-sample-run/SKILL.md" if skills else "speckit.sample.run.md"
13351336
)
1336-
manager.install_from_directory(generic_extension, "1.0.0", force=True)
1337+
if dev_symlink and not artifact.is_symlink():
1338+
pytest.skip("dev-mode symlinks are unavailable")
1339+
manager.install_from_directory(
1340+
generic_extension, "1.0.0", force=True, link_commands=dev_symlink,
1341+
)
13371342
assert artifact.is_file()
1343+
assert artifact.is_symlink() is dev_symlink
1344+
if dev_symlink:
1345+
assert artifact.resolve().is_relative_to((manager.extensions_dir / "sample").resolve())
13381346
assert manager.remove("sample")
13391347
assert not artifact.exists()
13401348

0 commit comments

Comments
 (0)