Tests should clean up after running

[donaldcampbelljr]

Currently, it appears that tests for pypiper are generating results in a tmp directory that is not being cleaned up once pytesting is finished.

[nsheff]

sounds like a good candidate for a context manager :)

[donaldcampbelljr]

Related, the unit test test_pipeline_manager.py is leaving 'failed' flag files after closing the test: example: default_pipeline_name_sample_pipeline_failed.flag

However, they should be cleaned up via the function:

    OUTFOLDER = "tests/Data/pipeline_output"

    @classmethod
    def _clean(cls):
        for d in glob.glob(cls.OUTFOLDER + "*"):
            if os.path.isdir(d):
                print("Removing " + d)
                shutil.rmtree(d)

Also added status clearing at the very end of the unit test.

Next steps: -Investigate using context manager

[vreuter]

sounds like a good candidate for a context manager :)

Yes this and/or the tmp_path fixture from pytest