Skip to content

Commit 1a27fc2

Browse files
authored
Add recursive rm test (fsspec#613)
* Add recursive rm test * update black in pre-commit * newlint
1 parent 49443fc commit 1a27fc2

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed

.pre-commit-config.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ repos:
66
- id: end-of-file-fixer
77
- id: trailing-whitespace
88
- repo: https://github.com/psf/black
9-
rev: 22.1.0
9+
rev: 22.3.0
1010
hooks:
1111
- id: black
1212
exclude: ^docs/

s3fs/tests/test_s3fs.py

+19
Original file line numberDiff line numberDiff line change
@@ -2274,6 +2274,25 @@ def test_lsdir(s3):
22742274
assert d in s3.ls(test_bucket_name)
22752275

22762276

2277+
def test_rm_recursive_folder(s3):
2278+
s3.touch(test_bucket_name + "/sub/file")
2279+
s3.rm(test_bucket_name + "/sub", recursive=True)
2280+
assert not s3.exists(test_bucket_name + "/sub/file")
2281+
assert not s3.exists(test_bucket_name + "/sub")
2282+
2283+
s3.touch(test_bucket_name + "/sub/file")
2284+
s3.touch(test_bucket_name + "/sub/") # placeholder
2285+
s3.rm(test_bucket_name + "/sub", recursive=True)
2286+
assert not s3.exists(test_bucket_name + "/sub/file")
2287+
assert not s3.exists(test_bucket_name + "/sub")
2288+
2289+
s3.touch(test_bucket_name + "/sub/file")
2290+
s3.rm(test_bucket_name, recursive=True)
2291+
assert not s3.exists(test_bucket_name + "/sub/file")
2292+
assert not s3.exists(test_bucket_name + "/sub")
2293+
assert not s3.exists(test_bucket_name)
2294+
2295+
22772296
def test_copy_file_without_etag(s3, monkeypatch):
22782297

22792298
s3.touch(test_bucket_name + "/copy_tests/file")

0 commit comments

Comments
 (0)