Correct four inaccurate comments in cancel_test.go and rename a constant (closes #33)

Fix the comments the re-review of
#6 found misdescribing their
tests; no test's behaviour changes.

State the property the walkClock tests rely on, that the index load's
Done cost does not grow with the record count, instead of a wrong fixed
figure. Describe poolUnwind so it is true of every use, and mark the
tests that have no bound. Record that hashWorker's results-send exit is
reachable through stop but has no test that fails without it. Rename
walkCancelInFlightDirs to walkCancelInFlightFiles, since it counts
files. Note at the top why the file departs from the
one-test-file-per-source-file convention.

Model: opus-4-8 (implementation); opus-5-5 (rework)

cancel_test.go

@@ -706,12 +706,12 @@ func TestFeedHashJobsClosesJobsWhenCancelled(t *testing.T) {
 // hashWorker's other cancellation exit, abandoning the send of a
 // result, is reachable from the scan: stop cancels the pool before it
 // drains results, so a worker waiting on that send can leave through
-// it. TestScanHashWriteFailureUnwindsPool reaches it in some runs only,
+// it. The tests that stop a scan mid-hash, among them
-// depending on whether the workers have filled the results channel when
+// TestScanHashWriteFailureUnwindsPool, reach it in some runs only,
-// stop runs, and no test fails without it, since stop's drain frees a
+// depending on timing, and no test fails without it, since stop's
-// waiting worker anyway. This test, for its part, catches a removed
+// drain frees a waiting worker anyway. This test, for its part, catches
-// drop check in some runs only: a worker that hashes the run anyway
+// a removed drop check in some runs only: a worker that hashes the run
-// then picks at random between sending the result and leaving.
+// anyway then picks at random between sending the result and leaving.
 func TestHashWorkerDropsQueuedRuns(t *testing.T) {
 	t.Parallel()