The tests in the file should be converted to use pytest fixtures. That would clean it up nicely.
The first four tests can use one fixture, and a second fixture can be used for the last three.
Also, the final configuration and running of the Simulation should be done within those last three tests, instead of separately in the bos_run function.
arichar6
commented
4 years ago
https://github.com/NRL-Plasma-Physics-Division/block-on-spring/blob/bb77b7fbe1904df6098222a3cf5ef12b6929a500/test_spring.py#L6
The tests in the file should be converted to use pytest fixtures. That would clean it up nicely.
The first four tests can use one fixture, and a second fixture can be used for the last three.
Also, the final configuration and running of the Simulation should be done within those last three tests, instead of separately in the
bos_runfunction.