CancelError exception thrown by robot

[Douglas Hays]

Run dijit/tests/robot/typematic.html (I tried both Chrome 20 and IE9).
After the first test, the robot calls seqPromise.cancel() which causes a CancelError? exception, although all tests pass.

[haysmark]

I tried duplicating the first test and still got only one error. We only started noticing it now because of the new promise instrumentation.

[bill]

This is due to the code I added in [29101], and is actually happening on every test. I don't know why it only displays the error the first time.

[29101] changed robot to use a chain of promises to execute all the "commands" registered via sequence().

It's actually now redundant for the typematic test and other tests to create and return their own Deferred object. It would be better if they just returned the seqPromise promise chain. However, they create their own Deferred for historical reasons, and that's leading to tricky timing between when the tests' Deferred resolves vs. when the chain of promises resolves.

Every robot test fixture is something like:

var d = new doh.Deferred();

...

robot.sequence(d.getTestCallback(function(){
   doh.t(...);
}), 500);

return d;

The sequence of actions is:

0. all but the last action in the seqPromise queue runs, and the 500ms delay runs too
1. now the seqPromise queue has 1 item: the callback above
2. the callback starts and finishes running
3. getTestCallback() resolves the Deferred "d", triggering this text fixture to end and the new test fixture to start running
4. before() advice runs, and tries to clear any leftover actions from the previous test

The problem is that even though the final callback function has finished executing, seqPromise hasn't quite yet been marked as fulfilled.

[Douglas Hays]

Though not eloquent, a simple setTimeout (and saving the current seqPromise in a local for deferred execution) seems to solve the issue.

[haysmark]

Option A: add setTimeout to robot.js.

[haysmark]

Option B: add setTimeout to runner.js.

[haysmark]

Alternatively, as Bill suggested, we could add the setTimeout to runner.js. I was initially worried about putting the setTimeout in robot.js but if we save the seqPromise it should be OK. I attached both versions; if there are no objections, I will commit the robot.js version.

[bill]

I'm OK with that.

For the record, for 2.0 I'd like to have a cleaner API for writing robot tests. I filed #15644 to track that.

[haysmark]

In [29184]:

Allow promise to mark itself as fulfilled before checking to see if it needs to be canceled. Fixes #15642. !strict

[Bill Keese <[email protected]…>]

In 59ba49664c1a6b0feb13623be32e10b9679d5cbb/util:

Error: Processor CommitTicketReference failed

Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled?