Bug 1285085: Testing for a windows 4x larger than the screen in each dimension should be enough for this test. r?whimboo draft
authorMilan Sreckovic <milan@mozilla.com>
Fri, 09 Sep 2016 12:42:34 -0400
changeset 412253 57f87610b47593640a1335450f1ff70dec62d469
parent 410784 4ee5ddeeee281b9b6f564fc56efb1713da8d7eac
child 530910 32dbd979f767f479d3109539bd7fbbb0c603b6eb
push id29092
push usermsreckovic@mozilla.com
push dateFri, 09 Sep 2016 16:43:06 +0000
reviewerswhimboo
bugs1285085
milestone51.0a1
Bug 1285085: Testing for a windows 4x larger than the screen in each dimension should be enough for this test. r?whimboo MozReview-Commit-ID: 4r6N0Xkrm6j
testing/marionette/harness/marionette/tests/unit/test_set_window_size.py
--- a/testing/marionette/harness/marionette/tests/unit/test_set_window_size.py
+++ b/testing/marionette/harness/marionette/tests/unit/test_set_window_size.py
@@ -50,17 +50,17 @@ class TestSetWindowSize(MarionetteTestCa
         size = self.marionette.set_window_size(width, height)
         self.assertIsNotNone(size, "Response is None")
         self.assertEqual(size['width'], width,
                          "New width is %s but should be %s" % (size['width'], width))
         self.assertEqual(size['height'], height,
                          "New height is %s but should be %s" % (size['height'], height))
 
     def test_possible_to_request_window_larger_than_screen(self):
-        self.marionette.set_window_size(100000, 100000)
+        self.marionette.set_window_size(4 * self.max_width, 4 * self.max_height)
         size = self.marionette.window_size
 
         # In X the window size may be greater than the bounds of the screen
         self.assertGreaterEqual(size["width"], self.max_width)
         self.assertGreaterEqual(size["height"], self.max_height)
 
     def test_that_we_can_maximise_the_window(self):
         # valid size