liangshuo-1 25454f498b test(update): isolate stamp writes from real ~/.lark-cli/skills.stamp (#858) ...

Five tests in cmd/update mocked SkillsUpdateOverride to return success
and let runSkillsAndStamp call WriteStamp, but did not isolate
LARKSUITE_CLI_CONFIG_DIR. Each run clobbered the real
~/.lark-cli/skills.stamp with the mock version ("2.0.0" or "1.0.0"),
causing skillscheck to fire a misleading drift notice on every
subsequent lark-cli invocation.

Add t.Setenv("LARKSUITE_CLI_CONFIG_DIR", t.TempDir()) at the top of:
  - TestUpdateNpm_JSON
  - TestUpdateNpm_Human
  - TestUpdateForce_JSON
  - TestUpdateDevVersion_JSON
  - TestUpdateWindows_NpmSuccess_JSON

Scope is limited to tests that mock SkillsUpdateOverride to success;
tests that invoke real npx are pre-existing and out of scope here.

Change-Id: I7a78a6c70f276b51333253acc115e0109c01a851

2026-05-13 13:52:22 +08:00

..

update_test.go

test(update): isolate stamp writes from real ~/.lark-cli/skills.stamp (#858)

2026-05-13 13:52:22 +08:00

update.go

feat: add skills version drift notice and unify update flow (#723)

2026-05-07 10:52:35 +08:00