Merge lp:~zematynnad/selenium-simple-test/fix_symlinks_850054 into lp:selenium-simple-test

Proposed by Danny Tamez
Status: Merged
Approved by: Corey Goldberg
Approved revision: 181
Merged at revision: 181
Proposed branch: lp:~zematynnad/selenium-simple-test/fix_symlinks_850054
Merge into: lp:selenium-simple-test
Diff against target: 12 lines (+1/-1)
1 file modified
src/sst/runtests.py (+1/-1)
To merge this branch: bzr merge lp:~zematynnad/selenium-simple-test/fix_symlinks_850054
Reviewer Review Type Date Requested Status
Corey Goldberg (community) Approve
Review via email: mp+75363@code.launchpad.net

Commit message

Simply changes the os.walk to also followlinks.

Description of the change

Simply changes the os.walk to also followlinks.

To post a comment you must log in.
Revision history for this message
Corey Goldberg (coreygoldberg) wrote :

nice.

a directory of symlinks pointing to branches in project dirs is a neat way to have SST find tests.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/sst/runtests.py'
2--- src/sst/runtests.py 2011-08-31 13:55:34 +0000
3+++ src/sst/runtests.py 2011-09-14 15:10:30 +0000
4@@ -72,7 +72,7 @@
5 browser_platform, session_name, javascript_disabled,
6 webdriver_remote_url, screenshots_on, found_tests, failfast, debug
7 )
8- for root, _, _ in os.walk(test_dir)
9+ for root, _, _ in os.walk(test_dir, followlinks=True)
10 if os.path.abspath(root) != shared_directory and
11 not os.path.split(root)[1].startswith('_')
12 )

Subscribers

People subscribed via source and target branches