launchpad-reviewers team mailing list archive

["Thiago F. Pappacena" <pappacena@xxxxxxxxx>]

Thiago F. Pappacena has proposed merging ~pappacena/launchpad:test-fix-no-pkg-filter into launchpad:master.

Commit message:
Fixing broken test that expects wrong error message when no filter is filled on package search page.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~pappacena/launchpad/+git/launchpad/+merge/377191
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~pappacena/launchpad:test-fix-no-pkg-filter into launchpad:master.

diff --git a/lib/lp/soyuz/stories/ppa/xx-delete-packages.txt b/lib/lp/soyuz/stories/ppa/xx-delete-packages.txt
index 7f0f2b7..da6db24 100644
--- a/lib/lp/soyuz/stories/ppa/xx-delete-packages.txt
+++ b/lib/lp/soyuz/stories/ppa/xx-delete-packages.txt
@@ -420,13 +420,14 @@ filter is 'Any Status', but the user can choose another.
     ['']
 
 When the user selects 'Published' filter and update the results, no
-records are presented.
+records are presented. No error message should be shown, since no text
+filter was added.
 
     >>> user_browser.getControl(
     ...     name='field.status_filter').value = ['published']
     >>> user_browser.getControl("Filter").click()
     >>> print_ppa_packages(user_browser.contents)
-    No matching package for ''.
+
 
 When they select 'Superseded' the SUPERSEDED source shows up again.