From 23fda434ff373699bbbd2a5de9fdd03283aa1e99 Mon Sep 17 00:00:00 2001 From: rcmerci Date: Fri, 24 Apr 2026 14:29:15 +0800 Subject: [PATCH] fix(cli-e2e): dont delete skill.md in e2e cleanup --- cli-e2e/spec/non_sync_cases.edn | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/cli-e2e/spec/non_sync_cases.edn b/cli-e2e/spec/non_sync_cases.edn index 8e76d98849..f732c22fcc 100644 --- a/cli-e2e/spec/non_sync_cases.edn +++ b/cli-e2e/spec/non_sync_cases.edn @@ -1030,13 +1030,18 @@ :stdout-not-contains ["\"status\"" "\"data\""]}, :covers {:commands ["skill show"], :options {:global ["--output"]}}, :tags [:skill]} + ;; Repo root already contains tracked `.agents/skills/logseq-cli/SKILL.md`, + ;; so this local-install case must run from an isolated cwd. {:id "skill-install-local", + :setup + ["rm -rf ./tmp/cli-e2e-skill-install-local" + "mkdir -p ./tmp/cli-e2e-skill-install-local"], :cmds - ["{{cli}} skill install >/dev/null" - "python3 -c 'import pathlib; p=pathlib.Path(\".agents/skills/logseq-cli/SKILL.md\"); print(\"installed\" if p.exists() else \"missing\")'"], + ["cd ./tmp/cli-e2e-skill-install-local && {{cli}} skill install >/dev/null" + "python3 -c 'import pathlib; p=pathlib.Path(\"./tmp/cli-e2e-skill-install-local/.agents/skills/logseq-cli/SKILL.md\"); print(\"installed\" if p.exists() else \"missing\")'"], :expect {:exit 0, :stdout-contains ["installed"]}, :covers {:commands ["skill install"], :options {:skill []}}, - :cleanup ["rm -rf ./.agents/skills/logseq-cli"], + :cleanup ["rm -rf ./tmp/cli-e2e-skill-install-local"], :tags [:skill]} {:id "skill-install-global-preserves-other-skills", :setup