Merge lp:~sinzui/launchpad/unicode-project-group-bug into lp:launchpad

[Curtis Hovey]

Reporting a bug from a project group accepts unicode bug summaries

The report a bug from a project group form fails if the summary
contains unicode. form forwards the request to the selected project,
but the summary is assumed to be ascii. The call to URLEncode fails
when give unicode.

--------------------------------------------------------------------

RULES

    Pre-implementation: no one
    * ProjectGroupFileBugGuidedView.projectgroup_search_action needs
      to decode the summary before passing it to URLEncode.

    ADDENDUM:
    * The redirect does not go to the bug domain. It should.

QA

    * Visit https://bugs.qastaging.launchpad.net/ubuntu-fr-website-project/+filebug
    * Choose Forum Ubuntu-fr
    * Enter "Le bouton liste à puces n'est pas correct" in the summary field.
    * Choose Continue
    * Verify the page redirect to the Forum Ubuntu-fr project and
      shows the matches and/or bug form.
    * Verify the url using the bugs host.

LINT

    lib/lp/bugs/browser/bugtarget.py
    lib/lp/bugs/browser/tests/test_bugtarget_filebug.py

LoC

    I have a 3000 line credit this week.

TEST

    ./bin/test -vvc -t ProjectGroup lp.bugs.browser.tests.test_bugtarget_filebug

IMPLEMENTATION

I discovered that the redirect url went to the wrong host when I was
investigating a fix. I added the missing rootsite arg and test for it.
I expected the test to verify that the summary is encoded as utf8 before
passing it to URLEncode(). Note that will bug tags have to be unicode in
tests, the validator requires that they are ascii; it is not possibly
to have a similar problem with bug tags.
    lib/lp/bugs/browser/bugtarget.py
    lib/lp/bugs/browser/tests/test_bugtarget_filebug.py

[Richard Harding (rharding)]

review: Approve

=== modified file 'lib/lp/bugs/browser/bugtarget.py'
--- lib/lp/bugs/browser/bugtarget.py	2012-11-06 05:56:49 +0000
+++ lib/lp/bugs/browser/bugtarget.py	2012-11-13 17:58:22 +0000
@@ -1081,10 +1081,12 @@
     @action("Continue", name="projectgroupsearch")
     def projectgroup_search_action(self, action, data):
         """Redirect to the chosen product's form."""
-        base = canonical_url(data['product'], view_name='+filebug')
+        base = canonical_url(
+            data['product'], view_name='+filebug', rootsite='bugs')
+        title = data['title'].encode('utf8')
         query = urllib.urlencode([
             ('field.actions.search', 'Continue'),
-            ('field.title', data['title']),
+            ('field.title', title),
             ('field.tags', ' '.join(data['tags'])),
             ])
         url = '%s?%s' % (base, query)

=== modified file 'lib/lp/bugs/browser/tests/test_bugtarget_filebug.py'
--- lib/lp/bugs/browser/tests/test_bugtarget_filebug.py	2012-11-06 04:34:48 +0000
+++ lib/lp/bugs/browser/tests/test_bugtarget_filebug.py	2012-11-13 17:58:22 +0000
@@ -813,6 +813,51 @@
         self.assertIn("Thank you for your bug report.", msg)
 
 
+class ProjectGroupFileBugGuidedViewTestCase(TestCaseWithFactory):
+
+    layer = DatabaseFunctionalLayer
+
+    def makeProjectGroupFileBugView(self, product_name, bug_title, tags=''):
+        project_group = self.factory.makeProject()
+        owner = project_group.owner
+        product = self.factory.makeProduct(
+            owner=owner, name=product_name, project=project_group)
+        with person_logged_in(owner):
+            product.official_malone = True
+        form = {
+            'field.product': product_name,
+            'field.title': bug_title,
+            'field.tags': tags,
+            'field.actions.projectgroupsearch': 'Continue',
+            }
+        view = create_initialized_view(
+            project_group, name='+filebug', form=form, rootsite='bugs')
+        return view
+
+    def test_redirect_to_project(self):
+        # The view redirects to the select sub project.
+        view = self.makeProjectGroupFileBugView('fnord', 'A bug', u'is os')
+        response = view.request.response
+        self.assertEqual(302, response.getStatus())
+        self.assertEqual(
+            'http://bugs.launchpad.dev/fnord/+filebug?'
+            'field.actions.search=Continue&'
+            'field.title=A+bug&'
+            'field.tags=is+os',
+            response.getHeader('Location'))
+
+    def test_redirect_to_project_unicode_summary(self):
+        # The summary is reencoded properly when it contains unicode.
+        view = self.makeProjectGroupFileBugView('fnord', u'caf\xe9', '')
+        response = view.request.response
+        self.assertEqual(
+            'http://bugs.launchpad.dev/fnord/+filebug?'
+            'field.actions.search=Continue&'
+            'field.title=caf%C3%A9&'
+            'field.tags=',
+            response.getHeader('Location'))
+
+
 class TestFileBugRequestCache(TestCaseWithFactory):
     # Tests to ensure the request cache contains the expected values for
     # file bug views.

=== modified file 'lib/lp/bugs/stories/guided-filebug/xx-bug-reporting-guidelines.txt'
--- lib/lp/bugs/stories/guided-filebug/xx-bug-reporting-guidelines.txt	2012-08-03 00:18:27 +0000
+++ lib/lp/bugs/stories/guided-filebug/xx-bug-reporting-guidelines.txt	2012-11-13 17:58:22 +0000
@@ -66,7 +66,7 @@
 
     >>> for context_name, context_path, view in contexts:
     ...     filebug_url = (
-    ...         'http://launchpad.dev/%s/+filebug' % (context_path,))
+    ...         'http://bugs.launchpad.dev/%s/+filebug' % (context_path,))
     ...     user_browser.open(filebug_url)
     ...     user_browser.getControl('Summary', index=0).value = (
     ...         "It doesn't work")
@@ -78,28 +78,28 @@
     ...     print_acknowledgement_message(user_browser)
     *
     Ubuntu
-      <http://launchpad.dev/ubuntu/+filebug>
+      <http://bugs.launchpad.dev/ubuntu/+filebug>
     Ubuntu bug reporting guidelines:
     The version of Ubuntu you're using.
     See http://example.com for more details.
     Thank you for filing a bug for https://launchpad.dev/ubuntu
     *
     Mozilla
-      <http://launchpad.dev/firefox/+filebug?field.actions.search=Continue&field.title=It+doesn%27t+work&field.tags=>
+      <http://bugs.launchpad.dev/firefox/+filebug?field.actions.search=Continue&field.title=It+doesn%27t+work&field.tags=>
     Mozilla Firefox bug reporting guidelines:
     The version of Firefox you're using.
     See http://example.com for more details.
     Thank you for filing a bug for https://launchpad.dev/firefox
     *
     Firefox
-      <http://launchpad.dev/firefox/+filebug>
+      <http://bugs.launchpad.dev/firefox/+filebug>
     Mozilla Firefox bug reporting guidelines:
     The version of Firefox you're using.
     See http://example.com for more details.
     Thank you for filing a bug for https://launchpad.dev/firefox
     *
     alsa-utils in Ubuntu
-      <http://launchpad.dev/ubuntu/+source/alsa-utils/+filebug>
+      <http://bugs.launchpad.dev/ubuntu/+source/alsa-utils/+filebug>
     alsa-utils (Ubuntu) bug reporting guidelines:
     The version of alsa-utils in Ubuntu you're using.
     See http://example.com for more details.