lee/graphify
1
0
Fork 0
mirror of https://github.com/safishamsi/graphify.git synced 2026-09-23 14:05:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(watch): pin update discovers newly-added files/dirs (#1837)

Browse Source 
#1837 (update silently fails to discover new files) was the same
nested-gitignore scoping bug (#1873/#1880), already fixed. The existing
test only does a single build; add the build -> add new file+dir ->
update -> discovered sequence the report describes, with a nested `*`
scratch dir as a scoping guard. No production change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
safishamsi
2026-07-14 23:20:36 +01:00
co-authored by Claude Opus 4.8
parent 060dd6317f
commit f80e4fac35
1 changed files with 32 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_watch.py
+32
View File
@@ -219,6 +219,38 @@ def test_update_rebuilds_with_nested_star_gitignore(tmp_path):
assert not any("scratch/junk.py" in s for s in sources)
def test_update_discovers_newly_added_files_and_dirs(tmp_path):
"""#1837: after an initial build, a plain `graphify update` (full re-scan, no
change-list) must discover brand-new files AND new directories. The reported
silent no-op was the #1873 nested-gitignore scoping bug zeroing the re-scan;
this pins the build -> add -> update -> discovered sequence the earlier test
(single build) did not cover, with a nested `*` scratch dir as a guard."""
import json
from graphify.watch import _rebuild_code
corpus = tmp_path / "corpus"
(corpus / "src").mkdir(parents=True)
(corpus / "src" / "a.py").write_text("def alpha(): return 1\n", encoding="utf-8")
assert _rebuild_code(corpus, acquire_lock=False) is True
# Add a brand-new file and a brand-new nested directory after the first build,
# plus a scratch dir that ignores only itself.
(corpus / "src" / "new.py").write_text("def added(): return 2\n", encoding="utf-8")
(corpus / "monitor").mkdir()
(corpus / "monitor" / "dash.py").write_text("def board(): return 3\n", encoding="utf-8")
(corpus / "scratch").mkdir()
(corpus / "scratch" / ".gitignore").write_text("*\n", encoding="utf-8")
(corpus / "scratch" / "junk.py").write_text("x = 1\n", encoding="utf-8")
assert _rebuild_code(corpus, acquire_lock=False) is True
sources = {n.get("source_file", "") for n in
json.loads((corpus / "graphify-out" / "graph.json").read_text())["nodes"]}
assert any("src/new.py" in s for s in sources), "new file not discovered by update (#1837)"
assert any("monitor/dash.py" in s for s in sources), "new directory not discovered (#1837)"
assert not any("scratch/junk.py" in s for s in sources)
def test_rebuild_honors_persisted_excludes(tmp_path):
"""#1886: `--exclude` recorded at extract time must survive into update/watch/
hook rebuilds. Before the fix only the initial scan applied the excludes, so