mirror of
https://github.com/Ar9av/obsidian-wiki.git
synced 2026-09-14 20:36:34 +08:00
Arnav
9c260f78aa
The daily-update skill's Setup Mode references scripts/daily-update.sh, the launchd plist, and wiki-notify.sh via $OBSIDIAN_WIKI_REPO/scripts/ — same class of gap as #143 (the Stop hook), which didn't cover this directory. force-include scripts/ under obsidian_wiki/_data/scripts/ so pip/uv installs actually have it, and exclude __pycache__/*.pyc from the wheel target since force-include copies straight from the working tree (a local scripts/__pycache__ was otherwise leaking stray .pyc files in).
80 lines
3.0 KiB
Python
80 lines
3.0 KiB
Python
"""Regression guard for issue #152.
|
|
|
|
The daily-update skill's Setup Mode references helper scripts under
|
|
``scripts/`` (daily-update.sh, the launchd plist, wiki-notify.sh) that are
|
|
committed to the repo but were never force-included into the published wheel
|
|
— the same packaging-gap class as #143 (the Stop hook), which that fix did
|
|
not cover. These tests pin the packaging config that force-includes
|
|
``scripts/`` and the skill instructions that resolve it via
|
|
``$OBSIDIAN_WIKI_REPO``.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from pathlib import Path
|
|
import unittest
|
|
|
|
try:
|
|
import tomllib
|
|
except ModuleNotFoundError: # Python < 3.11
|
|
tomllib = None
|
|
|
|
|
|
ROOT = Path(__file__).resolve().parents[1]
|
|
SCRIPTS_DIR = ROOT / "scripts"
|
|
PACKAGED_SCRIPTS_DEST = "obsidian_wiki/_data/scripts"
|
|
|
|
REFERENCED_SCRIPTS = (
|
|
"daily-update.sh",
|
|
"com.obsidian-wiki.daily-update.plist",
|
|
"wiki-notify.sh",
|
|
)
|
|
|
|
|
|
@unittest.skipIf(tomllib is None, "tomllib requires Python 3.11+")
|
|
class ScriptsPackagingTest(unittest.TestCase):
|
|
def setUp(self) -> None:
|
|
self.pyproject = tomllib.loads((ROOT / "pyproject.toml").read_text())
|
|
|
|
def test_referenced_scripts_exist(self) -> None:
|
|
for name in REFERENCED_SCRIPTS:
|
|
with self.subTest(script=name):
|
|
self.assertTrue((SCRIPTS_DIR / name).is_file(), f"{name} must exist to be bundled")
|
|
|
|
def test_wheel_force_includes_scripts_dir(self) -> None:
|
|
mapping = self.pyproject["tool"]["hatch"]["build"]["targets"]["wheel"]["force-include"]
|
|
self.assertEqual(
|
|
mapping.get("scripts"),
|
|
PACKAGED_SCRIPTS_DEST,
|
|
"wheel must force-include scripts/ under _data/scripts/",
|
|
)
|
|
|
|
def test_scripts_dir_not_excluded_from_sdist(self) -> None:
|
|
# Unlike /.claude, scripts/ isn't pruned, so the default VCS-tracked
|
|
# sdist already carries it — the wheel (built from the sdist) can
|
|
# force-include it from there. If this ever gets added to the sdist
|
|
# exclude list, it would also need a matching force-include re-add.
|
|
sdist_exclude = self.pyproject["tool"]["hatch"]["build"]["targets"]["sdist"]["exclude"]
|
|
self.assertNotIn("/scripts", sdist_exclude)
|
|
|
|
def test_wheel_excludes_pycache_from_force_included_dirs(self) -> None:
|
|
# force-include copies straight from the working tree (not a
|
|
# VCS-filtered view), so a local scripts/__pycache__ can otherwise
|
|
# leak stray .pyc files into the published wheel.
|
|
wheel_exclude = self.pyproject["tool"]["hatch"]["build"]["targets"]["wheel"]["exclude"]
|
|
self.assertIn("**/__pycache__", wheel_exclude)
|
|
|
|
|
|
class DailyUpdateSkillTest(unittest.TestCase):
|
|
def setUp(self) -> None:
|
|
self.skill = (ROOT / ".skills" / "daily-update" / "SKILL.md").read_text()
|
|
|
|
def test_skill_resolves_scripts_via_repo_var(self) -> None:
|
|
for name in REFERENCED_SCRIPTS:
|
|
with self.subTest(script=name):
|
|
self.assertIn(f"$OBSIDIAN_WIKI_REPO/scripts/{name}", self.skill)
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main()
|