768826 - [CLI] Custom handler reports import error

Status: RESOLVED WONTFIX

(Testing Graveyard :: Mozmill, defect)

Description

[Dave Hunt [:davehunt] [he/him] ⌚BST]

When attempting to create a custom report handler and specifying it on the command line I see import errors. Despite these errors, the custom handler appears to be working.

$ mozmill -b /Applications/FirefoxNightly.app/ -t tests/functional/testPrivateBrowsing/testAboutPrivateBrowsing.js --handler reports.py:DashboardReport --report=stdout
ERROR | /Users/dhunt/workspace/mozmill-automation-gh/mozmill-automation/reports.py:5: RuntimeWarning: Parent module '/Users/dhunt/workspace/mozmill-automation-gh/mozmill-automation/reports' not found while handling absolute import
  from mozmill.report import Report
TEST-START | tests/functional/testPrivateBrowsing/testAboutPrivateBrowsing.js | setupModule
...

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

What's the code you have used here? Would be nice if you could put up a minimized testcase. Could be fixed by bug 768820 or at least dependent on.

Comment 2

[Dave Hunt [:davehunt] [he/him] ⌚BST]

I used the same code as attached to bug 768820, but when testing it appeared that any imports in the file would trigger this error.

$ mozmill -b /Applications/FirefoxNightly.app/ -t tests/functional/testPrivateBrowsing/testAboutPrivateBrowsing.js --handler handler.py:Handler
ERROR | handler.py:1: RuntimeWarning: Parent module 'handler' not found while handling absolute import
  import sys
TEST-START | tests/functional/testPrivateBrowsing/testAboutPrivateBrowsing.js | setupModule
...

handler.py is simply:

import sys

class Handler(object):
    pass

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

So I assume that this also works now?

Comment 4

[Dave Hunt [:davehunt] [he/him] ⌚BST]

No, what makes you assume that?

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Oh wait. This is for CLI not the Mozmill class. Updating summary.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Mozmill will reach its end of life soon. We are currently working on getting all the tests for Firefox ported to Marionette. For status updates please see bug 1080766.