Bug 1453009 - Remove collection tests from Marionette. r?whimboo draft
authorAndreas Tolfsen <ato@sny.no>
Tue, 10 Apr 2018 19:31:25 +0100
changeset 781206 084f9383686066a834b95d95f04b55d58a2f518d
parent 781205 db8d479a5e77be28e7bc183ec1f0cc104d46d374
push id106246
push userbmo:ato@sny.no
push dateThu, 12 Apr 2018 18:42:24 +0000
reviewerswhimboo
bugs1453009
milestone61.0a1
Bug 1453009 - Remove collection tests from Marionette. r?whimboo Now that we test collections upstream in WPT, the overlapping tests can be removed from the Mn test suite. MozReview-Commit-ID: DJZHeKs8FDo
testing/marionette/harness/marionette_harness/tests/unit/test_execute_script.py
--- a/testing/marionette/harness/marionette_harness/tests/unit/test_execute_script.py
+++ b/testing/marionette/harness/marionette_harness/tests/unit/test_execute_script.py
@@ -420,58 +420,8 @@ class TestExecuteChrome(WindowManagerMix
     def test_system_sandbox_wrappedjsobject(self):
         pass
 
     def test_access_chrome_objects_in_event_listeners(self):
         pass
 
     def test_return_value_on_alert(self):
         pass
-
-
-class TestElementCollections(MarionetteTestCase):
-
-    def assertSequenceIsInstance(self, seq, typ):
-        for item in seq:
-            self.assertIsInstance(item, typ)
-
-    def test_array(self):
-        self.marionette.navigate(inline("<p>foo <p>bar"))
-        els = self.marionette.execute_script("return Array.from(document.querySelectorAll('p'))")
-        self.assertIsInstance(els, list)
-        self.assertEqual(2, len(els))
-        self.assertSequenceIsInstance(els, HTMLElement)
-
-    def test_html_all_collection(self):
-        self.marionette.navigate(inline("<p>foo <p>bar"))
-        els = self.marionette.execute_script("return document.all")
-        self.assertIsInstance(els, list)
-        # <html>, <head>, <body>, <p>, <p>
-        self.assertEqual(5, len(els))
-        self.assertSequenceIsInstance(els, HTMLElement)
-
-    def test_html_collection(self):
-        self.marionette.navigate(inline("<p>foo <p>bar"))
-        els = self.marionette.execute_script("return document.getElementsByTagName('p')")
-        self.assertIsInstance(els, list)
-        self.assertEqual(2, len(els))
-        self.assertSequenceIsInstance(els, HTMLElement)
-
-    def test_html_form_controls_collection(self):
-        self.marionette.navigate(inline("<form><input><input></form>"))
-        els = self.marionette.execute_script("return document.forms[0].elements")
-        self.assertIsInstance(els, list)
-        self.assertEqual(2, len(els))
-        self.assertSequenceIsInstance(els, HTMLElement)
-
-    def test_html_options_collection(self):
-        self.marionette.navigate(inline("<select><option><option></select>"))
-        els = self.marionette.execute_script("return document.querySelector('select').options")
-        self.assertIsInstance(els, list)
-        self.assertEqual(2, len(els))
-        self.assertSequenceIsInstance(els, HTMLElement)
-
-    def test_node_list(self):
-        self.marionette.navigate(inline("<p>foo <p>bar"))
-        els = self.marionette.execute_script("return document.querySelectorAll('p')")
-        self.assertIsInstance(els, list)
-        self.assertEqual(2, len(els))
-        self.assertSequenceIsInstance(els, HTMLElement)