search

zopefoundation / Products.PluggableAuthService

Pluggable Zope authentication / authorization framework
Other
9 stars 18 forks source link

[WIP] Python 2 / 3 compatibility. #14

Closed icemac closed 6 years ago

icemac commented 6 years ago

15 should be merged beforehand to get rid of the ZServer dependency to allow running the test on Python 3.

Closes #18.

icemac commented 6 years ago

To get the tests of this PR green, #15 should be fixed and merged here. This allows to reduce duplicate work on both branches.

icemac commented 6 years ago

I rebased this PR onto #15 after merging it to master. Currently some Python 2.7 tests are failing which are fine on master. I'll look into them over the next days.

icemac commented 6 years ago

Now testing against Python 3.6, too. These tests are currently allowed to fail. We are currently at: Total: 485 tests, 12 failures, 60 errors, 8 skipped in 0.847 seconds.

davisagli commented 6 years ago

@icemac I worked on this and have the tests passing on Python 2 and 3. This currently requires a couple fixes in other packages which are checked out in buildout.cfg:

icemac commented 6 years ago

@davisagli Cool! Thank you for working on this. I think we can get the required changes merged at the Zope-4-Welcome-Sprint this week.