These tests should NOT fail

[ilaflott]

Describe your changes

split up cli tests, add a new one based on the catalogbuilder interface, so we'll know if it changes again via a failed test.

Issue ticket number and link (if applicable)

149

Checklist before requesting a review

• [x] I ran my code
• [x] I tried to make my code readable
• [x] I tried to comment my code
• [x] I wrote a new test, if applicable N/A - [x] I wrote new instructions/documentation, if applicable
• [x] I ran pytest and inspected it's output
• [x] I ran pylint and attempted to implement some of it's feedback

[ilaflott]

pylint score in current main branch is 3.34. These edits bring the code to about a 4.70. The aim is to address enough of the complaints that the score will easily fall if low quality code is committed in the future.

quick summary of changes:

1. generally, lots of pylint cleanup.
2. camel-case --> snake case changes,
3. whitespace removal,
4. long line breakups,
5. add # pylint: disable=unused-argument, to ignore unused arguments in certain places for now
6. asking pylint to ignore a fake-import-problem with netCDF4.Dataset
7. add doc strings where easy to
8. import order fixes
9. add basic cli tests for all fre <tool> calls,
10. add two extra cli tests for fre catalog builder calls to make sure we have something that might catch an API change on that front.

[ilaflott]

this PR now contains autogenerated documentation from python doc-strings! https://github.com/NOAA-GFDL/fre-cli/actions/runs/10372226899/job/28714459606?pr=151

[ilaflott]

ready to go