Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.85%. Comparing base (649b2c8) to head (9852b63).
Report is 1 commits behind head on master.

@@            Coverage Diff             @@
##           master     #708       /-   ##
==========================================
  Coverage   95.78%   95.85%    0.06%     
==========================================
  Files          23       23              
  Lines        1686     1712       26     
  Branches      194      198        4     
==========================================
  Hits         1615     1641       26     
  Misses         46       46              
  Partials       25       25              

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

Hmm, there's a weird test failure... Otherwise, looks good.

Also, if you'd like to try and upgrade pytest, there is a workaround mentioned in this comment I've yet to try: pytest-dev/pytest-forked#67 (comment)

Hmm, there's a weird test failure... Otherwise, looks good.

This test failure seems to be a pytest internal error and happens on macos only. All tests work well on my machine and other linux ci action. I don't have a macos environment to debug it...

Maybe it is related to pytest-dev/pytest-forked#78. Traceback shows the same "Bad file descriptor" message when running action with coverage disabled.

The test passed after setting bind address to 0.0.0.0. 19e89ad

Have no idea why binding to localhost in this test causes pytest internal error on macos.