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

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

Issue 29345400: Issue 4093 - Adapt unit tests for changes to Subscription.title (Closed)
Patch Set: Created May 31, 2016, 10:22 a.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 | « no previous file | no next file » | 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
@@ -28,17 +28,17 @@
equal(typeof SpecialSubscription, "function", "typeof SpecialSubscription");
equal(typeof RegularSubscription, "function", "typeof RegularSubscription");
equal(typeof ExternalSubscription, "function", "typeof ExternalSubscription");
equal(typeof DownloadableSubscription, "function", "typeof DownloadableSubscription");
});
test("Subscriptions with state", function()
{
- compareSubscription("~fl~", ["url=~fl~", "title=" + Utils.getString("newGroup_title")]);
+ compareSubscription("~fl~", ["url=~fl~"]);
Sebastian Noack 2016/05/31 10:28:34 Didn't we completely removed that syntax?
Wladimir Palant 2016/05/31 10:32:58 We removed special treatment for these subscriptio
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",
"disabled=true", "lastSuccess=8", "lastDownload=12", "lastCheck=16", "softExpiration=18", "expires=20", "downloadStatus=foo",
"errors=3", "version=24", "requiredVersion=0.6"], function(subscription)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld