removing commented-out lines, e.g. models in pytest.mark.parametrize calls, or at least explaining why they're commented out
avoiding aliasing the tmp_path variable as output_folder in tests.
I imagine that we'd have a PR that just addresses these kinds of changes without changing any other functionality, to ease verification that the tests are still correct.
As raised by @d33bs in https://github.com/manubot/manubot-ai-editor/pull/62#discussion_r1831812271 and https://github.com/manubot/manubot-ai-editor/pull/62#discussion_r1831811363, the tests have a few areas where they could be simplified or improved.
Some suggestions were:
pytest.mark.parametrizecalls, or at least explaining why they're commented outtmp_pathvariable asoutput_folderin tests.I imagine that we'd have a PR that just addresses these kinds of changes without changing any other functionality, to ease verification that the tests are still correct.