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

Unified Diff: trunk/src/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js

Issue 217743002: Revert 260222 "Separate audio player tests from special file tes..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js
===================================================================
--- trunk/src/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js (revision 260296)
+++ trunk/src/chrome/test/data/extensions/api_test/file_manager_browsertest/open_special_types.js (working copy)
@@ -58,6 +58,89 @@
}
/**
+ * Tests if the audio player shows up for the selected image and that the audio
+ * is loaded successfully.
+ *
+ * @param {string} path Directory path to be tested.
+ */
+function audioOpen(path) {
+ var appId;
+ var audioAppId;
+
+ var expectedFilesBefore =
+ TestEntryInfo.getExpectedRows(path == RootPath.DRIVE ?
+ BASIC_DRIVE_ENTRY_SET : BASIC_LOCAL_ENTRY_SET).sort();
+ var expectedFilesAfter =
+ expectedFilesBefore.concat([ENTRIES.newlyAdded.getExpectedRow()]).sort();
+
+ StepsRunner.run([
+ function() {
+ setupAndWaitUntilReady(null, path, this.next);
+ },
+ // Select the song.
+ function(inAppId) {
+ appId = inAppId;
+
+ // Add an additional audio file.
+ addEntries(['local', 'drive'], [ENTRIES.newlyAdded], this.next);
+ },
+ function(result) {
+ chrome.test.assertTrue(result);
+ waitForFileListChange(appId, expectedFilesBefore.length).then(this.next);
+ },
+ function(actualFilesAfter) {
+ chrome.test.assertEq(expectedFilesAfter, actualFilesAfter);
+ callRemoteTestUtil(
+ 'openFile', appId, ['Beautiful Song.ogg'], this.next);
+ },
+ // Wait for the audio player window.
+ function(result) {
+ chrome.test.assertTrue(result);
+ waitForWindow('audio_player.html').then(this.next);
+ },
+ // Wait for the changes of the player status.
+ function(inAppId) {
+ audioAppId = inAppId;
+ waitForElement(audioAppId, 'audio-player[playing]').then(this.next);
+ },
+ // Get the source file name.
+ function(element) {
+ chrome.test.assertEq(
+ 'filesystem:chrome-extension://hhaomjibdihmijegdhdafkllkbggdgoj/' +
+ 'external' + path + '/Beautiful%20Song.ogg',
+ element.attributes.currenttrackurl);
+
+ // Open another file.
+ callRemoteTestUtil(
+ 'openFile', appId, ['newly added file.ogg'], this.next);
+ },
+ // Wait for the changes of the player status.
+ function(result) {
+ chrome.test.assertTrue(result);
+ var query = 'audio-player' +
+ '[playing]' +
+ '[currenttrackurl$="newly%20added%20file.ogg"]';
+ waitForElement(audioAppId, query).then(this.next);
+ },
+ // Get the source file name.
+ function(element) {
+ chrome.test.assertEq(
+ 'filesystem:chrome-extension://hhaomjibdihmijegdhdafkllkbggdgoj/' +
+ 'external' + path + '/newly%20added%20file.ogg',
+ element.attributes.currenttrackurl);
+
+ // Close window
+ closeWindowAndWait(audioAppId).then(this.next);
+ },
+ // Wait for the audio player.
+ function(result) {
+ chrome.test.assertTrue(result);
+ checkIfNoErrorsOccured(this.next);
+ }
+ ]);
+}
+
+/**
* Tests if we can open and unmount a zip file.
* @param {string} path Directory path to be tested.
*/
@@ -128,10 +211,18 @@
galleryOpen(RootPath.DOWNLOADS);
};
+testcase.audioOpenDownloads = function() {
+ audioOpen(RootPath.DOWNLOADS);
+};
+
testcase.galleryOpenDrive = function() {
galleryOpen(RootPath.DRIVE);
};
+testcase.audioOpenDrive = function() {
+ audioOpen(RootPath.DRIVE);
+};
+
testcase.zipOpenDownloads = function() {
zipOpen(RootPath.DOWNLOADS);
};
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698