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

Unified Diff: chrome/test/data/extensions/api_test/media_galleries/scan/test.js

Issue 174013009: Add end to end test for Media galleries scan API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix mock Created 6 years, 10 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/media_galleries/cancel_scan/test.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/media_galleries/scan/test.js
diff --git a/chrome/test/data/extensions/api_test/media_galleries/scan/test.js b/chrome/test/data/extensions/api_test/media_galleries/scan/test.js
index aae404fb71e405bf848dd5b3631ffb93192ce108..21c061eca4ba95163d1bb0ee4fad1df3d72d17b1 100644
--- a/chrome/test/data/extensions/api_test/media_galleries/scan/test.js
+++ b/chrome/test/data/extensions/api_test/media_galleries/scan/test.js
@@ -4,34 +4,38 @@
var mediaGalleries = chrome.mediaGalleries;
-function StartAndCancelMediaScanTest() {
- function StartMediaScanTest() {
- var startEventListener = function(details) {
- chrome.test.assertEq('start', details.type);
- mediaGalleries.onScanProgress.removeListener(startEventListener);
- CancelMediaScanTest();
- }
- mediaGalleries.onScanProgress.addListener(startEventListener);
+function MediaScanTest() {
+ var scanProgress = 'start';
+ var initialGalleryCount = 0;
- mediaGalleries.startMediaScan();
+ function OnScanResultsAdded(galleries) {
+ chrome.test.assertEq(initialGalleryCount + 1, galleries.length);
+ chrome.test.succeed();
}
- function CancelMediaScanTest() {
- var cancelEventListener = function(details) {
- chrome.test.assertEq('cancel', details.type);
- mediaGalleries.onScanProgress.removeListener(cancelEventListener);
- chrome.test.succeed();
- };
- mediaGalleries.onScanProgress.addListener(cancelEventListener);
+ function OnScanProgress(details) {
+ chrome.test.assertEq(scanProgress, details.type);
+ if (scanProgress == 'start') {
+ scanProgress = 'finish';
+ } else {
+ scanProgress = 'done';
+ chrome.test.runWithUserGesture(function() {
+ mediaGalleries.addScanResults(OnScanResultsAdded);
+ });
+ }
+ }
- mediaGalleries.cancelMediaScan();
+ function OnInitialMediaGalleries(galleries) {
+ initialGalleryCount = galleries.length;
+ mediaGalleries.onScanProgress.addListener(OnScanProgress);
+ mediaGalleries.startMediaScan();
}
- StartMediaScanTest();
+ mediaGalleries.getMediaFileSystems(OnInitialMediaGalleries);
}
CreateDummyWindowToPreventSleep();
chrome.test.runTests([
- StartAndCancelMediaScanTest,
+ MediaScanTest,
]);
« no previous file with comments | « chrome/test/data/extensions/api_test/media_galleries/cancel_scan/test.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698