Merge lp:~jcsackett/launchpad/get-loc-for-bugtracker-fix into lp:launchpad

Proposed by j.c.sackett
Status: Merged
Approved by: Curtis Hovey
Approved revision: no longer in the source branch.
Merged at revision: 15965
Proposed branch: lp:~jcsackett/launchpad/get-loc-for-bugtracker-fix
Merge into: lp:launchpad
Prerequisite: lp:~jcsackett/launchpad/bug-tracker-inactive-projects
Diff against target: 71 lines (+13/-37)
2 files modified
lib/lp/bugs/browser/tests/test_bugtracker_views.py (+13/-0)
lib/lp/bugs/stories/bugtracker/xx-bugtracker.txt (+0/-37)
To merge this branch: bzr merge lp:~jcsackett/launchpad/get-loc-for-bugtracker-fix
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+124528@code.launchpad.net

Commit message

Do not show deactivated projects on but tracker pages.

Description of the change

Summary
=======
This branch gains LoC for a related bugfix by removing unnecessary doctest and
creating equivalent unit tests.

Implementation
==============
The doctest for the related project portlet is removed. There's not much worth
"documenting" and this was largely testing behavior. A unit test is added to
the bugtracker view test to ensure this behavior remains tested.

Tests
=====
bin/test -vvct test_bugtracker_view

QA
==
None

LoC
===
This removes LoC.

Lint
====

Checking for conflicts and issues in changed files.

Linting changed files:
  lib/lp/bugs/browser/tests/test_bugtracker_views.py
  lib/lp/bugs/browser/bugtracker.py
  lib/lp/bugs/stories/bugtracker/xx-bugtracker.txt

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/bugs/browser/tests/test_bugtracker_views.py'
2--- lib/lp/bugs/browser/tests/test_bugtracker_views.py 2012-09-14 21:44:35 +0000
3+++ lib/lp/bugs/browser/tests/test_bugtracker_views.py 2012-09-14 21:44:35 +0000
4@@ -24,7 +24,20 @@
5
6 layer = DatabaseFunctionalLayer
7
8+ def test_related_projects(self):
9+ # Related products and projectgroups are shown by the view.
10+ tracker = self.factory.makeBugTracker()
11+ project_group = self.factory.makeProject()
12+ product = self.factory.makeProduct()
13+ admin = getUtility(ILaunchpadCelebrities).admin.teamowner
14+ with person_logged_in(admin):
15+ project_group.bugtracker = tracker
16+ product.bugtracker = tracker
17+ view = create_initialized_view(tracker, name='+index')
18+ self.assertEqual([project_group, product], view.related_projects)
19+
20 def test_linked_projects_only_shows_active_projects(self):
21+ # Inactive projects are not shown as the related projects.
22 tracker = self.factory.makeBugTracker()
23 active_product = self.factory.makeProduct()
24 inactive_product = self.factory.makeProduct()
25
26=== modified file 'lib/lp/bugs/stories/bugtracker/xx-bugtracker.txt'
27--- lib/lp/bugs/stories/bugtracker/xx-bugtracker.txt 2012-07-17 14:29:17 +0000
28+++ lib/lp/bugs/stories/bugtracker/xx-bugtracker.txt 2012-09-14 21:44:35 +0000
29@@ -688,43 +688,6 @@
30 Copy, Cut and Delete operations should work on selections
31 --> —: None
32
33-
34-Related projects portlet
35-------------------------
36-
37-Related projects and project groups are shown in a portlet on the bug
38-tracker page.
39-
40-First we need to change a project group and a project to use our new bug
41-tracker:
42-
43- >>> admin_browser.open('http://launchpad.dev/mozilla/+edit')
44- >>> admin_browser.getControl('Bug Tracker:').value = 'testbugzilla'
45- >>> admin_browser.getControl('Change Details').click()
46-
47- >>> admin_browser.open(
48- ... 'http://launchpad.dev/jokosher/+configure-bugtracker')
49- >>> admin_browser.getControl(name='field.bugtracker'
50- ... ).displayValue = ['In a registered bug tracker:']
51- >>> admin_browser.getControl(name='field.bugtracker.bugtracker'
52- ... ).value = 'testbugzilla'
53- >>> admin_browser.getControl('Change').click()
54-
55-Now the Mozilla Project and Jokosher will appear in the Related projects
56-portlet on the bug tracker page:
57-
58- >>> admin_browser.open(
59- ... 'http://launchpad.dev/bugs/bugtrackers/testbugzilla')
60- >>> print extract_text(find_portlet(
61- ... admin_browser.contents, 'Related projects'))
62- Related projects
63- You can link a registered project to a
64- registered bug tracker in the project's “Change
65- details” page.
66- The Mozilla Project
67- Jokosher
68-
69-
70 Info portlet
71 ------------
72