Merge pull request #60 from jaraco/bugfix/55-isfile-missing

is_file returns False for non-existent files

Author: jaraco

CHANGES.rst

@@ -9,6 +9,8 @@ close the same file handle.
 #56 and bpo-41035: ``Path._next`` now honors
 subclasses.
 
+#55: ``Path.is_file()`` now returns False for non-existent names.
+
 v3.1.0
 ======
 

test_zipp.py

@@ -107,6 +107,11 @@ def test_iterdir_and_types(self):
             (i,) = h.iterdir()
             assert i.is_file()
 
+    def test_is_file_missing(self):
+        for alpharep in self.zipfile_alpharep():
+            root = zipp.Path(alpharep)
+            assert not root.joinpath('missing.txt').is_file()
+
     def test_iterdir_on_file(self):
         for alpharep in self.zipfile_alpharep():
             root = zipp.Path(alpharep)
@@ -175,15 +180,15 @@ def test_read(self):
     def test_joinpath(self):
         for alpharep in self.zipfile_alpharep():
             root = zipp.Path(alpharep)
-            a = root.joinpath("a")
+            a = root.joinpath("a.txt")
             assert a.is_file()
             e = root.joinpath("b").joinpath("d").joinpath("e.txt")
             assert e.read_text() == "content of e"
 
     def test_traverse_truediv(self):
         for alpharep in self.zipfile_alpharep():
             root = zipp.Path(alpharep)
-            a = root / "a"
+            a = root / "a.txt"
             assert a.is_file()
             e = root / "b" / "d" / "e.txt"
             assert e.read_text() == "content of e"

zipp.py

@@ -263,7 +263,7 @@ def is_dir(self):
         return not self.at or self.at.endswith("/")
 
     def is_file(self):
-        return not self.is_dir()
+        return self.exists() and not self.is_dir()
 
     def exists(self):
         return self.at in self.root._name_set()