Merge lp://staging/~mvo/software-center/hopefully-fix-lp1008229 into lp://staging/software-center
Status: | Merged |
---|---|
Merged at revision: | 3067 |
Proposed branch: | lp://staging/~mvo/software-center/hopefully-fix-lp1008229 |
Merge into: | lp://staging/software-center |
Diff against target: |
20 lines (+2/-1) 1 file modified
softwarecenter/db/debfile.py (+2/-1) |
To merge this branch: | bzr merge lp://staging/~mvo/software-center/hopefully-fix-lp1008229 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gary Lasker (community) | Approve | ||
Review via email: mp+113060@code.staging.launchpad.net |
Description of the change
Trivial branch to merge some vars to ensure that they are defined, even if the file is not there.
From reviewing the code it seems like its the only way to trigger the bug in #1008229.
A possible test for this would be:
=== modified file 'tests/
--- tests/test_
+++ tests/test_
@@ -32,6 +32,12 @@
def setUp(self):
self.db = get_test_db()
+ def test_regression
+ debfileapplication = DebFileApplicat
+ debfileapplicat
+ debfiledetails = debfileapplicat
+ self.assertEqua
+
def test_get_
but its really a hack as I couldn't find a better way to trigger it.
We should cherry pick this to 5.2 too