search

instructure / ic-ajax

ember-friendly jQuery.ajax wrapper
MIT License
167 stars 29 forks source link

Allow fixtures that simulate an error response #6

Closed balinterdi closed 10 years ago

balinterdi commented 10 years ago

Currently all fixtures will be resolved and thus mean success. This does not allow testing scenarios for error requests from the backend.

This commit fixes that.

ryanflorence commented 10 years ago

Awesome, I have been intending to do this, at the dentist now but will check this out today.

On Feb 6, 2014, at 6:00 AM, Balint Erdi notifications@github.com wrote:

Currently all fixtures will be resolved and thus mean success. This does not allow testing scenarios for error requests from the backend.

You can merge this Pull Request by running

git pull https://github.com/balinterdi/ic-ajax allow-error-fixtures Or view, comment on, or merge it at:

https://github.com/instructure/ic-ajax/pull/6

Commit Summary

Reject the promise if the fixture's textStatus is not success File Changes

M main.js (6) M test.js (13) Patch Links:

https://github.com/instructure/ic-ajax/pull/6.patch https://github.com/instructure/ic-ajax/pull/6.diff — Reply to this email directly or view it on GitHub.