Merge lp:~joetalbott/ubuntu-ci-services-itself/ds-test-path into lp:ubuntu-ci-services-itself

Proposed by Joe Talbott
Status: Merged
Approved by: Chris Johnston
Approved revision: 226
Merged at revision: 226
Proposed branch: lp:~joetalbott/ubuntu-ci-services-itself/ds-test-path
Merge into: lp:ubuntu-ci-services-itself
Diff against target: 13 lines (+2/-0)
1 file modified
tests/data_store/test.py (+2/-0)
To merge this branch: bzr merge lp:~joetalbott/ubuntu-ci-services-itself/ds-test-path
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Chris Johnston (community) Approve
Review via email: mp+205642@code.launchpad.net

Commit message

data-store - Make tests runnable without having to set PYTHONPATH ahead of time.

Description of the change

data-store - Make tests runnable without having to set PYTHONPATH ahead of time.

To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:226
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/132/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/132/rebuild

review: Approve (continuous-integration)
Revision history for this message
Andy Doan (doanac) wrote :

probably okay. but we might want to make sure the PYTHONPATH is setup properly in tests/run.py so that we don't have to patch the PYTHONPATH for our testcode

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/data_store/test.py'
2--- tests/data_store/test.py 2014-02-07 13:54:16 +0000
3+++ tests/data_store/test.py 2014-02-10 19:13:19 +0000
4@@ -17,8 +17,10 @@
5 import os
6 import urllib2
7 import unittest
8+import sys
9 import yaml
10
11+sys.path.append(os.path.join(os.path.dirname(__file__), '..', '..', 'ci-utils'))
12
13 from ci_utils import data_store
14

Subscribers

People subscribed via source and target branches