584413 - [manifests] Support sub-level directories in mozmill-restart tests

Status: RESOLVED WONTFIX

(Testing Graveyard :: Mozmill, defect)

Description

[Aaron Train [:aaronmt]]

Currently, Mozmill will hang when running mozmill-restart tests that are within a sub-level directory, i.e., (restartTests/softwareUpdate/testXMLErrorAutoProxy/)

As per a comment in a separate bug, I stumbled upon this while trying to understand why Mozmill was hanging. 

http://github.com/mozautomation/mozmill/blob/master/mozmill/mozmill/__init__.py#L640

Is this desirable? Undesirable?

Comment 1

[cmtalbert]

Side effect of the hacky implementation of the restart mechanism.  It will be fixed for 2.0 when we move to a more sane way of loading tests and modules.

Thanks for tracking the issue.  Can I get you to add in the exact command line you launched with so that we can reliably reproduce this test case once we make those changes and ensure we've fixed this?

Comment 2

[Aaron Train [:aaronmt]]

Sure. To currently reproduce, just dive into firefox/restartTests/testExtensionInstallUninstall

Create a folder: 'extensions'. Move the three test files into that folder.

With the testExtensionInstallUninstall as the present working directory

My command line ran: mozmill-restart -t extensions/ -b ~/Downloads/Minefield.app/ --show-errors

Comment 3

[cmtalbert]

We are moving to manifests for all complicated restart tests.  We will continue to support simple directory lists, but we're going to devote our effort to extending our capabilities on the manifest side of things not on the directory side of things.

--> WONTFIX

Comment 4

[Jeff Hammel]

Actually this will also be fixed when bug 584464  is landed anyway. In that patch, normal and restart tests are found identically