Commit bc5848fe authored by Steffen van Bergerem's avatar Steffen van Bergerem Committed by Benjamin Neff

Use new ajax error handling for reshare failures

parent d609238e
......@@ -99,11 +99,7 @@
Diaspora.Mobile.PostActions.toggleActive(link);
},
error: function(response) {
if (response.status === 0) {
alert(Diaspora.I18n.t("errors.connection"));
} else {
alert(response.responseText);
}
Diaspora.Mobile.Alert.handleAjaxError(response);
},
complete: function() {
Diaspora.Mobile.PostActions.hideLoader(link);
......
......@@ -198,7 +198,6 @@ describe("Diaspora.Mobile.PostActions", function(){
Diaspora.Mobile.PostActions.initialize();
this.reshareLink = $(".stream .reshare-action");
spyOn(window, "confirm").and.returnValue(true);
spyOn(window, "alert");
});
it("always calls showLoader before sending request and hideLoader after receiving response", function(){
......@@ -223,16 +222,12 @@ describe("Diaspora.Mobile.PostActions", function(){
expect(Diaspora.Mobile.PostActions.toggleActive).toHaveBeenCalledWith(this.reshareLink);
});
it("pops an alert on server errors", function() {
it("lets Diaspora.Mobile.Alert handle AJAX errors", function() {
spyOn(Diaspora.Mobile.Alert, "handleAjaxError");
this.reshareLink.click();
jasmine.Ajax.requests.mostRecent().respondWith({status: 400, responseText: "reshare failed"});
expect(window.alert).toHaveBeenCalledWith("reshare failed");
});
it("pops an alert on network errors", function() {
this.reshareLink.click();
jasmine.Ajax.requests.mostRecent().abort();
expect(window.alert).toHaveBeenCalledWith(Diaspora.I18n.t("errors.connection"));
expect(Diaspora.Mobile.Alert.handleAjaxError).toHaveBeenCalled();
expect(Diaspora.Mobile.Alert.handleAjaxError.calls.argsFor(0)[0].responseText).toBe("reshare failed");
});
});
});
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment