diff --git a/fs/tests/test_mkdir.py b/fs/tests/test_mkdir.py index 9101114..0199f73 100644 --- a/fs/tests/test_mkdir.py +++ b/fs/tests/test_mkdir.py @@ -1,4 +1,5 @@ import os.path +import pytest import fs import unittest @@ -15,6 +16,7 @@ def test_mkdir(): fs.mkdir(path) assert os.path.exists(path) is True + os.rmdir(path) def test_mkdir_recursive(): @@ -27,6 +29,7 @@ def test_mkdir_recursive(): fs.mkdir(path) assert os.path.exists(path) is True + os.removedirs(path) def test_mkdir_recursive_fail(): @@ -36,7 +39,5 @@ def test_mkdir_recursive_fail(): if (os.path.exists(path)): raise ValueError("Directory %s already exists!" % path) - try: - fs.mkdir(path, recursive=False) - except FileNotFoundError: - pass \ No newline at end of file + with pytest.raises(FileNotFoundError): + fs.mkdir(path, recursive=False) \ No newline at end of file diff --git a/fs/tests/test_rename.py b/fs/tests/test_rename.py index e05ea0a..4f0ccd4 100644 --- a/fs/tests/test_rename.py +++ b/fs/tests/test_rename.py @@ -14,6 +14,7 @@ def test_rename_file(): fs.rename(TEST_FILE, rename_file) assert os.path.exists(rename_file) is True + os.remove(rename_file) def test_rename_directory(): @@ -25,4 +26,5 @@ def test_rename_directory(): fs.rename(TEST_DIR_2, rename_dir) - assert os.path.exists(rename_dir) is True \ No newline at end of file + assert os.path.exists(rename_dir) is True + os.rmdir(rename_dir) \ No newline at end of file diff --git a/fs/tests/test_touch.py b/fs/tests/test_touch.py index d0b0f16..5d1a35e 100644 --- a/fs/tests/test_touch.py +++ b/fs/tests/test_touch.py @@ -26,6 +26,9 @@ def test_touch_on_new_file(): assert os.path.exists(new_file) is True + os.remove(new_file) + + @pytest.mark.skipif(os.name == "nt", reason="does not work on windows") def test_touch_on_directory():