Change to check=True for subprocess

Skylion007 · Jan 16, 2024 · cd48c84 · cd48c84
1 parent 84b2bdd
commit cd48c84
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/fsspec/tests/test_fuse.py b/fsspec/tests/test_fuse.py
@@ -123,7 +123,7 @@ def test_chmod(mount_local):
         ["cp", str(mount_dir / "text"), str(mount_dir / "new")],
         stdout=subprocess.PIPE,
         stderr=subprocess.PIPE,
-        check=False,
+        check=True,
     )
 
     assert cp.stderr == b""

diff --git a/fsspec/tests/test_spec.py b/fsspec/tests/test_spec.py
@@ -1147,7 +1147,7 @@ def test_posix_tests_bash_stat(path, expected, glob_files_folder):
             f"cd {glob_files_folder} && shopt -s globstar && stat -c %N {bash_path}",
         ],
         capture_output=True,
-        check=False,
+        check=True,
     )
     # Remove the last element always empty
     bash_output = bash_output.stdout.decode("utf-8").replace("'", "").split("\n")[:-1]