launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:weaken-person-packages-test into launchpad:master.

Commit message:
Weaken timing-dependent xx-person-packages test

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/390740

Depending on how slow the test system is, the PPA deletion test in xx-person-packages.txt might print something like "11 seconds ago" rather than "a moment ago".  We aren't specifically testing the timing here, so weaken this test.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:weaken-person-packages-test into launchpad:master.

diff --git a/lib/lp/soyuz/stories/soyuz/xx-person-packages.txt b/lib/lp/soyuz/stories/soyuz/xx-person-packages.txt
index d815121..95ff407 100644
--- a/lib/lp/soyuz/stories/soyuz/xx-person-packages.txt
+++ b/lib/lp/soyuz/stories/soyuz/xx-person-packages.txt
@@ -400,7 +400,7 @@ Then delete the 'source2' package.
     ...         print(extract_text(empty_section))
     >>> print_ppa_packages(admin_browser.contents)
     Source             Published   Status     Series   Section  Build Status
-    source2 - 666...   a moment ago Deleted   ...
+    source2 - 666...               Deleted    ...
     >>> update_cached_records()
 
 Now re-list the PPA's packages, 'source2' was deleted but still