Code review comment for lp:~rvb/launchpad/fix-missing-addseries-link

Revision history for this message
Henning Eggers (henninge) wrote :

Am 14.03.2011 13:29, schrieb Raphaël Victor Badin:
> Fixed, thanks for the careful review.

You are welcome! You forgot to fix the imports, though. ;-) Here is what the
format-imports utility script came up with.

1=== modified file 'lib/lp/registry/browser/tests/test_distribution.py'
2--- lib/lp/registry/browser/tests/test_distribution.py 2011-03-14 12:22:17 +0000
3+++ lib/lp/registry/browser/tests/test_distribution.py 2011-03-14 13:51:27 +0000
4@@ -4,9 +4,14 @@
5
6 __metaclass__ = type
7
8+import soupmatchers
9+from testtools.matchers import (
10+ MatchesAny,
11+ Not,
12+ )
13+
14 from canonical.launchpad.webapp import canonical_url
15 from canonical.testing.layers import DatabaseFunctionalLayer
16-
17 from lp.registry.interfaces.series import SeriesStatus
18 from lp.testing import (
19 login_celebrity,
20@@ -15,11 +20,6 @@
21 )
22 from lp.testing.views import create_initialized_view
23
24-from testtools.matchers import MatchesAny
25-from testtools.matchers import Not
26-
27-import soupmatchers
28-
29
30 class TestDistributionPage(TestCaseWithFactory):
31 """A TestCase for the distribution index page."""

« Back to merge proposal