Merge branch 'master' into feature/traversable

Author: jaraco

.gitlab-ci.yml

@@ -11,7 +11,7 @@ qa:
 
 tests:
   script:
-    - tox -e py27-nocov,py34-nocov,py35-nocov,py36-nocov
+    - tox -e py27,py34,py35,py36,py37,py38
 
 coverage:
   script:

LICENSE

@@ -1,4 +1,4 @@
-Copyright 2017-2018 Brett Cannon, Barry Warsaw
+Copyright 2017-2019 Brett Cannon, Barry Warsaw
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.

importlib_resources/_py2.py

@@ -37,8 +37,7 @@ def _normalize_path(path):
     parent, file_name = os.path.split(str_path)
     if parent:
         raise ValueError("{!r} must be only a file name".format(path))
-    else:
-        return file_name
+    return file_name
 
 
 def open_binary(package, resource):
@@ -68,8 +67,7 @@ def open_binary(package, resource):
             message = '{!r} resource not found in {!r}'.format(
                 resource, package_name)
             raise FileNotFoundError(message)
-        else:
-            return BytesIO(data)
+        return BytesIO(data)
 
 
 def open_text(package, resource, encoding='utf-8', errors='strict'):
@@ -113,27 +111,28 @@ def path(package, resource):
     package_directory = Path(package.__file__).parent
     file_path = package_directory / resource
     # If the file actually exists on the file system, just return it.
+    if file_path.exists():
+        yield file_path
+        return
+
     # Otherwise, it's probably in a zip file, so we need to create a temporary
     # file and copy the contents into that file, hence the contextmanager to
     # clean up the temp file resource.
-    if file_path.exists():
-        yield file_path
-    else:
-        with open_binary(package, resource) as fp:
-            data = fp.read()
-        # Not using tempfile.NamedTemporaryFile as it leads to deeper 'try'
-        # blocks due to the need to close the temporary file to work on Windows
-        # properly.
-        fd, raw_path = tempfile.mkstemp()
+    with open_binary(package, resource) as fp:
+        data = fp.read()
+    # Not using tempfile.NamedTemporaryFile as it leads to deeper 'try'
+    # blocks due to the need to close the temporary file to work on Windows
+    # properly.
+    fd, raw_path = tempfile.mkstemp()
+    try:
+        os.write(fd, data)
+        os.close(fd)
+        yield Path(raw_path)
+    finally:
         try:
-            os.write(fd, data)
-            os.close(fd)
-            yield Path(raw_path)
-        finally:
-            try:
-                os.remove(raw_path)
-            except FileNotFoundError:
-                pass
+            os.remove(raw_path)
+        except FileNotFoundError:
+            pass
 
 
 def is_resource(package, name):
@@ -220,7 +219,7 @@ def contents(package):
         relpath = package_directory.relative_to(archive_path)
         with ZipFile(archive_path) as zf:
             toc = zf.namelist()
-        subdirs_seen = set()                        # type: Set
+        subdirs_seen = set()
         subdirs_returned = []
         for filename in toc:
             path = Path(filename)

importlib_resources/_py3.py

@@ -50,8 +50,7 @@ def _normalize_path(path) -> str:
     parent, file_name = os.path.split(str_path)
     if parent:
         raise ValueError('{!r} must be only a file name'.format(path))
-    else:
-        return file_name
+    return file_name
 
 
 def _get_resource_reader(
@@ -96,8 +95,7 @@ def open_binary(package: Package, resource: Resource) -> BinaryIO:
             message = '{!r} resource not found in {!r}'.format(
                 resource, package_name)
             raise FileNotFoundError(message)
-        else:
-            return BytesIO(data)
+        return BytesIO(data)
 
 
 def open_text(package: Package,
@@ -146,33 +144,34 @@ def path(package: Package, resource: Resource) -> Iterator[Path]:
     package = _get_package(package)
     reader = _get_resource_reader(package)
     if reader is not None:
-        try:
+        with suppress(FileNotFoundError):
             yield Path(reader.resource_path(resource))
             return
-        except FileNotFoundError:
-            pass
     # Fall-through for both the lack of resource_path() *and* if
     # resource_path() raises FileNotFoundError.
     package_directory = Path(package.__spec__.origin).parent
     file_path = package_directory / resource
+    # If the file actually exists on the file system, just return it.
     if file_path.exists():
         yield file_path
-    else:
-        with open_binary(package, resource) as fp:
-            data = fp.read()
-        # Not using tempfile.NamedTemporaryFile as it leads to deeper 'try'
-        # blocks due to the need to close the temporary file to work on
-        # Windows properly.
-        fd, raw_path = tempfile.mkstemp()
-        try:
-            os.write(fd, data)
-            os.close(fd)
-            yield Path(raw_path)
-        finally:
-            try:
-                os.remove(raw_path)
-            except FileNotFoundError:
-                pass
+        return
+
+    # Otherwise, it's probably in a zip file, so we need to create a temporary
+    # file and copy the contents into that file, hence the contextmanager to
+    # clean up the temp file resource.
+    with open_binary(package, resource) as fp:
+        data = fp.read()
+    # Not using tempfile.NamedTemporaryFile as it leads to deeper 'try'
+    # blocks due to the need to close the temporary file to work on
+    # Windows properly.
+    fd, raw_path = tempfile.mkstemp()
+    try:
+        os.write(fd, data)
+        os.close(fd)
+        yield Path(raw_path)
+    finally:
+        with suppress(FileNotFoundError):
+            os.remove(raw_path)
 
 
 def is_resource(package: Package, name: str) -> bool:

importlib_resources/docs/conf.py

@@ -53,7 +53,7 @@
 
 # General information about the project.
 project = 'importlib_resources'
-copyright = '2017-2018, Brett Cannon, Barry Warsaw'
+copyright = '2017-2019, Brett Cannon, Barry Warsaw'
 author = 'Brett Cannon, Barry Warsaw'
 
 # The version info for the project you're documenting, acts as replacement for

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = importlib_resources
-version = attr: version.__version__
+version = file: importlib_resources/version.txt
 author = Barry Warsaw
 author_email = [email protected]
 url = http://importlib-resources.readthedocs.io/

tox.ini

@@ -1,13 +1,13 @@
 [tox]
 # Don't cover Python 3.7 since this is just a shim for that version.  Do at
 # least make sure we don't regress!
-envlist = {py27,py34,py35,py36}-{nocov,cov,diffcov},{py37,py38}-nocov,qa,docs
+envlist = {py27,py34,py35,py36}{,-cov,-diffcov},{py37,py38},qa,docs
 skip_missing_interpreters = True
 
 
 [testenv]
 commands =
-    nocov: python -m unittest discover
+    !cov,!diffcov: python -m unittest discover
     cov,diffcov: python -m coverage run {[coverage]rc} -m unittest discover {posargs}
     cov,diffcov: python -m coverage combine {[coverage]rc}
     cov: python -m coverage html {[coverage]rc}