--- 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,17 +1,17 @@
# 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/.
import unittest
from marionette_driver import errors
-from marionette_harness import MarionetteTestCase, skip
+from marionette_harness import MarionetteTestCase
class TestServerQuitApplication(MarionetteTestCase):
def tearDown(self):
if self.marionette.session is None:
self.marionette.start_session()
@@ -148,50 +148,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:")
- @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:")
- @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"),
@@ -231,38 +222,30 @@ class TestQuitRestart(MarionetteTestCase
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:")
- @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)
@@ -274,55 +257,43 @@ class TestQuitRestart(MarionetteTestCase
timeout = self.marionette.DEFAULT_SHUTDOWN_TIMEOUT
self.marionette.DEFAULT_SHUTDOWN_TIMEOUT = 10
with self.assertRaisesRegexp(IOError, "a requested application quit did not happen"):
self.marionette.quit(in_app=True, callback=lambda: False)
finally:
self.marionette.DEFAULT_SHUTDOWN_TIMEOUT = timeout
- @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)
@@ -331,21 +302,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)