698898 - Talos should use mozrunner

Status: RESOLVED WONTFIX

(Testing :: Talos, defect)

Description

[William Lachance (:wlach)]

Subject says it all. We currently use a fragile abstraction called "bcontroller.py", run as a seperate project for this, but it would simplify things immensely to standardize on the proven mozrunner abstraction for this functionality.

Comment 1

[William Lachance (:wlach)]

Attachment: Work in progress on making talos use mozrunner

Here's a patch which tries to make this happen. As it says, it only really works on the mobile case (and I just tested tpan). It doesn't work at all for the desktop case ATM -- some of the code is there, but for some reason writing to the log file isn't working in that case. It's possible this code just needs a re-think anyway, as our MozRunner interface probably shouldn't fundamentally rely on providing an interface to capturing standard output (this isn't even possible on Fennec!)

Comment 2

[William Lachance (:wlach)]

Note for future: My patch here is badly out of date and much of the discussion here is irrelevant. It's unclear to me whether we actually do want to use mozrunner for talos. It would likely be a great deal of work.

Comment 3

[Julien Pagès (:parkouss)]

I don't think talos would need mozrunner now that we are going to drop android support.

I think in Talos case mozprocess is sufficient. Please reopen the bug if you don't agree with that!