diff options
author | Corey Farwell <coreyf@rwell.org> | 2014-11-22 17:10:22 -0500 |
---|---|---|
committer | Corey Farwell <coreyf@rwell.org> | 2014-11-23 13:13:53 -0500 |
commit | cb38675f4822ae6ffd500d99f82dd4b7c591c329 (patch) | |
tree | 321f91dd95461feb26ebf8bf31d0eb85688a39c3 | |
parent | e38e20da6500184df19654abe34322dd3c2fd99b (diff) | |
download | servo-cb38675f4822ae6ffd500d99f82dd4b7c591c329.tar.gz servo-cb38675f4822ae6ffd500d99f82dd4b7c591c329.zip |
Make sure directories aren't getting tested
-rw-r--r-- | python/servo/testing_commands.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/python/servo/testing_commands.py b/python/servo/testing_commands.py index 125d841f174..9498f1beb2c 100644 --- a/python/servo/testing_commands.py +++ b/python/servo/testing_commands.py @@ -30,12 +30,12 @@ class MachCommands(CommandBase): self.context.built_tests = True def find_test(self, prefix): - candidates = [ - f for f in os.listdir(path.join(self.context.topdir, "target")) - if f.startswith(prefix + "-")] - if candidates: - return path.join(self.context.topdir, "target", candidates[0]) - return None + target_contents = os.listdir(path.join(self.context.topdir, "target")) + for filename in target_contents: + if filename.startswith(prefix + "-"): + filepath = path.join(self.context.topdir, "target", filename) + if path.isfile(filepath) and os.access(filepath, os.X_OK): + return filepath def run_test(self, prefix, args=[]): t = self.find_test(prefix) |