From a0accff258b014d95fff115be465b31a2d92bcb6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E7=8E=89=E6=BE=9C?= Date: Mon, 17 Aug 2026 21:33:35 +0800 Subject: [PATCH] test(skills): assert claim mode matches source across platforms --- internal/upgrade/skill_publish_rename_test.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/internal/upgrade/skill_publish_rename_test.go b/internal/upgrade/skill_publish_rename_test.go index 6b6a8c72..b60104d6 100644 --- a/internal/upgrade/skill_publish_rename_test.go +++ b/internal/upgrade/skill_publish_rename_test.go @@ -372,6 +372,11 @@ func TestCrossPlatformCoverageNoReplaceRenameFallback(t *testing.T) { source := filepath.Join(dir, "source") destination := filepath.Join(dir, "destination") seedUpgradeSkill(t, source, "payload", false) + sourceInfo, err := os.Lstat(source) + if err != nil { + t.Fatal(err) + } + sourceMode := sourceInfo.Mode().Perm() testseam.Swap(t, &skillPathRenameNoReplaceAtomic, func(string, string) error { return errNoReplaceRenameUnsupported }) calls := 0 originalRename := skillPathRename @@ -388,8 +393,8 @@ func TestCrossPlatformCoverageNoReplaceRenameFallback(t *testing.T) { assertUpgradeSkillContent(t, destination, "payload") assertSourceConsumed(t, source) info, err := os.Lstat(destination) - if err != nil || info.Mode().Perm() != 0o755 { - t.Fatalf("published mode = %v, %v; want 0755", info.Mode(), err) + if err != nil || info.Mode().Perm() != sourceMode { + t.Fatalf("published mode = %v, %v; want source mode %v", info.Mode(), err, sourceMode) } })