Merge lp:~sergiusens/music-app/test_asset_location into lp:music-app/trusty

Proposed by Sergio Schvezov
Status: Merged
Approved by: Victor Thompson
Approved revision: 204
Merged at revision: 209
Proposed branch: lp:~sergiusens/music-app/test_asset_location
Merge into: lp:music-app/trusty
Diff against target: 23 lines (+3/-4)
1 file modified
tests/autopilot/music_app/tests/__init__.py (+3/-4)
To merge this branch: bzr merge lp:~sergiusens/music-app/test_asset_location
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Victor Thompson Approve
Andy Doan (community) Approve
Review via email: mp+191228@code.launchpad.net

Commit message

Using package location to determine asset location

To post a comment you must log in.
Revision history for this message
Andy Doan (doanac) :
review: Approve
Revision history for this message
Victor Thompson (vthompson) wrote :

Appears to work straight from the branch.

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/music_app/tests/__init__.py'
2--- tests/autopilot/music_app/tests/__init__.py 2013-10-14 14:24:55 +0000
3+++ tests/autopilot/music_app/tests/__init__.py 2013-10-15 15:40:57 +0000
4@@ -15,6 +15,7 @@
5 import shutil
6 #import subprocess
7 import logging
8+import music_app
9
10 from autopilot.input import Mouse, Touch, Pointer
11 from autopilot.platform import model
12@@ -139,10 +140,8 @@
13 logger.debug("Mediascanner path set to %s" % mediascannerpath)
14
15 #set content path
16- if self.test_type == 'local' or self.test_type == 'click':
17- content_dir = os.path.join(self.working_dir, 'music_app/content/')
18- else:
19- content_dir = '/usr/lib/python2.7/dist-packages/music_app/content/'
20+ content_dir = os.path.join(os.path.dirname(music_app.__file__),
21+ 'content')
22
23 logger.debug("Content dir set to %s" % content_dir)
24

Subscribers

People subscribed via source and target branches

to status/vote changes: