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

Unified Diff: chrome/test/data/extensions/api_test/filebrowser_component/remote_search.js

Issue 11938034: drive: Replace local FakeDriveService with google_apis::FakeDriveService part 3 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: start-index -> start-offset Created 7 years, 11 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 | « chrome/test/data/extensions/api_test/filebrowser_component/remote.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/api_test/filebrowser_component/remote_search.js
diff --git a/chrome/test/data/extensions/api_test/filebrowser_component/remote_search.js b/chrome/test/data/extensions/api_test/filebrowser_component/remote_search.js
index 91e8ec7c4c3f5e34a728aee4c7eb366c8c071c36..2cd05d0c106a20ec1ecb23d80be1dd51aec1cba8 100644
--- a/chrome/test/data/extensions/api_test/filebrowser_component/remote_search.js
+++ b/chrome/test/data/extensions/api_test/filebrowser_component/remote_search.js
@@ -39,7 +39,7 @@ function verifyFileAccessible(entry, successCallback, errorCallback) {
}
function verifyDirectoryAccessible(entry,
- expectedChildrenNumber,
+ expectedNumChildren,
successCallback,
errorCallback) {
chrome.test.assertTrue(!!entry.createReader);
@@ -47,7 +47,7 @@ function verifyDirectoryAccessible(entry,
var children = [];
function onDirectoryRead() {
- chrome.test.assertEq(expectedChildrenNumber, children.length);
+ chrome.test.assertEq(expectedNumChildren, children.length);
successCallback();
}
@@ -67,13 +67,14 @@ function verifyDirectoryAccessible(entry,
readNext();
}
-function verifySearchResult(entries, nextFeed, expectedNextFeed) {
+function verifySearchResult(entries,
+ nextFeed,
+ expectedResult,
+ expectedNextFeed) {
chrome.test.assertTrue(!!entries);
- chrome.test.assertEq(2, entries.length);
+ chrome.test.assertEq(1, entries.length);
+ chrome.test.assertEq(expectedResult, entries[0].entry.fullPath);
chrome.test.assertEq(expectedNextFeed, nextFeed);
-
- chrome.test.assertEq('/drive/Folder', entries[0].entry.fullPath);
- chrome.test.assertEq('/drive/Folder/File.aBc', entries[1].entry.fullPath);
}
chrome.test.runTests([
@@ -87,7 +88,7 @@ chrome.test.runTests([
},
function driveSearch() {
var params = {
- 'query': 'foo',
+ 'query': 'F', // Matches "Folder" and "File.aBc".
'sharedWithMe': false,
'nextFeed': ''
};
@@ -95,21 +96,45 @@ chrome.test.runTests([
chrome.fileBrowserPrivate.searchDrive(
params,
function(entries, nextFeed) {
- verifySearchResult(entries, nextFeed, 'https://next_feed/');
+ // The first search should return the followings:
+ var expectedResult = '/drive/Folder';
+ var expectedNextFeed =
+ 'http://localhost/?start-offset=1&max-results=1';
+ verifySearchResult(
+ entries, nextFeed, expectedResult, expectedNextFeed);
+ var directoryEntry = entries[0].entry;
+
var nextParams = {
- 'query': 'foo',
+ 'query': 'F', // Matches "Folder" and "File.aBc".
'sharedWithMe': false,
'nextFeed': nextFeed
};
chrome.fileBrowserPrivate.searchDrive(
nextParams,
function(entries, nextFeed) {
- verifySearchResult(entries, nextFeed, '');
+ // The second search should return the followings:
+ var expectedResult = '/drive/Folder/File.aBc';
+ var expectedNextFeed = '';
+ verifySearchResult(entries,
+ nextFeed,
+ expectedResult,
+ expectedNextFeed);
+ var fileEntry = entries[0].entry;
- var directoryVerifier = verifyDirectoryAccessible.bind(null,
- entries[0].entry, 1, chrome.test.succeed, errorCallback);
+ // Check if directoryEntry ('/drive/Folder') is accessible
+ // and contains one child.
+ var expectedNumChildren = 1;
+ var directoryVerifier =
+ verifyDirectoryAccessible.bind(null,
+ directoryEntry,
+ expectedNumChildren,
+ chrome.test.succeed,
+ errorCallback);
- verifyFileAccessible(entries[1].entry, directoryVerifier,
+ // Check if the fileEntry ('/drive/Folder/File.aBc' is
+ // accessible). Note that |directoryVerifier| will be called
+ // upon success of verifyFileAccessible.
+ verifyFileAccessible(fileEntry, directoryVerifier,
errorCallback);
});
});
« no previous file with comments | « chrome/test/data/extensions/api_test/filebrowser_component/remote.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698