diff options
| author | Paul Buetow <paul@buetow.org> | 2026-04-13 08:09:33 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-04-13 08:10:16 +0300 |
| commit | f2dd8d8a515c1a2a220836231ad1a671a5e9b73d (patch) | |
| tree | 5b19585afb01b60d03d24a96b57bc7b986ea4cc0 /internal/stats/stats_test.go | |
| parent | 56002ff942de1bfb0ce467ec37a692b8c4ca01e9 (diff) | |
ask: serialize concurrent CLI with repo lock and stale PID recovery
Add advisory lock under .git/hexai-ask.lock around Taskwarrior execution,
with metadata (PID and process basename) and Linux /proc comm checks to
remove orphan lock files when the recorded holder is gone or not ask.
Extract internal/filelock for shared flock helpers; stats uses it too.
Made-with: Cursor
Diffstat (limited to 'internal/stats/stats_test.go')
| -rw-r--r-- | internal/stats/stats_test.go | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/internal/stats/stats_test.go b/internal/stats/stats_test.go index 47e3068..fc043a5 100644 --- a/internal/stats/stats_test.go +++ b/internal/stats/stats_test.go @@ -9,6 +9,8 @@ import ( "sync" "testing" "time" + + "codeberg.org/snonux/hexai/internal/filelock" ) func TestUpdateAndSnapshot_Single(t *testing.T) { @@ -309,7 +311,7 @@ func TestTakeSnapshot_ZeroWindowSeconds(t *testing.T) { } // TestUpdate_CancelledContext covers the context cancellation branch in -// acquireFileLock when the lock is already held. +// filelock.AcquireExclusive when the lock is already held. func TestUpdate_CancelledContext(t *testing.T) { dir := t.TempDir() t.Setenv("XDG_CACHE_HOME", dir) @@ -320,14 +322,14 @@ func TestUpdate_CancelledContext(t *testing.T) { t.Fatal(err) } - // Hold the lock file to force acquireFileLock to spin. + // Hold the lock file to force filelock.AcquireExclusive to spin. lockPath := filepath.Join(statsDir, lockFileName) lf, err := os.OpenFile(lockPath, os.O_CREATE|os.O_RDWR, 0o600) if err != nil { t.Fatal(err) } defer func() { _ = lf.Close() }() - unlock, err := acquireFileLock(context.Background(), lf) + unlock, err := filelock.AcquireExclusive(context.Background(), lf) if err != nil { t.Fatal(err) } |
