Bug 1319237 - Improve window position test assertions. r=maja_zf, a=test-only
authorAndreas Tolfsen <ato@mozilla.com>
Thu, 02 Feb 2017 14:57:19 +0000
changeset 378507 7310c81675f4e2127f1231c1367c95a7ae012d7d
parent 378506 fc404ef132629e39963e7cf0dd2049fb7e4b4b5d
child 378508 3df0618b87a35c823e6b1e973991ac4dffd7e5e2
push id1419
push userjlund@mozilla.com
push dateMon, 10 Apr 2017 20:44:07 +0000
treeherdermozilla-release@5e6801b73ef6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmaja_zf, test-only
bugs1319237
milestone53.0a2
Bug 1319237 - Improve window position test assertions. r=maja_zf, a=test-only MozReview-Commit-ID: qmFS0JEqx3
testing/marionette/harness/marionette_harness/tests/unit/test_window_position.py
--- a/testing/marionette/harness/marionette_harness/tests/unit/test_window_position.py
+++ b/testing/marionette/harness/marionette_harness/tests/unit/test_window_position.py
@@ -5,28 +5,29 @@
 from marionette_driver.errors import InvalidArgumentException
 
 from marionette_harness import MarionetteTestCase
 
 
 class TestWindowPosition(MarionetteTestCase):
     def test_get_types(self):
         position = self.marionette.get_window_position()
-        self.assertTrue(isinstance(position["x"], int))
-        self.assertTrue(isinstance(position["y"], int))
+        self.assertIsInstance(position["x"], int)
+        self.assertIsInstance(position["y"], int)
 
     def test_set_types(self):
         for x, y in (["a", "b"], [1.2, 3.4], [True, False], [[], []], [{}, {}]):
+            print("testing invalid type position ({},{})".format(x, y))
             with self.assertRaises(InvalidArgumentException):
                 self.marionette.set_window_position(x, y)
 
     def test_out_of_bounds_arguments(self):
-        with self.assertRaises(InvalidArgumentException):
-            self.marionette.set_window_position(-1, 0)
-        with self.assertRaises(InvalidArgumentException):
-            self.marionette.set_window_position(0, -1)
+        for x, y in [(-1,0), (0,-1), (-1,-1)]:
+            print("testing out of bounds position ({},{})".format(x, y))
+            with self.assertRaises(InvalidArgumentException):
+                self.marionette.set_window_position(x, y)
 
-    def test_move(self):
+    def test_move_to_new_position(self):
         old_position = self.marionette.get_window_position()
         new_position = {"x": old_position["x"] + 10, "y": old_position["y"] + 10}
         self.marionette.set_window_position(new_position["x"], new_position["y"])
-        self.assertNotEqual(old_position['x'], new_position["x"])
-        self.assertNotEqual(old_position['y'], new_position["y"])
+        self.assertNotEqual(old_position["x"], new_position["x"])
+        self.assertNotEqual(old_position["y"], new_position["y"])