Treeherder Bug Filer Reporter
	Description • 7 years ago
treeherder

Filed by: rvandermeulen [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=78049876&repo=mozilla-aurora

https://queue.taskcluster.net/v1/task/QCmLlalESdem_n9ACbippg/runs/0/artifacts/public/logs/live_backing.log

	Henrik Skupin [:whimboo][⌚️UTC+1]
	Comment 1 • 7 years ago

Btw. this test doesn't exist on mozilla-central anymore. May the related change needs an uplift to aurora for fixing this issue?

	Andreas Tolfsen ❲:ato❳
	Comment 2 • 7 years ago

The test does exist on central as far as I can tell: http://searchfox.org/mozilla-central/source/testing/marionette/test_wait.js#67

I suspect what happens here is that the environment on Android debug (the try job RyanVM linked to above) is so slow that it doesn’t manage to evaluate the promise twice in 100 ms:

    add_task(function* test_until_interval() {
      let nevals = 0;
      yield wait.until((resolve, reject) => {
        ++nevals;
         reject();
      }, 100, 100);
      equal(2, nevals);
    });

I would be in favour of disabling this timing-dependent test on Android debug because (1) it is not a test that is imperative for debug builds, and (2) we would do other platforms a disservice by making the test slower to run.

	BMO Automation
	Comment 4 • 1 year ago

Moving bug to Testing::Marionette Client and Harness component per bug 1815831.