hooks: fix #2382 regression / improve tests and docs

[marcelotduarte]

Closes #2435
Closes #2433
#2376

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.77%. Comparing base (c3b273e) to head (6e80828).
Report is 71 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2443   +/-   ##
=======================================
  Coverage   79.77%   79.77%           
=======================================
  Files          27       27           
  Lines        4089     4089           
=======================================
  Hits         3262     3262           
  Misses        827      827           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[marcelotduarte]

Please do a test without freeze_support so you can see the message (before it was in a loop).

[ntindle]

Won't be able to test today unfortunately, Just realized its almost 1am

[ntindle]

Works on windows

[ntindle]

testing mac next

[ntindle]

On mac, when commenting out multiprocessing.freeze_support() 5 of the 7 multiprocessing tests still pass
With the freeze_support enabled, all tests pass.

The failing tests are sample1-spawn and sample2-spawn. They fail in expected ways. The passing tests have no output but that may be a config issue on my side (or false passes)

[marcelotduarte]

The failing tests are sample1-spawn and sample2-spawn. They fail in expected ways.

They fail but do not enter an infinite loop as before, right?

[ntindle]

Correct