test_docs.py in wheel fails

[mtelka]

The test_docs.py included in wheel fails with the following assertion:

AssertionError: could not find my setup.py

Maybe the test should pass (or skip) instead of failure if there is no setup.py available.

[mtelka]

This is the quick and dirty workaround:

--- persistent-5.0/src/persistent/tests/test_docs.py.orig
+++ persistent-5.0/src/persistent/tests/test_docs.py
@@ -36,7 +36,7 @@
         prev, here = here, os.path.dirname(here)
         if here == prev:
             # Let's avoid infinite loops at root
-            raise AssertionError('could not find my setup.py')
+            return unittest.TestSuite()
 
     docs = os.path.join(here, 'docs', 'api')
 

[d-maurer]

Marcel Telka wrote at 2023-8-10 06:38 -0700:

This is the quick and dirty workaround: ``` --- persistent-5.0/src/persistent/tests/test_docs.py.orig +++ persistent-5.0/src/persistent/tests/test_docs.py @@ -36,7 +36,7 @@ prev, here = here, os.path.dirname(here) if here == prev: # Let's avoid infinite loops at root - raise AssertionError('could not find my setup.py') + return unittest.TestSuite()

I do not think this is the right solution. Are you sure you are running the tests correctly? When I run the tests on `master` with `zope-testrunner --test-path ../persistent/src/ -v`, I get ``` Ran 771 tests with 0 failures, 0 errors and 6 skipped in 0.399 seconds. ```. (I am in a sibling folder of `persistent`).

[mtelka]

Yes, when the test is run in the git repo (or in sdist) then it pass. I'm talking about wheel. There is no setup.py in the wheel so test_docs.py cannot pass there. Maybe the better solution would be just to exclude test_docs.py from wheel. Or, maybe even better, simply exclude all tests from wheel?

[d-maurer]

Marcel Telka wrote at 2023-8-10 08:22 -0700:

Yes, when the test is run in the git repo (or in sdist) then it pass. I'm talking about wheel. There is no `setup.py` in the wheel so `test_docs.py` cannot pass there. Maybe the better solution would be just to exclude `test_docs.py` from wheel. Or, maybe even better, simply exclude all tests from wheel?

You are right! I think a good solution would be to skrip the test in this case (with the explaining message `setup.py unavailable`). Would you like to sign a contributor agreement and provide a pull request?

[mtelka]

Would you like to sign a contributor agreement ...

Sorry, no.