Remove B2G related tests in dom/events/test/marionette/




DOM: Events
2 months ago
2 months ago


(Reporter: ahal, Unassigned)


(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)




2 months ago
+++ This bug was initially created as a clone of Bug #1422006 +++

Running |mach test marionette| results in:
Exception: Test file names must be of the form ''. Invalid test names:

  File "/home/ahal/hg/mozilla-central/testing/", line 128, in test
    argv=extra_args, test_objects=tests, **kwargs)
  File "/home/ahal/hg/mozilla-central/python/mach/mach/", line 129, in dispatch
    return self._run_command_handler(handler, context=context, **kwargs)
  File "/home/ahal/hg/mozilla-central/python/mach/mach/", line 90, in _run_command_handler
    result = fn(**kwargs)
  File "/home/ahal/hg/mozilla-central/testing/marionette/", line 93, in run_marionette_test
    return run_marionette(tests, topsrcdir=self.topsrcdir, **kwargs)
  File "/home/ahal/hg/mozilla-central/testing/marionette/", line 63, in run_marionette
    failed = MarionetteHarness(MarionetteTestRunner, args=vars(args)).run()
  File "/home/ahal/hg/mozilla-central/testing/marionette/harness/marionette_harness/", line 72, in run
  File "/home/ahal/hg/mozilla-central/testing/marionette/harness/marionette_harness/runner/", line 850, in run_tests
  File "/home/ahal/hg/mozilla-central/testing/marionette/harness/marionette_harness/runner/", line 801, in _add_tests
    " Invalid test names:\n  {}".format('\n  '.join(invalid_tests)))

We should fix this and make sure marionette tests also get picked up when specifying a path.
Those are js files which were most likely related to B2G. Marionette does not have a JS runner anymore. So how are those tests been run? Is there a Python test module as entry point?
Flags: needinfo?(ahalberstadt)

Comment 2

2 months ago
Looks like |mach test| is dispatching to |mach marionette-test|. I'm not really sure why this is happening, maybe the correct fix is to remove those tests? Maybe the test resolving logic needs to be updated?

Ftr, I'm filing all the issues I can find with |mach test| and haven't looked into this particular one in much detail.
Flags: needinfo?(ahalberstadt)
I checked and those are indeed B2G tests. The DOM team should care about removing those tests.
Component: Marionette → DOM: Events
Product: Testing → Core
Summary: Error running |mach test marionette| → Remove B2G related tests in dom/events/test/marionette/


2 months ago
Priority: -- → P2
You need to log in before you can comment on or make changes to this bug.