Remove tests directory from wheel package

[dcolmenares-r7]

Environment details

• OS type and version: MacOS v11.6.1
• Python version: python --version - Python 3.8.12
• pip version: pip --version - pip 21.3.1
• google-cloud-org-policy version: pip show google-cloud-org-policy

Name: google-cloud-org-policy
Version: 1.2.1
Summary: UNKNOWN
Home-page: https://github.com/googleapis/python-org-policy
Author: Google LLC
Author-email: googleapis-packages@google.com
License: Apache 2.0
Requires: google-api-core, proto-plus
Required-by: divvycloud, google-cloud-asset

Steps to reproduce

Building this library using the standard build steps:

python -m pip install --upgrade setuptools wheel
python setup.py sdist bdist_wheel

This builds the wheel package in the dist/ folder. Unzipping that we see multiple unit tests.

unzip -l google_cloud_org_policy-1.2.1-py2.py3-none-any.whl | grep 'tests'

Code example

N/A

Stack trace

% unzip -l google_cloud_org_policy-1.2.1-py2.py3-none-any.whl | grep 'tests'
      600  02-17-2022 20:21   tests/__init__.py
      600  02-17-2022 20:21   tests/unit/__init__.py
      600  02-17-2022 20:21   tests/unit/gapic/__init__.py
      600  02-17-2022 20:21   tests/unit/gapic/orgpolicy_v2/__init__.py
   112030  02-17-2022 20:21   tests/unit/gapic/orgpolicy_v2/test_org_policy.py

I would be happy to submit a PR with the proposed change. Adding

packages=setuptools.PEP420PackageFinder.find(exclude=['tests*'])

to setup.py - https://github.com/googleapis/python-org-policy/blob/main/setup.py#L39, looks like it fixes it but whatever works for you all. The reason for requesting this change is anyone using a tests folder as part of their package structure will be affected by this, as site-packages takes precedence in many setups.

Thanks

[busunkim96]

Hi @dcolmenares-r7,

That sounds reasonable, I've opened #121.