Merge ~cjwatson/launchpad:narrative-doctest-fixes into launchpad:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 0c9e77e72c5a93fceaa8071ebef6dbc3b0bbeb85
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:narrative-doctest-fixes
Merge into: launchpad:master
Diff against target: 93 lines (+7/-49)
3 files modified
lib/lp/bugs/stories/upstream-bugprivacy/xx-upstream-bug-privacy.txt (+0/-39)
lib/lp/services/gpg/doc/gpg-signatures.txt (+6/-9)
lib/lp/translations/utilities/doc/gettext_mo_exporter.txt (+1/-1)
Reviewer Review Type Date Requested Status
Ioana Lasc (community) Approve
Review via email: mp+406853@code.launchpad.net

Commit message

Fix various minor lint issues in doctest narratives

Description of the change

These were all caught by lp-lint-doctest as "narrative exceeds 78 characters", but the fixes vary.

To post a comment you must log in.
Revision history for this message
Ioana Lasc (ilasc) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lib/lp/bugs/stories/upstream-bugprivacy/xx-upstream-bug-privacy.txt b/lib/lp/bugs/stories/upstream-bugprivacy/xx-upstream-bug-privacy.txt
2index a0ae7e0..168907b 100644
3--- a/lib/lp/bugs/stories/upstream-bugprivacy/xx-upstream-bug-privacy.txt
4+++ b/lib/lp/bugs/stories/upstream-bugprivacy/xx-upstream-bug-privacy.txt
5@@ -181,42 +181,3 @@ context.
6 ... """))
7 HTTP/1.1 404 Not Found
8 ...
9-
10-Sample Person views a bug, which they're about to set private:
11-
12- >>> print(http(br"""
13- ... GET /firefox/+bug/4/+edit HTTP/1.1
14- ... Authorization: Basic dGVzdEBjYW5vbmljYWwuY29tOnRlc3Q=
15- ... """))
16- HTTP/1.1 200 Ok
17- ...
18- ...Reflow problems with complex page layouts...
19- ...
20-
21-Sample Person sets the bug private and is made an explicit subscriber
22-in the process.
23-
24- ... POST /firefox/+bug/4/+secrecy HTTP/1.1
25- ... Authorization: Basic dGVzdEBjYW5vbmljYWwuY29tOnRlc3Q=
26- ... Content-Length: 429
27- ... Content-Type: multipart/form-data; boundary=---------------------------10389799518848978361196772104
28- ...
29- ... -----------------------------10389799518848978361196772104
30- ... Content-Disposition: form-data; name="field.private.used"
31- ...
32- ...
33- ... -----------------------------10389799518848978361196772104
34- ... Content-Disposition: form-data; name="field.private"
35- ...
36- ... on
37- ... -----------------------------10389799518848978361196772104
38- ... Content-Disposition: form-data; name="UPDATE_SUBMIT"
39- ...
40- ... Change
41- ... -----------------------------10389799518848978361196772104--
42- ... """)
43- HTTP/1.1 200 Ok
44- ...
45- ...Cc:...
46- ...Sample Person...
47- ...
48diff --git a/lib/lp/services/gpg/doc/gpg-signatures.txt b/lib/lp/services/gpg/doc/gpg-signatures.txt
49index 2c46970..4f16924 100644
50--- a/lib/lp/services/gpg/doc/gpg-signatures.txt
51+++ b/lib/lp/services/gpg/doc/gpg-signatures.txt
52@@ -137,9 +137,12 @@ signed text can not be attributed solely to either key:
53 ... -----END PGP SIGNATURE-----
54 ... """
55
56-XXX: See discussion on the launchpad@ mailing list for an explanation to this
57-change. -- Guilherme Salgado, 2007-04-05
58-(https://lists.ubuntu.com/mailman/private/launchpad/2007-April/015085.html)
59+Originally we could test for the exception text "Single signature expected,
60+found multiple signatures", but this stopped working as of
61+https://ubuntu.com/security/notices/USN-432-2
62+(https://launchpad.net/ubuntu/+source/gpgme1.0/1.1.0-1ubuntu0.1), and GPGME
63+now only gives us a rather less informative "Bad data" exception. We don't
64+care too much about the details as long as it fails.
65
66 >>> master_sig = gpghandler.getVerifiedSignature(content)
67 ... # doctest: +IGNORE_EXCEPTION_MODULE_IN_PYTHON2
68@@ -147,12 +150,6 @@ change. -- Guilherme Salgado, 2007-04-05
69 ...
70 lp.services.gpg.interfaces.GPGVerificationError: ...
71
72-# >>> master_sig = gpghandler.getVerifiedSignature(content)
73-# ... # doctest: +IGNORE_EXCEPTION_MODULE_IN_PYTHON2
74-# Traceback (most recent call last):
75-# ...
76-# lp.services.gpg.interfaces.GPGVerificationError: Single signature expected, found multiple signatures
77-
78 The text below was signed by key that's is not part of the
79 imported keyring. Note that we have extra debug information containing
80 the GPGME error codes (they may be helpful).
81diff --git a/lib/lp/translations/utilities/doc/gettext_mo_exporter.txt b/lib/lp/translations/utilities/doc/gettext_mo_exporter.txt
82index e98b784..366d9a5 100644
83--- a/lib/lp/translations/utilities/doc/gettext_mo_exporter.txt
84+++ b/lib/lp/translations/utilities/doc/gettext_mo_exporter.txt
85@@ -8,7 +8,7 @@ friendly.
86
87 You can read more about MO file format on:
88
89-http://www.gnu.org/software/gettext/manual/html_chapter/gettext_10.html#MO-Files
90+https://www.gnu.org/software/gettext/manual/html_node/MO-Files.html
91
92 >>> from zope.interface.verify import verifyObject
93 >>> from lp.services.helpers import bytes_to_tarfile

Subscribers

People subscribed via source and target branches

to status/vote changes: