[v0.0.2] Wiring bug in individual error handlers #2

Closed
ryan-self opened this Issue Sep 28, 2012 · 1 comment

Comments

Projects
None yet
1 participant
Owner

ryan-self commented Sep 28, 2012

In v0.0.2, error handlers are not being fired when a particular command fails.

Example:

execPlan.add('./command_that_does_not_exist', function () { console.log('error'); });
execPlan.add('./command_that_does_exist', function () { console.log('error2'); });
execPlan.execute();

The error handler for the first step is not executed.

@ryan-self ryan-self added a commit that referenced this issue Sep 29, 2012

@ryan-self ryan-self Issue #2: Include test cases that must be pased for bug to be fixed. …
…These test cases include checking that in an exec plan of 4 commands, an error handler in each position should be called as expected.
f5ce47c

@ryan-self ryan-self added a commit that referenced this issue Sep 29, 2012

@ryan-self ryan-self Issue #1, #2: Complete feature to introduce "error continuing" policy…
…, with the ability for individual error handlers to override that policy. Also, fix bug originally blocking this feature. This bug entailed the individual error handlers not being called for corresponding commands.
0a8897c

ryan-self was assigned Sep 29, 2012

Owner

ryan-self commented Sep 29, 2012

All test cases required to pass for this bug have passed; therefore, closing this bug. This fix wil be included in v0.0.3.

ryan-self closed this Sep 29, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment