Apply ruff rule RUF059

RUF059 Unpacked variable is never used

Author: DimitriPapadopoulos

fsspec/core.py

@@ -292,7 +292,7 @@ def open_files(
     - For implementations in separate packages see
       https://filesystem-spec.readthedocs.io/en/latest/api.html#other-known-implementations
     """
-    fs, fs_token, paths = get_fs_token_paths(
+    fs, _fs_token, paths = get_fs_token_paths(
         urlpath,
         mode,
         num=num,

fsspec/implementations/reference.py

@@ -391,7 +391,7 @@ def __getitem__(self, key):
 
     def __setitem__(self, key, value):
         if "/" in key and not self._is_meta(key):
-            field, chunk = key.rsplit("/", 1)
+            field, _chunk = key.rsplit("/", 1)
             record, i, _ = self._key_to_record(key)
             subdict = self._items.setdefault((field, record), {})
             subdict[i] = value

fsspec/implementations/tar.py

@@ -118,7 +118,7 @@ def _get_dirs(self):
     def _open(self, path, mode="rb", **kwargs):
         if mode != "rb":
             raise ValueError("Read-only filesystem implementation")
-        details, offset = self.index[path]
+        details, _offset = self.index[path]
         if details["type"] != "file":
             raise ValueError("Can only handle regular files")
         return self.tar.extractfile(path)

fsspec/implementations/tests/test_cached.py

@@ -699,7 +699,7 @@ def test_local_filecache_basic(local_filecache):
 
 
 def test_local_filecache_does_not_change_when_original_data_changed(local_filecache):
-    old_data, original_file, cache_location, fs = local_filecache
+    old_data, original_file, _cache_location, fs = local_filecache
     new_data = b"abc"
 
     with fs.open(original_file, "rb") as f:
@@ -899,7 +899,7 @@ def test_filecache_serialization(impl, fs):
 
 
 def test_add_file_to_cache_after_save(local_filecache):
-    (data, original_file, cache_location, fs) = local_filecache
+    (_data, original_file, cache_location, fs) = local_filecache
 
     fs.save_cache()
 

fsspec/implementations/tests/test_git.py

@@ -67,7 +67,7 @@ def _check_FileNotFoundError(f, *args, **kwargs):
 
 
 def test_file_existence_checks(repo):
-    d, sha = repo
+    d, _sha = repo
 
     fs, _ = fsspec.url_to_fs(f"git://{d}:abranch@")
 
@@ -95,7 +95,7 @@ def test_file_existence_checks(repo):
 
 
 def test_url(repo):
-    d, sha = repo
+    d, _sha = repo
     fs, _, paths = fsspec.core.get_fs_token_paths(f"git://file1::file://{d}")
     assert make_path_posix(d) in make_path_posix(fs.repo.path)
     assert paths == ["file1"]

fsspec/tests/conftest.py

@@ -93,7 +93,7 @@ def do_GET(self):
         content_range = f"bytes 0-{len(file_data) - 1}/{len(file_data)}"
         if ("Range" in self.headers) and ("ignore_range" not in self.headers):
             ran = self.headers["Range"]
-            b, ran = ran.split("=")
+            _b, ran = ran.split("=")
             start, end = ran.split("-")
             if start:
                 content_range = f"bytes {start}-{end}/{len(file_data)}"

fsspec/tests/test_api.py

@@ -42,7 +42,7 @@ def test_class_methods():
 
 def test_multi(m):
     m.pipe("/afile", b"data")
-    fs, token, paths = fsspec.core.get_fs_token_paths(["/afile", "/afile"])
+    _fs, _token, paths = fsspec.core.get_fs_token_paths(["/afile", "/afile"])
     assert len(paths) == 2
 
 

fsspec/tests/test_core.py

@@ -399,11 +399,11 @@ def test_url_to_fs(ftp_writable):
     data = b"hello"
     with fsspec.open(f"ftp://{username}:{password}@{host}:{port}/afile", "wb") as f:
         f.write(data)
-    fs, url = fsspec.core.url_to_fs(
+    _fs, url = fsspec.core.url_to_fs(
         f"simplecache::ftp://{username}:{password}@{host}:{port}/afile"
     )
     assert url == "/afile"
-    fs, url = fsspec.core.url_to_fs(f"ftp://{username}:{password}@{host}:{port}/afile")
+    _fs, url = fsspec.core.url_to_fs(f"ftp://{username}:{password}@{host}:{port}/afile")
     assert url == "/afile"
 
     with fsspec.open(f"ftp://{username}:{password}@{host}:{port}/afile.zip", "wb") as f:
@@ -414,11 +414,11 @@ def test_url_to_fs(ftp_writable):
                 f2.write(b"hello")
         f.write(data)
 
-    fs, url = fsspec.core.url_to_fs(
+    _fs, url = fsspec.core.url_to_fs(
         f"zip://inner::ftp://{username}:{password}@{host}:{port}/afile.zip"
     )
     assert url == "inner"
-    fs, url = fsspec.core.url_to_fs(
+    _fs, url = fsspec.core.url_to_fs(
         f"simplecache::zip::ftp://{username}:{password}@{host}:{port}/afile.zip"
     )
     assert url == ""
@@ -472,7 +472,7 @@ def test_repeated_argument():
     pytest.importorskip("adlfs")
     from fsspec.core import url_to_fs
 
-    fs, url = url_to_fs(
+    fs, _url = url_to_fs(
         "az://[email protected]/DATA",
         anon=False,
         account_name="ACCOUNT",

fsspec/tests/test_fuse.py

@@ -132,7 +132,7 @@ def test_chmod(mount_local):
 
 
 def test_seek_rw(mount_local):
-    source_dir, mount_dir = mount_local
+    _source_dir, mount_dir = mount_local
     fh = open(mount_dir / "text", "w")
     fh.write("teST")
     fh.seek(2)