Bug 1726460 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

This bug will track adding automated testing for Silent Restarts (Bug 1726239).

I am currently planning to land the Silent Restart feature without automated testing, because I don't believe that any of our testing frameworks can properly accommodate its testing needs. AFAIK, most of them are unable to tolerate Firefox restarting in the middle of a test. The exception is marionette, which makes it the obvious choice for this. Unfortunately, as things stand, marionette cannot connect to Firefox in its Silent Restart state because Firefox's marionette server doesn't start until a window opens.

I tried adding [these observer notifications](https://searchfox.org/mozilla-central/rev/d3683dbb252506400c71256ef3994cdbdfb71ada/browser/components/BrowserGlue.jsm#2599-2600) to the Silent Restart command line handler. Unfortunately, even then, the connection does not work. I started getting errors coming from Firefox' marionette server's [newSession](https://searchfox.org/mozilla-central/rev/d3683dbb252506400c71256ef3994cdbdfb71ada/remote/marionette/driver.js#404) function. I tried passing some parameters through to that function to prevent it from attempting to access windows, but wasn't enough either.

At that point, I decided that the scale of changes that it would be necessary to make to marionette were outside the scope of this project. So I'm filing this bug to hopefully get tests added at a later time. I'm attaching the test that I wrote before the scope of the necessary marionette changes became clear.
This bug will track adding automated testing for Silent Restarts (Bug 1726239).

I am currently planning to land the Silent Restart feature without automated testing, because I don't believe that any of our testing frameworks can properly accommodate its testing needs. AFAIK, most of them are unable to tolerate Firefox restarting in the middle of a test. The exception is marionette, which makes it the obvious choice for this. Unfortunately, as things stand, marionette cannot connect to Firefox in its Silent Restart state because Firefox's marionette server doesn't start until a window opens.

I tried adding [these observer notifications](https://searchfox.org/mozilla-central/rev/d3683dbb252506400c71256ef3994cdbdfb71ada/browser/components/BrowserGlue.jsm#2599-2600) to the Silent Restart command line handler. Unfortunately, even then, the connection does not work. I started getting errors coming from Firefox's marionette server's [newSession](https://searchfox.org/mozilla-central/rev/d3683dbb252506400c71256ef3994cdbdfb71ada/remote/marionette/driver.js#404) function. I tried passing some parameters through to that function to prevent it from attempting to access windows, but wasn't enough either.

At that point, I decided that the scale of changes that it would be necessary to make to marionette were outside the scope of this project. So I'm filing this bug to hopefully get tests added at a later time. I'm attaching the test that I wrote before the scope of the necessary marionette changes became clear.

Back to Bug 1726460 Comment 0