Merge lp:~rvb/launchpad/initseries-bug-795537 into lp:launchpad

[Raphaël Badin]

[r=benji][bug=795537] Add the link +initseries to a distroseries' index page.

This branch adds the link +initseries to the home page of a distroseries.
This link is only displayed:
- if the feature flag (soyuz.derived_series_ui.enabled) is enabled;
- to users with lp.Admin;
- if the series is not already initialized or currently initializing.

= Tests =

./bin/test -cvv test_distroseries test_differences_init_link_no_feature
./bin/test -cvv test_distroseries test_differences_init_link_admin
./bin/test -cvv test_distroseries test_differences_init_link_not_admin
./bin/test -cvv test_distroseries test_differences_init_link_initialized
./bin/test -cvv test_distroseries test_differences_init_link_series_initializing

= Q/A =

On DF:
- as admin: make sure that the link is present on a series homepage.
- as non admin: make sure that the link is not present on a series homepage.
- as admin: the link should not be present for a series already initialized, or a series initializing.

[Benji York (benji)]

This looks good.

review: Approve (code)

=== modified file 'lib/lp/registry/browser/distroseries.py'
--- lib/lp/registry/browser/distroseries.py	2011-06-09 13:11:57 +0000
+++ lib/lp/registry/browser/distroseries.py	2011-06-13 09:27:22 +0000
@@ -218,6 +218,7 @@
                  'queue',
                  'add_port',
                  'create_milestone',
+                 'initseries',
                  ]
         add_subscribe_link(links)
         links.append('admin')
@@ -275,6 +276,15 @@
         text = 'Show uploads'
         return Link('+queue', text, icon='info')
 
+    @enabled_with_permission('launchpad.Admin')
+    def initseries(self):
+        enabled = (
+             getFeatureFlag('soyuz.derived_series_ui.enabled') is not None and
+             not self.context.isInitializing() and
+             not self.context.isInitialized())
+        text = 'Initialize series'
+        return Link('+initseries', text, icon='edit', enabled=enabled)
+
 
 class DistroSeriesBugsMenu(ApplicationMenu, StructuralSubscriptionMenuMixin):
 
 
=== modified file 'lib/lp/registry/browser/tests/test_distroseries.py'
--- lib/lp/registry/browser/tests/test_distroseries.py	2011-06-09 13:11:57 +0000
+++ lib/lp/registry/browser/tests/test_distroseries.py	2011-06-13 09:27:22 +0000
@@ -328,6 +328,82 @@
         self.assertTrue(derived_series.isInitializing())
         self.assertThat(html_content, portlet_display)
 
+    def assertInitSeriesLinkPresent(self, series, person):
+        self._assertInitSeriesLink(series, person, True)
+
+    def assertInitSeriesLinkNotPresent(self, series, person):
+        self._assertInitSeriesLink(series, person, False)
+
+    def _assertInitSeriesLink(self, series, person, present=True):
+        # Helper method to check the presence/absence of the link to
+        # +iniseries.
+        if person == 'admin':
+            person = getUtility(ILaunchpadCelebrities).admin.teamowner
+
+        init_link_matcher = soupmatchers.HTMLContains(
+            soupmatchers.Tag(
+                'Initialize series', 'a',
+                text='Initialize series',
+                attrs={'href': '%s/+initseries' % canonical_url(series)}))
+
+        with person_logged_in(person):
+            view = create_initialized_view(
+                series,
+                '+index',
+                principal=person)
+            html_content = view()
+
+        if present:
+            self.assertThat(html_content, init_link_matcher)
+        else:
+            self.assertThat(html_content, Not(init_link_matcher))
+
+    def test_differences_init_link_no_feature(self):
+        # The link to +initseries is not displayed if the feature flag
+        # is not enabled.
+        series = self.factory.makeDistroSeries()
+
+        self.assertInitSeriesLinkNotPresent(series, 'admin')
+
+    def test_differences_init_link_admin(self):
+        # The link to +initseries is displayed to admin users if the
+        # feature flag is enabled.
+        set_derived_series_ui_feature_flag(self)
+        series = self.factory.makeDistroSeries()
+
+        self.assertInitSeriesLinkPresent(series, 'admin')
+
+    def test_differences_init_link_not_admin(self):
+        # The link to +initseries is not displayed to not admin users if the
+        # feature flag is enabled.
+        set_derived_series_ui_feature_flag(self)
+        series = self.factory.makeDistroSeries()
+        person = self.factory.makePerson()
+
+        self.assertInitSeriesLinkNotPresent(series, person)
+
+    def test_differences_init_link_initialized(self):
+        # The link to +initseries is not displayed if the series is
+        # already initialized (i.e. has any published package).
+        set_derived_series_ui_feature_flag(self)
+        series = self.factory.makeDistroSeries()
+        self.factory.makeSourcePackagePublishingHistory(
+            archive=series.main_archive,
+            distroseries=series)
+
+        self.assertInitSeriesLinkNotPresent(series, 'admin')
+
+    def test_differences_init_link_series_initializing(self):
+        # The link to +initseries is not displayed if the series is
+        # initializing.
+        set_derived_series_ui_feature_flag(self)
+        series = self.factory.makeDistroSeries()
+        parent_series = self.factory.makeDistroSeries()
+        job_source = getUtility(IInitialiseDistroSeriesJobSource)
+        job_source.create(series, [parent_series.id])
+
+        self.assertInitSeriesLinkNotPresent(series, 'admin')
+
 
 class TestMilestoneBatchNavigatorAttribute(TestCaseWithFactory):
     """Test the series.milestone_batch_navigator attribute."""
 
=== modified file 'lib/lp/registry/templates/distroseries-index.pt'
--- lib/lp/registry/templates/distroseries-index.pt	2011-06-09 12:00:48 +0000
+++ lib/lp/registry/templates/distroseries-index.pt	2011-06-13 09:27:22 +0000
@@ -147,6 +147,9 @@
           <li tal:condition="overview_menu/admin/enabled">
             <a tal:replace="structure overview_menu/admin/fmt:link" />
           </li>
+          <li tal:condition="overview_menu/initseries/enabled|nothing">
+            <a tal:replace="structure overview_menu/initseries/fmt:link" />
+          </li>
           <li tal:condition="overview_menu/subscribe/enabled|nothing">
             <a tal:replace="structure overview_menu/subscribe/fmt:link" />
           </li>