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

Unified Diff: chrome/test/data/extensions/api_test/search_engines_private/test.js

Issue 1109563003: Implement remaining chrome.searchEnginesPrivate methods. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix the event router bug so my test passes now Created 5 years, 8 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
Index: chrome/test/data/extensions/api_test/search_engines_private/test.js
diff --git a/chrome/test/data/extensions/api_test/search_engines_private/test.js b/chrome/test/data/extensions/api_test/search_engines_private/test.js
index 4e52b7e5462f15c469aa93b8f577240c3d334853..42afdb1fef83bb67d7b47e48fc38cc5a38f9a9ba 100644
--- a/chrome/test/data/extensions/api_test/search_engines_private/test.js
+++ b/chrome/test/data/extensions/api_test/search_engines_private/test.js
@@ -35,7 +35,77 @@ var availableTests = [
chrome.searchEnginesPrivate.getDefaultSearchEngines(function(engines) {
chrome.searchEnginesPrivate.setSelectedSearchEngine(engines[1].guid);
});
- }
+ },
+
+ function addNewSearchEngine() {
+ var testName = 'name';
+ var testKeyword = 'search.com';
+ var testUrl = 'http://search.com';
+ chrome.searchEnginesPrivate.addOtherSearchEngine(
+ testName, testKeyword, testUrl);
+ chrome.searchEnginesPrivate.getOtherSearchEngines(function(engines) {
+ for (var i = 0; i < engines.length; i++) {
+ if (engines[i].name == testName) {
+ chrome.test.assertEq(testKeyword, engines[i].keyword);
+ chrome.test.assertEq(testUrl, engines[i].url);
+ chrome.test.succeed();
+ return;
+ }
+ }
+ chrome.test.fail();
+ });
+ },
+
+ function updateSearchEngine() {
+ chrome.searchEnginesPrivate.addOtherSearchEngine(
+ 'name1', 'search1.com', 'http://search1.com');
+ chrome.searchEnginesPrivate.getOtherSearchEngines(function(engines) {
+ chrome.searchEnginesPrivate.updateSearchEngine(
+ engines[0].guid, 'name2', 'search2.com', 'http://search2.com');
+ chrome.searchEnginesPrivate.getOtherSearchEngines(function(newEngines) {
+ chrome.test.assertEq('name2', newEngines[0].name);
+ chrome.test.assertEq('search2.com', newEngines[0].keyword);
+ chrome.test.assertEq('http://search2.com', newEngines[0].url);
+ chrome.test.succeed();
+ });
+ });
+ },
+
+ function removeSearchEngine() {
+ chrome.searchEnginesPrivate.addOtherSearchEngine(
+ 'name1', 'search1.com', 'http://search1.com');
+ chrome.searchEnginesPrivate.addOtherSearchEngine(
+ 'name2', 'search2.com', 'http://search2.com');
+ chrome.searchEnginesPrivate.getOtherSearchEngines(function(engines) {
+ var engine0Guid = engines[0].guid;
+ chrome.searchEnginesPrivate.removeSearchEngine(engine0Guid);
+
+ chrome.searchEnginesPrivate.getOtherSearchEngines(function(newEngines) {
+ for (var i = 0; i < newEngines.length; i++) {
+ chrome.test.assertFalse(newEngines[i].guid == engine0Guid);
+ }
+ chrome.test.succeed();
+ });
+ });
+ },
+
+ function onOtherSearchEnginesChanged() {
+ chrome.searchEnginesPrivate.onOtherSearchEnginesChanged.addListener(
+ function(engines) {
+ for (var i = 0; i < engines.length; i++) {
+ if (engines[i].name == 'name1') {
+ chrome.test.assertEq('search1.com', engines[i].keyword);
+ chrome.test.assertEq('http://search1.com/%s', engines[i].url);
+ chrome.test.succeed();
+ return;
+ }
+ }
+ chrome.test.fail();
+ });
+
+ chrome.searchEnginesPrivate.addOtherSearchEngine(
+ 'name1', 'search1.com', 'http://search1.com/%s');
+ },
];
var testToRun = window.location.search.substring(1);

Powered by Google App Engine
This is Rietveld 408576698