u1-test-utils

Merge lp:~bloodearnest/u1-test-utils/strip-trailing-slash into lp:u1-test-utils

  1. strip-trailing-slash
  2. Merge into trunk
Proposed by Simon Davy
Status: Merged
Approved by: Simon Davy
Approved revision: 37
Merged at revision: 37
Proposed branch: lp:~bloodearnest/u1-test-utils/strip-trailing-slash
Merge into: lp:u1-test-utils
Diff against target: 13 lines (+2/-2)
1 file modified
u1testutils/sst/config.py (+2/-2)
To merge this branch: bzr merge lp:~bloodearnest/u1-test-utils/strip-trailing-slash
Reviewer Review Type Date Requested Status
Leo Arias (community) code review Approve
Review via email: mp+153179@code.launchpad.net

Commit message

strip trailing slash in sst base url

Description of the change

strip trailing slash in sst base url

To post a comment you must log in.
Revision history for this message
Leo Arias (elopio) wrote :

+1

review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'u1testutils/sst/config.py'
2--- u1testutils/sst/config.py 2011-12-22 20:09:52 +0000
3+++ u1testutils/sst/config.py 2013-03-13 16:03:21 +0000
4@@ -2,7 +2,7 @@
5 from sst.actions import set_base_url
6
7
8-def set_base_url_from_env(default_to='http://localhost:8000/'):
9+def set_base_url_from_env(default_to='http://localhost:8000'):
10 """Set the base URL for SST tests from the env or default."""
11 base_url = os.environ.get('SST_BASE_URL', default_to)
12- set_base_url(base_url)
13+ set_base_url(base_url.rstrip('/'))

Subscribers

People subscribed via source and target branches

to all changes: