feat: Introduce compatibility with native namespace packages (#59)

* feat: Introduce compatibility with native namespace packages

* add unit test presubmits

* update unit test versions

* update required checks

---------

Co-authored-by: ohmayr <omairnaveed@ymail.com>

Author: parthea

.github/sync-repo-settings.yaml

@@ -9,6 +9,15 @@ branchProtectionRules:
   requiredStatusCheckContexts:
     - 'cla/google'
     - 'OwlBot Post Processor'
+    - 'docs'
+    - 'docfx'
+    - 'lint'
+    - 'unit (3.7)'
+    - 'unit (3.8)'
+    - 'unit (3.9)'
+    - 'unit (3.10)'
+    - 'unit (3.11)'
+    - 'cover'
 permissionRules:
   - team: actools-python
     permission: admin

.github/workflows/unittest.yml

@@ -0,0 +1,57 @@
+on:
+  pull_request:
+    branches:
+      - main
+name: unittest
+jobs:
+  unit:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python: ['3.7', '3.8', '3.9', '3.10', '3.11']
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v3
+    - name: Setup Python
+      uses: actions/setup-python@v4
+      with:
+        python-version: ${{ matrix.python }}
+    - name: Install nox
+      run: |
+        python -m pip install --upgrade setuptools pip wheel
+        python -m pip install nox
+    - name: Run unit tests
+      env:
+        COVERAGE_FILE: .coverage-${{ matrix.python }}
+      run: |
+        nox -s unit-${{ matrix.python }}
+    - name: Upload coverage results
+      uses: actions/upload-artifact@v3
+      with:
+        name: coverage-artifacts
+        path: .coverage-${{ matrix.python }}
+
+  cover:
+    runs-on: ubuntu-latest
+    needs:
+        - unit
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v3
+    - name: Setup Python
+      uses: actions/setup-python@v4
+      with:
+        python-version: "3.8"
+    - name: Install coverage
+      run: |
+        python -m pip install --upgrade setuptools pip wheel
+        python -m pip install coverage
+    - name: Download coverage results
+      uses: actions/download-artifact@v3
+      with:
+        name: coverage-artifacts
+        path: .coverage-results/
+    - name: Report coverage results
+      run: |
+        coverage combine .coverage-results/.coverage*
+        coverage report --show-missing --fail-under=100

google/iam/__init__.py

@@ -55,15 +55,13 @@ def default(session):
     session.install("asyncmock", "pytest-asyncio")
 
     session.install("mock", "pytest", "pytest-cov")
-    session.install("-e", ".")
-
     constraints_path = str(
         CURRENT_DIRECTORY / "testing" / f"constraints-{session.python}.txt"
     )
 
     # Install grpc-google-iam-v1
     # This *must* be the last install command to get the package from source.
-    session.install("e", "..", "-c", constraints_path)
+    session.install("-e", ".", "-c", constraints_path)
 
     # Run py.test against the unit tests.
     session.run(
@@ -80,6 +78,7 @@ def default(session):
     )
 
 
+@nox.session(python=["3.7", "3.8", "3.9", "3.10", "3.11"])
 def unit(session):
     """Run the unit test suite."""
     default(session)
@@ -106,15 +105,13 @@ def system(session):
     # virtualenv's dist-packages.
     session.install("mock", "pytest", "google-cloud-testutils")
 
-    session.install("-e", ".")
-
     constraints_path = str(
         CURRENT_DIRECTORY / "testing" / f"constraints-{session.python}.txt"
     )
 
     # Install grpc-google-iam-v1
     # This *must* be the last install command to get the package from source.
-    session.install("e", "..", "-c", constraints_path)
+    session.install("-e", ".", "-c", constraints_path)
 
     # Run py.test against the system tests.
     if system_test_exists:
@@ -155,7 +152,7 @@ def test(session, library):
 
     session.cd(library)
 
-    unit(session)
+    default(session)
 
     # system tests are run on 3.7 only
     if session.python == "3.7":

noxfile.py

@@ -58,7 +58,7 @@
 s.move(templated_files / "CONTRIBUTING.rst")
 s.move(templated_files / "*.md")
 s.move(templated_files / "renovate.json")
-s.move(templated_files / ".github", excludes=["workflows/unittest.yml"])
+s.move(templated_files / ".github")
 s.move(templated_files / ".trampolinerc")
 
 python.py_samples(skip_readmes=True)

owlbot.py

@@ -39,15 +39,10 @@
 
 packages = [
     package
-    for package in setuptools.PEP420PackageFinder.find()
+    for package in setuptools.find_namespace_packages()
     if package.startswith("google")
 ]
 
-namespaces = ["google"]
-if "google.iam" in packages:
-    namespaces.append("google.iam")
-
-
 setuptools.setup(
     name=name,
     version=version,
@@ -74,7 +69,6 @@
     platforms="Posix; MacOS X; Windows",
     packages=packages,
     python_requires=">=3.7",
-    namespace_packages=namespaces,
     install_requires=dependencies,
     include_package_data=True,
     zip_safe=False,