Skip to content

Conversation

mitchnegus
Copy link
Member

@mitchnegus mitchnegus commented Mar 27, 2025

This converts tests in test_cli.py to be pytest style.

The size of the resulting test file is substantially reduced, since the new format relies heavily on parametrization across similar tests. It also uses pytest temporary directories provided by the tmp_path fixture to avoid needing to create, write, and reload configurations between tests.

@mitchnegus mitchnegus changed the title Convert FIREWHEEL CLI tests to pytest style test: Convert FIREWHEEL CLI tests to pytest style Mar 27, 2025
@mitchnegus mitchnegus marked this pull request as draft March 27, 2025 18:52
@mitchnegus mitchnegus force-pushed the cli-tests branch 3 times, most recently from d0d8d68 to 972fdfc Compare April 2, 2025 19:50
@mitchnegus mitchnegus force-pushed the cli-tests branch 3 times, most recently from 36419f0 to 54fcdf3 Compare April 9, 2025 19:24
@mitchnegus mitchnegus force-pushed the cli-tests branch 2 times, most recently from efb27e9 to ae53753 Compare April 25, 2025 00:57
@mitchnegus mitchnegus force-pushed the cli-tests branch 2 times, most recently from c787112 to 6376250 Compare May 2, 2025 16:29
@mitchnegus mitchnegus force-pushed the cli-tests branch 4 times, most recently from ec324f7 to e4d3686 Compare May 22, 2025 20:19
@mitchnegus mitchnegus force-pushed the cli-tests branch 2 times, most recently from 8a1cdad to 1c8d330 Compare July 7, 2025 14:45
@mitchnegus mitchnegus force-pushed the cli-tests branch 4 times, most recently from 36778ae to 887f694 Compare September 12, 2025 16:55
@mitchnegus mitchnegus force-pushed the cli-tests branch 4 times, most recently from b61cfe4 to 29dbe49 Compare September 23, 2025 19:19
@mitchnegus mitchnegus force-pushed the cli-tests branch 2 times, most recently from db02790 to 029ca32 Compare September 24, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant