Merge lp:~elopio/ubuntuone-testing/fix-1046476 into lp:ubuntuone-testing

[Leo Arias]

Fix the files URLs

[Vincent Ladeuil (vila)]

weird, why didn't get it caught earlier ? Is this code covered in tests ? If not, it may be worth adding a test. If yes, some test needs to be fixed no ?

I'm ok to land this with either a new test or a fixed one ;)

review: Needs Fixing

[Leo Arias (elopio)]

The files tests will fail because of this. But we didn't notice because jenkins was out of service. We might add a test just to open the main page, and hardcode the expected value. That's a good idea. I'll open a bug for that.

[Leo Arias (elopio)]

review: Approve

[Ubuntu One Auto Pilot (otto-pilot)]

Voting does not meet specified criteria. Required: Approve >= 1, Disapprove == 0, Needs Fixing == 0, Needs Information == 0, Resubmit == 0, Pending == 0. Got: 1 Approve, 1 Needs Fixing.

[Vincent Ladeuil (vila)]

Looks like I need to vote approve now to unblock this ?

review: Approve

=== modified file 'ubuntuone/web/tests/sst/shared/data/environment.py'
--- ubuntuone/web/tests/sst/shared/data/environment.py	2012-08-14 16:17:45 +0000
+++ ubuntuone/web/tests/sst/shared/data/environment.py	2012-09-05 18:41:21 +0000
@@ -20,9 +20,9 @@
 sso_urls = {'staging': 'https://loging.staging.ubuntu.com/',
             'edge': 'https://loging.ubuntu.com/',
             'production': 'https://loging.ubuntu.com/'}
-files_urls = {'staging': 'https://staging.one.ubuntu.com/',
-              'edge': 'https://edge.one.ubuntu.com/',
-              'production': 'https://one.ubuntu.com/'}
+files_urls = {'staging': 'https://files.staging.one.ubuntu.com/',
+              'edge': 'https://files.one.ubuntu.com/',
+              'production': 'https://files.one.ubuntu.com/'}
 
 class Environment: