You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the CLI it may be an easier to understand approach to move the commands to their own files.
Justification:
render-engine init is in its own file
render-engine tools are in their own file
The act of calling render-engine cli are in its own file
New entry is spread across two files test_cli.py and test_cli_commands.py
This may be bigger task that is move each of the commands to their own test file (which could require moving and calling things in different locations)
Solution:
Create tests_render-engine-new_entry.py
Moved all tests involving new entry in to those locations
IMO it makes sense to have all of the tests for the functionality in one place. I've been bitten by looking in one place but not the other for stuff like this. I'd much rather a smaller file for each command then one or two big ones - it's easier to find things.
Activity
brass75 commentedon Jun 20, 2025
IMO it makes sense to have all of the tests for the functionality in one place. I've been bitten by looking in one place but not the other for stuff like this. I'd much rather a smaller file for each command then one or two big ones - it's easier to find things.