Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/content/tests/subscriptionClasses.js

Issue 10964120: Made Synchronizer tests work again (Closed)
Patch Set: Updated for Synchronizer changes Created July 17, 2013, 12:33 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/content/index.html ('k') | chrome/content/tests/synchronizer.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/tests/subscriptionClasses.js
===================================================================
--- a/chrome/content/tests/subscriptionClasses.js
+++ b/chrome/content/tests/subscriptionClasses.js
@@ -34,33 +34,31 @@
test("Subscriptions with state", function()
{
compareSubscription("~fl~", ["url=~fl~", "title=" + Utils.getString("newGroup_title")]);
compareSubscription("http://test/default", ["url=http://test/default", "title=http://test/default"]);
compareSubscription("http://test/default_titled", ["url=http://test/default_titled", "title=test"], function(subscription)
{
subscription.title = "test";
});
- compareSubscription("http://test/non_default", ["url=http://test/non_default", "title=test", "nextURL=http://test2/",
- "disabled=true", "lastSuccess=8", "lastDownload=12", "lastCheck=16", "softExpiration=18", "expires=20", "downloadStatus=foo", "lastModified=bar",
- "errors=3", "requiredVersion=0.6", "alternativeLocations=http://foo/;q=0.5,http://bar/;q=2"], function(subscription)
+ compareSubscription("http://test/non_default", ["url=http://test/non_default", "title=test",
+ "disabled=true", "lastSuccess=8", "lastDownload=12", "lastCheck=16", "softExpiration=18", "expires=20", "downloadStatus=foo",
+ "errors=3", "version=24", "requiredVersion=0.6"], function(subscription)
{
subscription.title = "test";
- subscription.nextURL = "http://test2/";
subscription.disabled = true;
subscription.lastSuccess = 8;
subscription.lastDownload = 12;
subscription.lastCheck = 16;
subscription.softExpiration = 18;
subscription.expires = 20;
subscription.downloadStatus = "foo";
- subscription.lastModified = "bar";
subscription.errors = 3;
+ subscription.version = 24
subscription.requiredVersion = "0.6";
- subscription.alternativeLocations = "http://foo/;q=0.5,http://bar/;q=2";
});
compareSubscription("~wl~", ["url=~wl~", "disabled=true", "title=Test group"], function(subscription)
{
subscription.title = "Test group";
subscription.disabled = true;
});
});
})();
« no previous file with comments | « chrome/content/index.html ('k') | chrome/content/tests/synchronizer.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld