Bug 1494960 - Remove wpt-reduce mach command r=ato
authorJames Graham <james@hoppipolla.co.uk>
Sat, 29 Sep 2018 14:49:25 +0000
changeset 438902 70f1b33a2dacf7798be831b35d0dec474ea5b067
parent 438901 e540269f91e25d1735c619facf0af77115a8abb9
child 438903 ace8fc8c112ec192ae853157875f1be6ea43a755
push id34741
push userccoroiu@mozilla.com
push dateSun, 30 Sep 2018 09:45:27 +0000
treeherdermozilla-central@c3d7f8a2a6d4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersato
bugs1494960
milestone64.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 1494960 - Remove wpt-reduce mach command r=ato As far as I know this was unused and hasn't been well maintained since it's not that useful. Differential Revision: https://phabricator.services.mozilla.com/D7171
testing/web-platform/mach_commands.py
--- a/testing/web-platform/mach_commands.py
+++ b/testing/web-platform/mach_commands.py
@@ -147,34 +147,16 @@ class WebPlatformTestsUpdater(MozbuildOb
             update.run_update(logger, **kwargs)
         except Exception:
             import pdb
             import traceback
             traceback.print_exc()
 #            pdb.post_mortem()
 
 
-class WebPlatformTestsReduce(WebPlatformTestsRunner):
-
-    def run_reduce(self, **kwargs):
-        from wptrunner import reduce
-
-        self.setup_kwargs(kwargs)
-
-        kwargs["capture_stdio"] = True
-        logger = reduce.setup_logging(kwargs, {"mach": sys.stdout})
-        tests = reduce.do_reduce(**kwargs)
-
-        if not tests:
-            logger.warning("Test was not unstable")
-
-        for item in tests:
-            logger.info(item.id)
-
-
 class WebPlatformTestsCreator(MozbuildObject):
     template_prefix = """<!doctype html>
 %(documentElement)s<meta charset=utf-8>
 """
     template_long_timeout = "<meta name=timeout content=long>\n"
 
     template_body_th = """<title></title>
 <script src=/resources/testharness.js></script>
@@ -309,21 +291,16 @@ class WPTManifestDownloader(MozbuildObje
         manifestdownload.run(wpt_dir, self.topsrcdir, logger, force, manifest_update)
 
 
 def create_parser_update():
     from update import updatecommandline
     return updatecommandline.create_parser()
 
 
-def create_parser_reduce():
-    from wptrunner import wptcommandline
-    return wptcommandline.create_parser_reduce()
-
-
 def create_parser_create():
     import argparse
     p = argparse.ArgumentParser()
     p.add_argument("--no-editor", action="store_true",
                    help="Don't try to open the test in an editor")
     p.add_argument("-e", "--editor", action="store", help="Editor to use")
     p.add_argument("--long-timeout", action="store_true",
                    help="Test should be given a long timeout (typically 60s rather than 10s, but varies depending on environment)")
@@ -395,32 +372,16 @@ class MachCommands(MachCommandBase):
         return wpt_updater.run_update(**params)
 
     @Command("wpt-update",
              category="testing",
              parser=create_parser_update)
     def update_wpt(self, **params):
         return self.update_web_platform_tests(**params)
 
-    @Command("web-platform-tests-reduce",
-             category="testing",
-             conditions=[conditions.is_firefox],
-             parser=create_parser_reduce)
-    def unstable_web_platform_tests(self, **params):
-        self.setup()
-        wpt_reduce = self._spawn(WebPlatformTestsReduce)
-        return wpt_reduce.run_reduce(**params)
-
-    @Command("wpt-reduce",
-             category="testing",
-             conditions=[conditions.is_firefox],
-             parser=create_parser_reduce)
-    def unstable_wpt(self, **params):
-        return self.unstable_web_platform_tests(**params)
-
     @Command("web-platform-tests-create",
              category="testing",
              parser=create_parser_create)
     def create_web_platform_test(self, **params):
         self.setup()
         wpt_creator = self._spawn(WebPlatformTestsCreator)
         wpt_creator.run_create(self._mach_context, **params)