--- a/testing/marionette/harness/marionette_harness/tests/unit/test_quit_restart.py
+++ b/testing/marionette/harness/marionette_harness/tests/unit/test_quit_restart.py
@@ -1,19 +1,22 @@
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
from __future__ import absolute_import, print_function
+import sys
import urllib
+from unittest import skip, skipIf
+
from marionette_driver import errors
from marionette_driver.by import By
-from marionette_harness import MarionetteTestCase, skip
+from marionette_harness import MarionetteTestCase
def inline(doc):
return "data:text/html;charset=utf-8,{}".format(urllib.quote(doc))
class TestServerQuitApplication(MarionetteTestCase):
@@ -154,50 +157,41 @@ class TestQuitRestart(MarionetteTestCase
self.marionette.get_url()
self.marionette.start_session()
self.assertEqual(self.marionette.profile, self.profile)
self.assertNotEqual(self.marionette.session_id, self.session_id)
self.assertNotEqual(self.marionette.get_pref("startup.homepage_welcome_url"),
"about:about")
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_no_in_app_clean_restart(self):
# Test that in_app and clean cannot be used in combination
with self.assertRaises(ValueError):
self.marionette.restart(in_app=True, clean=True)
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_in_app_restart(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
self.marionette.restart(in_app=True)
self.assertEqual(self.marionette.profile, self.profile)
- self.assertEqual(self.marionette.session_id, self.session_id)
+ self.assertNotEqual(self.marionette.session_id, self.session_id)
# An in-app restart will keep the same process id only on Linux
if self.marionette.session_capabilities["platformName"] == "linux":
self.assertEqual(self.marionette.process_id, self.pid)
else:
self.assertNotEqual(self.marionette.process_id, self.pid)
self.assertNotEqual(self.marionette.get_pref("startup.homepage_welcome_url"),
"about:about")
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_in_app_restart_with_callback(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
self.marionette.restart(in_app=True,
callback=lambda: self.shutdown(restart=True))
self.assertEqual(self.marionette.profile, self.profile)
- self.assertEqual(self.marionette.session_id, self.session_id)
+ self.assertNotEqual(self.marionette.session_id, self.session_id)
# An in-app restart will keep the same process id only on Linux
if self.marionette.session_capabilities["platformName"] == "linux":
self.assertEqual(self.marionette.process_id, self.pid)
else:
self.assertNotEqual(self.marionette.process_id, self.pid)
self.assertNotEqual(self.marionette.get_pref("startup.homepage_welcome_url"),
@@ -227,51 +221,45 @@ class TestQuitRestart(MarionetteTestCase
self.assertTrue(self.is_safe_mode, "Safe Mode is not enabled")
finally:
self.marionette.quit(clean=True)
def test_in_app_restart_with_callback_not_callable(self):
with self.assertRaisesRegexp(ValueError, "is not callable"):
self.marionette.restart(in_app=True, callback=4)
+ @skipIf(sys.platform.startswith("win"),
+ "Bug 1433873 - Fix race condition in Marionette for in_app quit and restart")
def test_in_app_restart_with_callback_missing_shutdown(self):
try:
timeout_startup = self.marionette.DEFAULT_STARTUP_TIMEOUT
timeout_shutdown = self.marionette.DEFAULT_SHUTDOWN_TIMEOUT
self.marionette.DEFAULT_SHUTDOWN_TIMEOUT = 5
self.marionette.DEFAULT_STARTUP_TIMEOUT = 5
with self.assertRaisesRegexp(IOError, "the connection to Marionette server is lost"):
self.marionette.restart(in_app=True, callback=lambda: False)
finally:
self.marionette.DEFAULT_STARTUP_TIMEOUT = timeout_startup
self.marionette.DEFAULT_SHUTDOWN_TIMEOUT = timeout_shutdown
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_in_app_quit(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
self.marionette.quit(in_app=True)
self.assertEqual(self.marionette.session, None)
with self.assertRaisesRegexp(errors.MarionetteException, "Please start a session"):
self.marionette.get_url()
self.marionette.start_session()
self.assertEqual(self.marionette.profile, self.profile)
self.assertNotEqual(self.marionette.session_id, self.session_id)
self.assertNotEqual(self.marionette.get_pref("startup.homepage_welcome_url"),
"about:about")
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_in_app_quit_with_callback(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
self.marionette.quit(in_app=True, callback=self.shutdown)
self.assertEqual(self.marionette.session, None)
with self.assertRaisesRegexp(errors.MarionetteException, "Please start a session"):
self.marionette.get_url()
self.marionette.start_session()
self.assertEqual(self.marionette.profile, self.profile)
self.assertNotEqual(self.marionette.session_id, self.session_id)
@@ -301,55 +289,43 @@ class TestQuitRestart(MarionetteTestCase
});
</script>
"""))
self.marionette.find_element(By.TAG_NAME, "input").send_keys("foo")
self.marionette.quit(in_app=True)
self.marionette.start_session()
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_reset_context_after_quit_by_set_context(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
# Check that we are in content context which is used by default in
# Marionette
self.assertNotIn("chrome://", self.marionette.get_url(),
"Context does not default to content")
self.marionette.set_context("chrome")
self.marionette.quit(in_app=True)
self.assertEqual(self.marionette.session, None)
self.marionette.start_session()
self.assertNotIn("chrome://", self.marionette.get_url(),
"Not in content context after quit with using_context")
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_reset_context_after_quit_by_using_context(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
# Check that we are in content context which is used by default in
# Marionette
self.assertNotIn("chrome://", self.marionette.get_url(),
"Context does not default to content")
with self.marionette.using_context("chrome"):
self.marionette.quit(in_app=True)
self.assertEqual(self.marionette.session, None)
self.marionette.start_session()
self.assertNotIn("chrome://", self.marionette.get_url(),
"Not in content context after quit with using_context")
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_keep_context_after_restart_by_set_context(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
# Check that we are in content context which is used by default in
# Marionette
self.assertNotIn("chrome://", self.marionette.get_url(),
"Context doesn't default to content")
# restart while we are in chrome context
self.marionette.set_context("chrome")
self.marionette.restart(in_app=True)
@@ -358,21 +334,17 @@ class TestQuitRestart(MarionetteTestCase
if self.marionette.session_capabilities["platformName"] == "linux":
self.assertEqual(self.marionette.process_id, self.pid)
else:
self.assertNotEqual(self.marionette.process_id, self.pid)
self.assertIn("chrome://", self.marionette.get_url(),
"Not in chrome context after a restart with set_context")
- @skip("Bug 1363368 - Wrong window handles after in_app restarts")
def test_keep_context_after_restart_by_using_context(self):
- if self.marionette.session_capabilities["platformName"] != "windows_nt":
- skip("Bug 1363368 - Wrong window handles after in_app restarts")
-
# Check that we are in content context which is used by default in
# Marionette
self.assertNotIn("chrome://", self.marionette.get_url(),
"Context does not default to content")
# restart while we are in chrome context
with self.marionette.using_context('chrome'):
self.marionette.restart(in_app=True)