Make zip write use the remote path it should (#1046)

Author: martindurant

Diff for: fsspec/implementations/zip.py

@@ -3,7 +3,7 @@
 import datetime
 import zipfile
 
-from fsspec import open_files
+import fsspec
 from fsspec.archive import AbstractArchiveFileSystem
 from fsspec.utils import DEFAULT_BLOCK_SIZE
 
@@ -46,15 +46,9 @@ def __init__(
         """
         super().__init__(self, **kwargs)
         if isinstance(fo, str):
-            files = open_files(
+            fo = fsspec.open(
                 fo, mode=mode + "b", protocol=target_protocol, **(target_options or {})
             )
-            if len(files) != 1:
-                raise ValueError(
-                    'Path "{}" did not resolve to exactly'
-                    'one file: "{}"'.format(fo, files)
-                )
-            fo = files[0]
         self.of = fo
         self.fo = fo.__enter__()  # the whole instance is a context
         self.zip = zipfile.ZipFile(self.fo, mode=mode)
@@ -68,7 +62,11 @@ def _strip_protocol(cls, path):
 
     def __del__(self):
         if hasattr(self, "zip"):
-            self.zip.close()
+            self.close()
+
+    def close(self):
+        "Commits any write changes to the file. Done on ``del`` too."
+        self.zip.close()
 
     def _get_dirs(self):
         if self.dir_cache is None:
@@ -78,7 +76,7 @@ def _get_dirs(self):
                 for dirname in self._all_dirnames(self.zip.namelist())
             }
             for z in files:
-                f = {s: getattr(z, s) for s in zipfile.ZipInfo.__slots__}
+                f = {s: getattr(z, s, None) for s in zipfile.ZipInfo.__slots__}
                 f.update(
                     {
                         "name": z.filename,