Launchpad itself

Two of the ServiceFactory tests were incorrect, and only coincidentally
passed. They were testing that using getUserBrowser to traverse to
https://launchpad.test/api/devel/+services and
https://launchpad.test/api/devel/+services/invalid raised NotFound; but
this only worked because traversing to 'api' from getUserBrowser raises
NotFound (I think it doesn't set up quite the right kind of request),
and in fact https://launchpad.net/api/devel/+services works just fine.

Bring this more into line with test_service_traversal (which is correct)
by using test_traverse instead, and rewrite the test for traversing to
the service factory to assert its actual behaviour of exporting the
service factory (even though that isn't currently very useful).

Merged from https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/375033

26ef771... by Colin Watson on 2019-11-01

Fix incorrect ServiceFactory tests

146cebb... by Colin Watson on 2019-11-01

Fix incomplete messages from warning handler

The warning handler sometimes emitted messages like this (with local
paths shortened for brevity):

  The method SimpleViewClass from .../lib/lp/bugs/browser/../templates/buglisting-default.pt._getBatchNavigator
  The doctest <doctest xx-bug-subscriptions.txt[4]>, at the line:
      >>> browser.getControl('Save these changes').click()

This is clearly incomplete. With this change, we instead get:

.../env/lib/python2.7/site-packages/lazr/batchnavigator/_batchnavigator.py:82: PendingDeprecationWarning: cgi.parse_qs is deprecated, use urlparse.parse_qs instead
return cgi.parse_qs(query_string, keep_blank_values=True)

Merged from https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/374682

883633b... by Colin Watson on 2019-10-31