shutil.rmtree is dangerous, and test.tmpdir will clean up automatically, so it's not needed.