armi icon indicating copy to clipboard operation
armi copied to clipboard

Making Case.run more testable

Open john-science opened this issue 3 years ago • 0 comments

Description

While looking at a possible duplicate coveragerc file, I noticed that the important Case.run() method had no code coverage because it was so tremendously long it was untestable.

So I split the really long method out into something with helper methods, so it could be at least partially testable.


Checklist

  • [X] This PR has only one purpose or idea.
  • [X] Tests have been added/updated to verify that the new/changed code works.
  • [X] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
  • [X] The documentation is still up-to-date in the doc folder.
  • [X] The dependencies are still up-to-date in setup.py.

john-science avatar Sep 30 '22 03:09 john-science