mirror of
https://github.com/Hopiu/angular.js.git
synced 2026-03-16 23:30:23 +00:00
fix($q): make $q.reject support finally and catch
Add support for the functions `finally` and `catch` to the promise returned by `$q.reject` Closes #6048 Closes #6076
This commit is contained in:
parent
5ed721b9b5
commit
074b0675a1
2 changed files with 14 additions and 1 deletions
|
|
@ -223,7 +223,7 @@ function qFactory(nextTick, exceptionHandler) {
|
|||
|
||||
|
||||
reject: function(reason) {
|
||||
deferred.resolve(reject(reason));
|
||||
deferred.resolve(createInternalRejectedPromise(reason));
|
||||
},
|
||||
|
||||
|
||||
|
|
@ -380,6 +380,12 @@ function qFactory(nextTick, exceptionHandler) {
|
|||
* @returns {Promise} Returns a promise that was already resolved as rejected with the `reason`.
|
||||
*/
|
||||
var reject = function(reason) {
|
||||
var result = defer();
|
||||
result.reject(reason);
|
||||
return result.promise;
|
||||
};
|
||||
|
||||
var createInternalRejectedPromise = function(reason) {
|
||||
return {
|
||||
then: function(callback, errback) {
|
||||
var result = defer();
|
||||
|
|
|
|||
|
|
@ -959,6 +959,13 @@ describe('q', function() {
|
|||
mockNextTick.flush();
|
||||
expect(log).toEqual(['errorBroken(rejected)->throw(catch me!)', 'errorAffected(catch me!)->reject(catch me!)']);
|
||||
});
|
||||
|
||||
|
||||
it('should have functions `finally` and `catch`', function() {
|
||||
var rejectedPromise = q.reject('rejected');
|
||||
expect(rejectedPromise['finally']).not.toBeUndefined();
|
||||
expect(rejectedPromise['catch']).not.toBeUndefined();
|
||||
});
|
||||
});
|
||||
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue