634980 - Mozmill fails oddly when things are not ordinary files or directories

Status: RESOLVED FIXED

(Testing Graveyard :: Mozmill, defect)

Description

[Jeff Hammel]

(mozmill)│mozmill -t <(echo 'hello world')
INFO Passed: 0
INFO Failed: 0
INFO Skipped: 0
Traceback (most recent call last):
  File
  "/home/jhammel/mozmill/src/mozmill/mozmill/mozmill/__init__.py",
  line 586, in run
    self.run_tests(MozMill, normal_tests, runner, results)
  File
  "/home/jhammel/mozmill/src/mozmill/mozmill/mozmill/__init__.py",
  line 556, in run_tests
    mozmill.run(tests)
  File
  "/home/jhammel/mozmill/src/mozmill/mozmill/mozmill/__init__.py",
  line 262, in run
    self.run_tests(tests)
  File
  "/home/jhammel/mozmill/src/mozmill/mozmill/mozmill/__init__.py",
  line 232, in run_tests
    tests += self.collect_tests(path)
  File
  "/home/jhammel/mozmill/src/mozmill/mozmill/mozmill/__init__.py",
  line 248, in collect_tests
    for filename in sorted(os.listdir(path)):
OSError: [Errno 20] Not a directory: '/dev/fd/63'

This is neither a file nor a directory in python's mind:

import os
import sys
f = sys.argv[1]
print os.path.isfile(f)
print os.path.isdir(f)

│python tmp.py <(echo 'hello world')
False
False

It's highly likely (albeit unknown to me) whether we can run these
things or not:

diff --git a/mozmill/mozmill/__init__.py b/mozmill/mozmill/__init__.py
index 818c9b4..8342401 100644
--- a/mozmill/mozmill/__init__.py
+++ b/mozmill/mozmill/__init__.py
@@ -241,7 +241,7 @@ class MozMill(object):
     def collect_tests(self, path):
         """find all tests for a given path"""
         
-        if os.path.isfile(path):
+        if not os.path.isdir(path):
           return [path]
 
         files = []

Running again:

(mozmill)│mozmill -t <(echo 'hello world')
INFO Passed: 0
INFO Failed: 0
INFO Skipped: 0

Not helpful :(

Maybe if its not `os.path.isfile` we should assert its a directory?
Other ideas?

Comment 1

[cmtalbert]

(In reply to comment #0)

> Maybe if its not `os.path.isfile` we should assert its a directory?
> Other ideas?

Why not assert if what we're given is not a file and not a directory and throw an error?  (Command line validation style, so we don't even try to run the tests?)

Let's clean this up in 2.0

Comment 2

[Jeff Hammel]

Yeah, that seems fine

Comment 3

[Jeff Hammel]

Attachment: Mozmill fails oddly when things are not ordinary files or directories

Comment 4

[Andrew Halberstadt [:ahal]]

Comment on attachment 521610 [details] [diff] [review]
Mozmill fails oddly when things are not ordinary files or directories

Sorry, have to r- as this breaks symbolic links.

Comment 5

[Jeff Hammel]

(In reply to comment #4)
> Comment on attachment 521610 [details] [diff] [review] [review]
> Mozmill fails oddly when things are not ordinary files or directories
> 
> Sorry, have to r- as this breaks symbolic links.

Links to files or links to directories or both?  Should be easy enough to account for, in theory

Comment 6

[Jeff Hammel]

It would be nice to have a test for this.

Comment 7

[Jeff Hammel]

Attachment: this should respect symbolic links. does this solve the problem?

Comment 8

[Andrew Halberstadt [:ahal]]

Comment on attachment 535671 [details] [diff] [review]
this should respect symbolic links. does this solve the problem?

wfm

Comment 9

[Jeff Hammel]

pushed to master: https://github.com/mozautomation/mozmill/commit/4536279cbd36cc1fa0b6c5f81ed8d231dba1b734

If this causes any downstream problems, we should fix.  Should be identical for non-symbolic links, so I think it shouldn't cause any problems.  If it does, we can probably easily fix.