— Twitter: @ronaldoussoren Blog: https://blog.ronaldoussoren.net/
> On 13 Feb 2019, at 16:10, Giampaolo Rodola' <g.rod...@gmail.com> wrote: > > > > On Wed, Feb 13, 2019 at 2:27 PM Ronald Oussoren <ronaldousso...@mac.com > <mailto:ronaldousso...@mac.com>> wrote: > > >> On 13 Feb 2019, at 13:24, Giampaolo Rodola' <g.rod...@gmail.com >> <mailto:g.rod...@gmail.com>> wrote: >> >> >> Hello, >> after discovering os.makedirs() has no unit-tests >> (https://bugs.python.org/issue35982 <https://bugs.python.org/issue35982>) I >> was thinking about working on a PR to increase the test coverage of >> fs-related os.* functions. In order to do so I think it would be useful to >> add a convenience function to "just delete something if it exists", >> regardless if it's a file, directory, directory tree, etc., and include it >> into test.support module. > > Something like shutil.rmtree() with ignore_errors=True? > > shutil.rmtree() is about directories and can't be used against files. > support.rmpath() would take a path (meaning anything) and try to remove it. You’re right. I usually use shutil.rmtree for tests that need to create temporary files, and create a temporary directory for those files (that is, use tempfile.mkdtemp in setUp() and use shutil.rmtree in tearDown()). That way I don’t have to adjust house-keeping code when I make changes to test code. Ronald
_______________________________________________ Python-Dev mailing list Python-Dev@python.org https://mail.python.org/mailman/listinfo/python-dev Unsubscribe: https://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com