Merge lp:~zematynnad/ubuntu-webcatalog/fix_tests into lp:ubuntu-webcatalog

Proposed by Danny Tamez
Status: Merged
Approved by: Anthony Lenton
Approved revision: 91
Merged at revision: 90
Proposed branch: lp:~zematynnad/ubuntu-webcatalog/fix_tests
Merge into: lp:ubuntu-webcatalog
Diff against target: 39 lines (+4/-3)
3 files modified
src/webcatalog/tests/test_pep8.py (+1/-0)
src/webcatalog/tests/test_views.py (+2/-2)
test_requirements.txt (+1/-1)
To merge this branch: bzr merge lp:~zematynnad/ubuntu-webcatalog/fix_tests
Reviewer Review Type Date Requested Status
Anthony Lenton (community) Approve
Review via email: mp+99375@code.launchpad.net

Commit message

Fix tests on trunk

Description of the change

Overview
=========
This branch gets webcatlog trunk building again and fixes a few broken tests.

Details
========
test_requirements.txt was changed to require djang<1.4 so a 1.3x version would still be installed.

The links to http://ubuntu.com and https://canonical.com have changed to http://www.ubuntu.com and http://www.canonical.com respectively on the index page. It looks like the test was not yet updated.

Added a max_line_length attribute to the Options class as this was also causing the pep_8 test to fail.

To Test
=======
$fab bootstrap test

To post a comment you must log in.
Revision history for this message
Anthony Lenton (elachuni) wrote :

Thanks for cleaning this up Danny!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/webcatalog/tests/test_pep8.py'
2--- src/webcatalog/tests/test_pep8.py 2012-01-06 18:10:30 +0000
3+++ src/webcatalog/tests/test_pep8.py 2012-03-26 17:08:47 +0000
4@@ -57,6 +57,7 @@
5 show_pep8 = False
6 select = []
7 ignore = []
8+ max_line_length = 80
9 pep8.options = Options()
10 pep8.message = self.message
11 Options.physical_checks = pep8.find_checks('physical_line')
12
13=== modified file 'src/webcatalog/tests/test_views.py'
14--- src/webcatalog/tests/test_views.py 2012-03-20 22:23:31 +0000
15+++ src/webcatalog/tests/test_views.py 2012-03-26 17:08:47 +0000
16@@ -576,8 +576,8 @@
17 "https://bugs.launchpad.net/ubuntu-webcatalog/+filebug",
18 "http://developer.ubuntu.com",
19 "https://launchpad.net/ubuntu-webcatalog",
20- "http://ubuntu.com",
21- "https://canonical.com",
22+ "http://www.ubuntu.com",
23+ "http://www.canonical.com",
24 ]
25 response = self.client.get(reverse('wc-index'))
26 for link in links:
27
28=== modified file 'test_requirements.txt'
29--- test_requirements.txt 2011-11-22 08:49:31 +0000
30+++ test_requirements.txt 2012-03-26 17:08:47 +0000
31@@ -3,7 +3,7 @@
32 # Installed /.../virtualenv/lib/python2.6/site-packages/django_preflight-0.1-py2.6.egg
33 # error: Could not find required distribution django
34 -f bzr+http://bazaar.launchpad.net/~elachuni/canonical-identity-provider/ssoclient#egg=ssoclient-1.0
35-django
36+django<1.4
37 coverage
38 mock
39 piston-mini-client

Subscribers

People subscribed via source and target branches