Bug 1381876 - Compare window rect, not window size. r?automatedtester draft
authorAndreas Tolfsen <ato@sny.no>
Tue, 01 Aug 2017 18:21:04 +0100
changeset 641119 e595943b24788062c938daef66a0c12b97a8f842
parent 641118 0a29b248a18040afcf375d27572abead12fd93e5
child 641120 33aaa6f4d275e509a273ce4c6092104f275d6dcc
child 641126 cc266c94458f0752ee72dc0071f8bd9cc296e82a
push id72440
push userbmo:ato@sny.no
push dateSat, 05 Aug 2017 16:47:34 +0000
reviewersautomatedtester
bugs1381876
milestone57.0a1
Bug 1381876 - Compare window rect, not window size. r?automatedtester The dictionary from Marionette#window_size is a subset of what is returned from Marionette#window_rect, causing the self.assertEqual to complain that they are not equal. We want to compare the same types. MozReview-Commit-ID: L4XTlgtUYKW
testing/marionette/harness/marionette_harness/tests/unit/test_window_maximize.py
--- a/testing/marionette/harness/marionette_harness/tests/unit/test_window_maximize.py
+++ b/testing/marionette/harness/marionette_harness/tests/unit/test_window_maximize.py
@@ -55,21 +55,21 @@ class TestWindowMaximize(MarionetteTestC
         self.assertIn("x", rect)
         self.assertIn("y", rect)
         self.assertIsInstance(rect["width"], int)
         self.assertIsInstance(rect["height"], int)
         self.assertIsInstance(rect["x"], int)
         self.assertIsInstance(rect["y"], int)
 
     def test_maximize(self):
-        rect = self.marionette.maximize_window()
-        self.assert_window_rect(rect)
-        size = self.marionette.window_size
-        self.assertEqual(size, rect)
-        self.assert_window_maximized(size)
+        maximize_resp = self.marionette.maximize_window()
+        self.assert_window_rect(maximize_resp)
+        window_rect_resp = self.marionette.window_rect
+        self.assertEqual(maximize_resp, window_rect_resp)
+        self.assert_window_maximized(maximize_resp)
 
     def test_maximize_twice_restores(self):
         maximized = self.marionette.maximize_window()
         self.assert_window_maximized(maximized)
 
         restored = self.marionette.maximize_window()
         self.assert_window_restored(restored)