Merge lp:~elopio/selenium-simple-test/shared into lp:selenium-simple-test

Proposed by Leo Arias
Status: Merged
Approved by: Corey Goldberg
Approved revision: 217
Merged at revision: 222
Proposed branch: lp:~elopio/selenium-simple-test/shared
Merge into: lp:selenium-simple-test
Diff against target: 19 lines (+3/-0)
2 files modified
src/sst/runtests.py (+1/-0)
src/sst/selftests/shared/subdir/fail.py (+2/-0)
To merge this branch: bzr merge lp:~elopio/selenium-simple-test/shared
Reviewer Review Type Date Requested Status
Corey Goldberg (community) Approve
Review via email: mp+85007@code.launchpad.net

This proposal supersedes a proposal from 2011-11-22.

Commit message

Ignore subdirectories of the shared directory while looking for tests

Description of the change

Ignore subdirectories of the shared directory while looking for tests (Fixes LP: #893698)
Comes with a self test for Corey's enjoyment.

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

looks good.
self test was much enjoyed :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/sst/runtests.py'
--- src/sst/runtests.py 2011-11-21 22:29:20 +0000
+++ src/sst/runtests.py 2011-12-08 19:33:24 +0000
@@ -74,6 +74,7 @@
74 )74 )
75 for root, _, _ in os.walk(test_dir, followlinks=True)75 for root, _, _ in os.walk(test_dir, followlinks=True)
76 if os.path.abspath(root) != shared_directory and76 if os.path.abspath(root) != shared_directory and
77 not os.path.abspath(root).startswith(shared_directory+os.path.sep) and
77 not os.path.split(root)[1].startswith('_')78 not os.path.split(root)[1].startswith('_')
78 )79 )
7980
8081
=== added directory 'src/sst/selftests/shared/subdir'
=== added file 'src/sst/selftests/shared/subdir/fail.py'
--- src/sst/selftests/shared/subdir/fail.py 1970-01-01 00:00:00 +0000
+++ src/sst/selftests/shared/subdir/fail.py 2011-12-08 19:33:24 +0000
@@ -0,0 +1,2 @@
1raise AssertionError('files in a subdirectory of the shared directory '
2 'should not be run')

Subscribers

People subscribed via source and target branches