Codecov Report

Patch coverage: 99.04% and project coverage change: +2.68 🎉

Comparison is base (a1f5a9d) 93.21% compared to head (0de3bb7) 95.90%.

❗ Current head 0de3bb7 differs from pull request most recent head dea0bff. Consider uploading reports for the commit dea0bff to get more accurate results

@@            Coverage Diff             @@
##             main     #610      +/-   ##
==========================================
+ Coverage   93.21%   95.90%   +2.68%     
==========================================
  Files          32       59      +27     
  Lines        4851     9302    +4451     
==========================================
+ Hits         4522     8921    +4399     
- Misses        329      381      +52     

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@damien2012eng I think it's a bit weird that some in some of the test files, you are setting class attributes in setUpClass() using cls.x and in others you are setting them directly in the class definition (e.g. test_cli.py). I think it makes sense to be consistent – if any class attributes are needed, just define them in the class definition. setUpClass() should only be used if you need to create some files or do something else to set up the class other than defining attributes.