Bug 1453009 - Remove collection tests from Marionette. r=whimboo
authorAndreas Tolfsen <ato@sny.no>
Tue, 10 Apr 2018 19:31:25 +0100
changeset 413071 d87d4e86f054614a6cad43c948f5484f9ecd0b1b
parent 413070 172094bca9a6e2f841e629aae292647e212d92c6
child 413072 d0392a55774becf075a7e20ecfe852a497e96644
push id33833
push useraiakab@mozilla.com
push dateFri, 13 Apr 2018 09:41:15 +0000
treeherdermozilla-central@260e4c83c8a9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerswhimboo
bugs1453009
milestone61.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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)