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

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

Issue 8450003: Reimplemented whitelist handling (Closed)
Patch Set: Created Sept. 25, 2012, 3:48 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/tests/qunit.js ('k') | lib/rules.js » ('j') | lib/rules.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/tests/tests/suffixTreeManipulation.js
===================================================================
new file mode 100644
--- /dev/null
+++ b/chrome/content/tests/tests/suffixTreeManipulation.js
@@ -0,0 +1,205 @@
+(function()
+{
Thomas Greiner 2012/09/26 15:49:13 bracket on first line (as seen on last line)
Wladimir Palant 2012/09/27 14:55:06 Not really how I usually write it...
+ function getModuleGlobal(module)
+ {
+ let result = Cu.getGlobalForObject(require(module));
+ if (result == window)
+ {
+ // Work-around for bug 736316 - getGlobalForObject gave us our own window
+ let {XPIProvider} = Cu.import("resource://gre/modules/XPIProvider.jsm", null);
+ let addonID = "{d10d0bf8-f5b5-c8b4-a8b2-2b9879e08c5d}"
Thomas Greiner 2012/09/26 15:49:13 semicolon is missing
Wladimir Palant 2012/09/27 14:55:06 Bigger issue actually - that's the ID for Adblock
+ if (addonID in XPIProvider.bootstrapScopes)
+ result = XPIProvider.bootstrapScopes[addonID];
+ }
+
+ if ("require" in result)
+ result = result.require.scopes[module];
+ return result;
+ }
+
+ let {onWhitelistEntryAdded, onWhitelistEntryRemoved} = require("rules");
+ let rulesGlobal = getModuleGlobal("rules");
+
+ function saveRules()
+ {
+ this._rulesBackup = rulesGlobal.rules;
+ rulesGlobal.rules = {domain: {}};
+ }
+
+ function restoreRules()
+ {
+ rulesGlobal.rules = this._rulesBackup;
+ }
+
+ module("Suffix tree manipulation", {
+ setup: saveRules,
+ teardown: restoreRules
+ });
+
+ test("Adding entries", function()
+ {
+ let priority = " " + rulesGlobal.CUSTOM_RULE_PRIORITY;
+ let domains = rulesGlobal.rules.domain;
+
+ deepEqual(domains, {}, "Initial state");
+
+ onWhitelistEntryAdded("foo");
+ deepEqual(domains, {
+ "o": "of" + priority
+ }, "Added foo");
+
+ onWhitelistEntryAdded("goo");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": priority,
+ "g": priority
+ }
+ }
+ }, "Added goo");
+
+ onWhitelistEntryAdded("xyzfoo");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Added xyzfoo");
+
+ onWhitelistEntryAdded("o");
+ deepEqual(domains, {
+ "o": {
+ "": priority,
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Added o");
+
+ domains.o[""] = " 1234";
+ onWhitelistEntryAdded("o");
+ deepEqual(domains, {
+ "o": {
+ "": priority,
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Re-added o");
+ });
+
+ test("Removing entries", function()
+ {
+ let priority = " " + rulesGlobal.CUSTOM_RULE_PRIORITY;
+ let domains = rulesGlobal.rules.domain = {
+ "o": {
+ "": priority,
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ };
+
+ deepEqual(domains, {
+ "o": {
+ "": priority,
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Initial state");
+
+ onWhitelistEntryRemoved("o");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Removed o");
+
+ onWhitelistEntryRemoved("go");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ "": priority,
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Removed go");
+
+ onWhitelistEntryRemoved("foo");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Removed foo");
+
+ onWhitelistEntryRemoved("xxyzfoo");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ "z": "yx" + priority
+ },
+ "g": priority
+ }
+ }
+ }, "Removed xxyzfoo");
+
+ onWhitelistEntryRemoved("xyzfoo");
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ },
+ "g": priority
+ }
+ }
+ }, "Removed xyzfoo");
+
+ domains.o.o.g = " 1234";
+ deepEqual(domains, {
+ "o": {
+ "o": {
+ "f": {
+ },
+ "g": " 1234"
+ }
+ }
+ }, "Attempted to remove goo");
+ });
+})();
« no previous file with comments | « chrome/content/tests/qunit.js ('k') | lib/rules.js » ('j') | lib/rules.js » ('J')

Powered by Google App Engine
This is Rietveld