fix: exclude subpackage from setup.py if __init__.py is excluded (#…

…1009) (#1626)

* fix: exclude subpackage from `setup.py` if `__init__.py` is excluded

Fixes: #1009

* fix: added missing test data

* fix: lint test data

* change (sdist.git): exclude folders with no python file

* fix (sdist.git): make black happy

poetry/masonry/builders/sdist.py

@@ -237,7 +237,15 @@ def find_nearest_pkg(rel_path):
             if from_top_level == ".":
                 continue
 
-            is_subpkg = any([filename.endswith(".py") for filename in filenames])
+            is_subpkg = any(
+                [filename.endswith(".py") for filename in filenames]
+            ) and not all(
+                [
+                    self.is_excluded(Path(path, filename).relative_to(self._path))
+                    for filename in filenames
+                    if filename.endswith(".py")
+                ]
+            )
             if is_subpkg:
                 subpkg_paths.add(from_top_level)
                 parts = from_top_level.split(os.sep)

tests/masonry/builders/fixtures/excluded_subpackage/README.rst

@@ -0,0 +1,2 @@
+My Package
+==========

tests/masonry/builders/fixtures/excluded_subpackage/example/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.1.0"

tests/masonry/builders/fixtures/excluded_subpackage/example/test/excluded.py

@@ -0,0 +1,5 @@
+from .. import __version__
+
+
+def test_version():
+    assert __version__ == "0.1.0"

tests/masonry/builders/fixtures/excluded_subpackage/pyproject.toml

@@ -0,0 +1,18 @@
+[tool.poetry]
+name = "example"
+version = "0.1.0"
+description = ""
+authors = ["Sébastien Eustace <sebastien@eustace.io>"]
+exclude = [
+    "**/test/**/*",
+]
+
+[tool.poetry.dependencies]
+python = "^3.6"
+
+[tool.poetry.dev-dependencies]
+pytest = "^3.0"
+
+[build-system]
+requires = ["poetry>=0.12"]
+build-backend = "poetry.masonry.api"

tests/masonry/builders/test_sdist.py

@@ -473,3 +473,18 @@ def test_proper_python_requires_if_three_digits_precision_version_specified():
     parsed = p.parsestr(to_str(pkg_info))
 
     assert parsed["Requires-Python"] == "==2.7.15"
+
+
+def test_excluded_subpackage():
+    poetry = Factory().create_poetry(project("excluded_subpackage"))
+
+    builder = SdistBuilder(poetry, NullEnv(), NullIO())
+    setup = builder.build_setup()
+
+    setup_ast = ast.parse(setup)
+
+    setup_ast.body = [n for n in setup_ast.body if isinstance(n, ast.Assign)]
+    ns = {}
+    exec(compile(setup_ast, filename="setup.py", mode="exec"), ns)
+
+    assert ns["packages"] == ["example"]