diff options
| author | bolasblack | 2013-08-06 17:48:32 +0800 |
|---|---|---|
| committer | Igor Minar | 2013-08-09 09:02:58 -0700 |
| commit | a207665dad69248139b150cd3fe8ba13059bffb4 (patch) | |
| tree | 48be81627bcef0e8fe3f495c175c8b055c754cd0 /test/ng/qSpec.js | |
| parent | b3087421f20241f9bcb9b1f00ee9a30efc8b0899 (diff) | |
| download | angular.js-a207665dad69248139b150cd3fe8ba13059bffb4.tar.bz2 | |
feat($q): add shorthand for defining promise error handlers
Now we can instead this
promise.then(null, errorHandler)
with this
promise.catch(errorhandler)
Closes #2048
Closes #3476
Diffstat (limited to 'test/ng/qSpec.js')
| -rw-r--r-- | test/ng/qSpec.js | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/test/ng/qSpec.js b/test/ng/qSpec.js index 6d08cb15..4f2492fb 100644 --- a/test/ng/qSpec.js +++ b/test/ng/qSpec.js @@ -516,6 +516,10 @@ describe('q', function() { expect(typeof promise.then).toBe('function'); }); + it('should have a catch method', function() { + expect(typeof promise['catch']).toBe('function'); + }); + it('should have a always method', function() { expect(typeof promise.always).toBe('function'); }); @@ -881,6 +885,14 @@ describe('q', function() { }); }); + + describe('catch', function() { + it('should be a shorthand for defining promise error handlers', function() { + promise['catch'](error(1)).then(null, error(2)) + syncReject(deferred, 'foo'); + expect(logStr()).toBe('error1(foo)->reject(foo); error2(foo)->reject(foo)'); + }); + }); }); }); |
