Merge lp://staging/~barry/mailman.client/bilingual into lp://staging/mailman.client

Proposed by Barry Warsaw
Status: Merged
Merged at revision: 60
Proposed branch: lp://staging/~barry/mailman.client/bilingual
Merge into: lp://staging/mailman.client
Diff against target: 6418 lines (+4561/-1012)
23 files modified
.bzrignore (+3/-0)
MANIFEST.in (+2/-2)
README.rst (+1/-2)
TESTING.rst (+90/-0)
bin/test (+0/-35)
distribute_setup.py (+0/-477)
setup.py (+11/-13)
setup_helpers.py (+20/-19)
src/__init__.py (+1/-1)
src/mailmanclient/NEWS.rst (+6/-2)
src/mailmanclient/README.rst (+3/-4)
src/mailmanclient/__init__.py (+2/-2)
src/mailmanclient/_client.py (+100/-71)
src/mailmanclient/docs/using.rst (+130/-111)
src/mailmanclient/testing/documentation.py (+84/-0)
src/mailmanclient/testing/nose.py (+129/-0)
src/mailmanclient/tests/data/tape.yaml (+3905/-0)
src/mailmanclient/tests/test_docs.py (+0/-127)
src/mailmanclient/tests/test_domain.py (+47/-0)
src/mailmanclient/tests/utils.py (+0/-144)
template.py (+2/-2)
tox.ini (+15/-0)
unittest.cfg (+10/-0)
To merge this branch: bzr merge lp://staging/~barry/mailman.client/bilingual
Reviewer Review Type Date Requested Status
Florian Fuchs Pending
Review via email: mp+245537@code.staging.launchpad.net

Description of the change

mailmanclient's test suite no longer imports from the Mailman 3 code; it only uses the REST API.

It now uses `tox` to run the test suite against multiple versions of Python, and nose2 as the test runner.

Drop the use of distribute since it's been merged with setuptools.

Port to Python 3 (I've tested against 3.4 and 2.7 but I don't have a 2.6, 3.2, or 3.3 environment).

Added support for the new `queues` resource (py3 MM3 branch only; merged RSN).

Removed some unnecessary files and code; cleaned up some other stuff.

README.txt -> README.rst
using.txt -> using.rst
NEWS.txt -> NEWS.rst
Added a TESTING.rst file.

To post a comment you must log in.
Revision history for this message
Florian Fuchs (flo-fuchs) wrote :

Hi Barry,

thank you very much for this change! I ran the tests like described in TESTING.rst and also did some manual testing with fresh checkouts of the core, client and postorius. Worked like a charm.

I talked to Aurélien today about whether to wait with this merge until we have corresponing tests in Postorius and Hyperkitty. But we agreed that we better merge this branch now and then take care of Postorius and HK.

Flo

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches