32205961824
72%

Ran 19 Aug 2026 01:43AM UTC

Jobs 1

Files 2

Run time 1min

Badge

Embed ▾

Committed 19 Aug 2026 01:43AM UTC coverage: 70.326%. Remained the same

Build # 32205961824

Build Type

push

github

Committed by

web-flow

Commit Message

replace sleeps in watcher tests with event synchronisation (#7)

The four watcher tests guarded themselves with os.Getenv("GOOS") == "darwin",
which never matched: GOOS is a build variable, not something the go tool exports
into the test environment. The tests therefore ran everywhere, on fixed sleeps
that were tuned for the skip that never happened.

Waiting is now driven by the events themselves. A shared waitForEvent helper
consumes the channel until the expected path shows up, returning what it skipped
so a test can assert on what should not have been reported, and gives up on a
single deadline instead of a chain of sleeps.

The 100ms and 200ms "let the watcher initialize" sleeps are gone, the watched
path is registered with the kernel before NewFileWatcher and WatchRecursive
return, so events raised afterwards are queued regardless of when the consuming
goroutine is scheduled. TestWatchRecursive no longer needs its warmup file and
timed drain loop, since waitForEvent skips past unrelated events on its own.

The event buffer goes from 10 to 100, so an expected event cannot be dropped by
the callback's non-blocking send while the test is busy.

Package tests drop from about 2.6s to 1.3s.

Coverage Stats

237 of 337 relevant lines covered (70.33%)

4.19 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	32205961824.1	19 Aug 2026 01:43AM UTC	2	70.33	GitHub Action Run

go-pkgz / fileutils / 32205961824
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 32205961824

go-pkgz / fileutils / 32205961824 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 32205961824

go-pkgz / fileutils / 32205961824
72%

README BADGES
x