Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5577)

Unified Diff: appengine/swarming/ui/res/imp/common/pageable-data.html

Issue 2419613002: Add basic swarming UI tests
Patch Set: Make sure symlink isn't followed for uploading Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « appengine/swarming/ui/build/js/js.js ('k') | appengine/swarming/ui/res/imp/common/pageable-data-demo.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/swarming/ui/res/imp/common/pageable-data.html
diff --git a/appengine/swarming/ui/res/imp/common/pageable-data.html b/appengine/swarming/ui/res/imp/common/pageable-data.html
index 48356aa5466e5b56a40a80eb981cc6ddfeb60319..60afe025781077f3fa4e0fbee303617f06d5d200 100644
--- a/appengine/swarming/ui/res/imp/common/pageable-data.html
+++ b/appengine/swarming/ui/res/imp/common/pageable-data.html
@@ -155,10 +155,14 @@
}
this.set("_cursor", json.cursor || END);
this.set("busy", false);
+ // for unit testing
+ this.fire("_finishedLoading", {success:true});
}.bind(this)).catch(function(reason){
console.log("Reason for failure of request to " + this._url, reason);
sk.errorMessage("Could not get next set of results. Try reloading the page?", 0);
this.set("busy", false);
+ // for unit testing
+ this.fire("_finishedLoading", {success:false});
}.bind(this));
},
« no previous file with comments | « appengine/swarming/ui/build/js/js.js ('k') | appengine/swarming/ui/res/imp/common/pageable-data-demo.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698