← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] ~cjwatson/launchpad:doctest-source-line-length-noqa into launchpad:master

 

Colin Watson has proposed merging ~cjwatson/launchpad:doctest-source-line-length-noqa into launchpad:master.

Commit message:
Ignore various source line length errors in doctests

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/407267

This covers cases where it isn't reasonable or practical to wrap the source lines, such as long HTTP GET URLs.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:doctest-source-line-length-noqa into launchpad:master.
diff --git a/lib/lp/app/stories/basics/xx-lowercase-redirection.txt b/lib/lp/app/stories/basics/xx-lowercase-redirection.txt
index ae59e84..b70b2bd 100644
--- a/lib/lp/app/stories/basics/xx-lowercase-redirection.txt
+++ b/lib/lp/app/stories/basics/xx-lowercase-redirection.txt
@@ -32,7 +32,7 @@ When doing a POST to an invalid URL, we get an error:
     ... POST /UbUntU/hoary/+source/evolution/+pots/evolution-2.2/es/+translate HTTP/1.1
     ... Host: translations.launchpad.test
     ... Referer: https://launchpad.test/
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 500 Internal Server Error
     ...
     </ul><p>...POSTToNonCanonicalURL...
diff --git a/lib/lp/blueprints/stories/blueprints/xx-productseries.txt b/lib/lp/blueprints/stories/blueprints/xx-productseries.txt
index 2dc8d95..309a17f 100644
--- a/lib/lp/blueprints/stories/blueprints/xx-productseries.txt
+++ b/lib/lp/blueprints/stories/blueprints/xx-productseries.txt
@@ -67,7 +67,7 @@ so the targeting should NOT be automatically approved.
     ...
     ... Continue
     ... -----------------------------26999413214087432371486976730--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
     Content-Length: 0
@@ -112,7 +112,7 @@ OK, we will also pitch the e4x spec to the same series:
     ...
     ... Continue
     ... -----------------------------26999413214087432371486976730--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
     Content-Length: 0
@@ -203,7 +203,7 @@ because we are an admin, then we will move it back.
     ...
     ... Continue
     ... -----------------------------26999413214087432371486976730--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
     Content-Length: 0
@@ -246,7 +246,7 @@ And lets put it back:
     ...
     ... Continue
     ... -----------------------------26999413214087432371486976730--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
     Content-Length: 0
diff --git a/lib/lp/bugs/doc/bugwatch.txt b/lib/lp/bugs/doc/bugwatch.txt
index 415be41..17e6f6a 100644
--- a/lib/lp/bugs/doc/bugwatch.txt
+++ b/lib/lp/bugs/doc/bugwatch.txt
@@ -191,7 +191,7 @@ easier you can use IBugWatchSet.fromText().
     ...         mailto:foo.bar@xxxxxxxxxxxxx
     ...     A Google Code URL:
     ...         http://code.google.com/p/myproject/issues/detail?id=12345
-    ... """
+    ... """  # noqa
     >>> bug_watches = getUtility(IBugWatchSet).fromText(
     ...     text, bug_one, sample_person)
     >>> bugs_and_types = [
diff --git a/lib/lp/bugs/stories/bugs/xx-bugtask-assignee-widget.txt b/lib/lp/bugs/stories/bugs/xx-bugtask-assignee-widget.txt
index cff92cf..d19c86a 100644
--- a/lib/lp/bugs/stories/bugs/xx-bugtask-assignee-widget.txt
+++ b/lib/lp/bugs/stories/bugs/xx-bugtask-assignee-widget.txt
@@ -69,7 +69,7 @@ button:
     ... 
     ... Save Changes
     ... -----------------------------19759086281403130373932339922--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
 
@@ -143,7 +143,7 @@ But, you can also assign the task to another person, of course:
     ... 
     ... Save Changes
     ... -----------------------------19759086281403130373932339922--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
 
@@ -217,7 +217,7 @@ Lastly, the widget also allows you to simply assign the task to nobody
     ... 
     ... Save Changes
     ... -----------------------------19759086281403130373932339922--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
 
diff --git a/lib/lp/bugs/stories/bugtask-searches/xx-distribution-statistics-portlet.txt b/lib/lp/bugs/stories/bugtask-searches/xx-distribution-statistics-portlet.txt
index 4d20e84..4200c3d 100644
--- a/lib/lp/bugs/stories/bugtask-searches/xx-distribution-statistics-portlet.txt
+++ b/lib/lp/bugs/stories/bugtask-searches/xx-distribution-statistics-portlet.txt
@@ -15,7 +15,7 @@ Viewing critical bugs as Sample Person:
     >>> print(http(br"""
     ... GET /debian/+bugs?field.status%3Alist=New&field.status%3Alist=Confirmed&field.importance%3Alist=Critical&search=Search HTTP/1.1
     ... Authorization: Basic dGVzdEBjYW5vbmljYWwuY29tOnRlc3Q=
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 200 Ok
     ...No results for search...
 
@@ -24,7 +24,7 @@ Viewing bugs "assigned to me", as Sample Person:
     >>> print(http(br"""
     ... GET /debian/+bugs?field.status%3Alist=New&field.status%3Alist=Confirmed&field.assignee=name12&search=Search HTTP/1.1
     ... Authorization: Basic dGVzdEBjYW5vbmljYWwuY29tOnRlc3Q=
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 200 Ok
     ...1 result...
 
@@ -42,7 +42,7 @@ Viewing unassigned bugs as Sample Person:
     >>> print(http(br"""
     ... GET /debian/+bugs?field.status%3Alist=New&field.status%3Alist=Confirmed&field.status-empty-marker=1&field.importance-empty-marker=1&field.assignee=&assignee_option=none&search=Search HTTP/1.1
     ... Authorization: Basic dGVzdEBjYW5vbmljYWwuY29tOnRlc3Q=
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 200 Ok
     ...2 results...
 
diff --git a/lib/lp/bugs/stories/guided-filebug/xx-filebug-attachments.txt b/lib/lp/bugs/stories/guided-filebug/xx-filebug-attachments.txt
index ac1f245..cdaa202 100644
--- a/lib/lp/bugs/stories/guided-filebug/xx-filebug-attachments.txt
+++ b/lib/lp/bugs/stories/guided-filebug/xx-filebug-attachments.txt
@@ -103,6 +103,6 @@ treat all empty-equivalent values equally.
     ... 
     ... Submit Bug Report
     ... -----------------------------2051078912280543729816242321--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other...
     Location: http://bugs.launchpad.test/firefox/+bug/...
diff --git a/lib/lp/registry/stories/productseries/xx-productseries-review.txt b/lib/lp/registry/stories/productseries/xx-productseries-review.txt
index 0bdea46..638a98d 100644
--- a/lib/lp/registry/stories/productseries/xx-productseries-review.txt
+++ b/lib/lp/registry/stories/productseries/xx-productseries-review.txt
@@ -19,7 +19,7 @@ to bazaar. Also changes the name of the productseries to 'newname'.
     ...
     ... Change
     ... -----------------------------10572808480422220968425074--
-    ... """))
+    ... """))  # noqa
     HTTP/1.1 303 See Other
     ...
     Location: http://localhost/bazaar/newname...
diff --git a/lib/lp/registry/stories/team-polls/xx-poll-condorcet-voting.txt b/lib/lp/registry/stories/team-polls/xx-poll-condorcet-voting.txt
index 3499cf1..48bfe44 100644
--- a/lib/lp/registry/stories/team-polls/xx-poll-condorcet-voting.txt
+++ b/lib/lp/registry/stories/team-polls/xx-poll-condorcet-voting.txt
@@ -49,6 +49,7 @@
     ... Referer: https://launchpad.test/
     ...
     ... token=xn9FDCTp4m&showvote=Show+My+Vote&option_12=&option_13=&option_14=&option_15="""))
+    ... # noqa
     HTTP/1.1 200 Ok
     ...
                     <p>Your current vote is as follows:</p>
@@ -85,6 +86,7 @@
     ... Referer: https://launchpad.test/
     ...
     ... token=xn9FDCTp4m&option_12=2&option_13=3&option_14=4&option_15=1&changevote=Change+Vote"""))
+    ... # noqa
     HTTP/1.1 200 Ok
     ...
     ...Your vote was changed successfully.</p>
@@ -173,6 +175,7 @@
     ... Referer: https://launchpad.test/
     ...
     ... option_16=4&option_17=2&option_18=1&option_19=3&changevote=Change+Vote"""))
+    ... # noqa
     HTTP/1.1 200 Ok
     ...
                     <p>Your current vote is as follows:</p>
diff --git a/lib/lp/services/messages/doc/message.txt b/lib/lp/services/messages/doc/message.txt
index 76f95a1..f51d39a 100644
--- a/lib/lp/services/messages/doc/message.txt
+++ b/lib/lp/services/messages/doc/message.txt
@@ -113,7 +113,7 @@ unadulterated, email into the Librarian and builds all the MessageChunks.
     ... =00>=00
     ... --===============0294339828==--
     ... The epilogue is not included, including Unicode\N{COPYRIGHT SIGN}
-    ... """.encode('latin1')
+    ... """.encode('latin1')  # noqa
     >>> msg = msgset.fromEmail(raw_msg)
 
 Once the email is stuffed into the Message and MessageChunk tables and
diff --git a/lib/lp/translations/utilities/doc/gettext_po_parser.txt b/lib/lp/translations/utilities/doc/gettext_po_parser.txt
index 09f370d..5b62a50 100644
--- a/lib/lp/translations/utilities/doc/gettext_po_parser.txt
+++ b/lib/lp/translations/utilities/doc/gettext_po_parser.txt
@@ -91,7 +91,7 @@ POHeader.
     ... Content-Type: text/plain; charset=UTF-8
     ... Content-Transfer-Encoding: 8bit
     ... Report-Msgid-Bugs-To: serrador@xxxxxxxxxxxxx
-    ... Plural-Forms: n != 1'''
+    ... Plural-Forms: n != 1'''  # noqa
     >>> new_header = POHeader(new_header_string)
 
 Plural forms are now the default ones. No plural forms.