ci: use tox in ci

[moe-ad]

Related to #296

[moe-ad]

Moving to "Ready for review" so I can observe workflow runs.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.44%. Comparing base (06567a3) to head (2364517).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2031      +/-   ##
==========================================
- Coverage   88.61%   86.44%   -2.17%     
==========================================
  Files          90       90              
  Lines       10231    10231              
==========================================
- Hits         9066     8844     -222     
- Misses       1165     1387     +222     

[moe-ad]

Adding this issue too:

• Pytest-cov is extremely slow with xdist too pytest-dev/pytest-cov#669

[PProfizi]

Yeah. pytest-cov is the culprit and not any of the other plugins. Another related issue:

• 4-5x slowdown using pytest-cov pytest-dev/pytest-cov#159

Thanks @moe-ad, does that mean we cannot have coverage with the proposed changes?

[moe-ad]

@PProfizi I can't really say for now. I am still trying out different things to see if we can get coverage to work normally.

[moe-ad]

@jorgepiloto @PProfizi looks like we are good to go.

[jorgepiloto]

Thanks for the very big effort here, @moe-ad. All the research you performed is super valuable and will prevent us from this tricky issue in the future.

[jorgepiloto]

This is impressive. Just approved it. Thanks a lot, @moe-ad.