Clean up tests to use reportPromise.

chrome/test/data/file_manager/unit_tests/media_scanner_unittest.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * Dummy private APIs.
  */
 var chrome;
 
 /**
@@ skipping 59 matching lines @@
           return new Promise(function(resolve, reject) {
             dir.getFile(filename, {create: true}, resolve, reject);
           });
         }
       })).then(function() { return dir; });
 }
 
 /**
  * Verifies that scanning an empty filesystem produces an empty list.
  */
-function testEmptyList(errorIf) {
+function testEmptyList(callback) {
   var scanner = new MediaScanner([]);
-  scanner.getFiles().then(function(files) {
-    errorIf(files.length !== 0);
-  });
+  reportPromise(
+      scanner.getFiles().then(function(files) {
+        assertEquals(0, files.length);
+      }),
+      callback);
 }
 
 /**
  * Verifies that scanning a simple single-level directory structure works.
  */
-function testSingleLevel(errorIf) {
+function testSingleLevel(callback) {
   var filenames = [
       'foo',
       'foo.jpg',
       'bar.gif',
       'baz.avi',
       'foo.mp3',
       'bar.txt'
   ];
   var expectedFiles = [
       '/testSingleLevel/foo.jpg',
       '/testSingleLevel/bar.gif',
       '/testSingleLevel/baz.avi'
   ];
-  makeTestFilesystemRoot('testSingleLevel')
-      .then(populateDir.bind(null, filenames))
-      .then(
-          /**
-           * Scans the directory.
-           * @param {!DirectoryEntry} root
-           */
-          function(root) {
-            var scanner = new MediaScanner([root]);
-            return scanner.getFiles();
-          })
-      .then(
-          /**
-           * Verifies the results of the media scan.
-           * @param {!Array.<!FileEntry>} scanResults
-           */
-          function(scanResults) {
-            assertEquals(expectedFiles.length, scanResults.length);
-            scanResults.forEach(function(result) {
-              // Verify that the scanner only returns files.
-              assertTrue(result.isFile, result.fullPath + ' is not a file');
-              assertTrue(expectedFiles.indexOf(result.fullPath) != -1,
-                  result.fullPath + ' not found in control set');
-            });
-            // Signal test completion with no errors.
-            errorIf(false);
-          })
-      .catch(
-          function(e) {
-            // Catch failures and print them.
-            console.error(e);
-            errorIf(e);
-          });
+  reportPromise(
+      makeTestFilesystemRoot('testSingleLevel')
+          .then(populateDir.bind(null, filenames))
+          .then(
+              /**
+               * Scans the directory.
+               * @param {!DirectoryEntry} root
+               */
+              function(root) {
+                var scanner = new MediaScanner([root]);
+                return scanner.getFiles();
+              })
+          .then(
+              /**
+               * Verifies the results of the media scan.
+               * @param {!Array.<!FileEntry>} scanResults
+               */
+              function(scanResults) {
+                assertEquals(expectedFiles.length, scanResults.length);
+                scanResults.forEach(function(result) {
+                  // Verify that the scanner only returns files.
+                  assertTrue(result.isFile, result.fullPath + ' is not a file');
+                  assertTrue(expectedFiles.indexOf(result.fullPath) != -1,
+                      result.fullPath + ' not found in control set');
+                });
+              }),
+          callback);
 }
 
-function testMultiLevel(errorIf) {
+function testMultiLevel(callback) {
   var filenames = [
       'foo.jpg',
       'bar',
       [
           'foo.0',
           'bar.0.jpg'
       ],
       [
           'foo.1',
           'bar.1.gif',
           [
               'foo.1.0',
               'bar.1.0.avi'
           ]
       ]
   ];
   var expectedFiles = [
       '/testMultiLevel/foo.jpg',
       '/testMultiLevel/foo.0/bar.0.jpg',
       '/testMultiLevel/foo.1/bar.1.gif',
       '/testMultiLevel/foo.1/foo.1.0/bar.1.0.avi'
   ];
 
-  makeTestFilesystemRoot('testMultiLevel')
-      .then(populateDir.bind(null, filenames))
-      .then(
-          /**
-           * Scans the directory.
-           * @param {!DirectoryEntry} root
-           */
-          function(root) {
-            var scanner = new MediaScanner([root]);
-            return scanner.getFiles();
-          })
-      .then(
-          /**
-           * Verifies the results of the media scan.
-           * @param {!Array.<!FileEntry>} scanResults
-           */
-          function(scanResults) {
-            assertEquals(expectedFiles.length, scanResults.length);
-            scanResults.forEach(function(result) {
-              // Verify that the scanner only returns files.
-              assertTrue(result.isFile, result.fullPath + ' is not a file');
-              assertTrue(expectedFiles.indexOf(result.fullPath) != -1,
-                  result.fullPath + ' not found in control set');
-            });
-            // Signal test completion with no errors.
-            errorIf(false);
-          })
-      .catch(
-          function(e) {
-            // Catch failures and print them.
-            console.error(e);
-            errorIf(e);
-          });
-
-  errorIf(false);
+  reportPromise(
+      makeTestFilesystemRoot('testMultiLevel')
+          .then(populateDir.bind(null, filenames))
+          .then(
+              /**
+               * Scans the directory.
+               * @param {!DirectoryEntry} root
+               */
+              function(root) {
+                var scanner = new MediaScanner([root]);
+                return scanner.getFiles();
+              })
+          .then(
+              /**
+               * Verifies the results of the media scan.
+               * @param {!Array.<!FileEntry>} scanResults
+               */
+              function(scanResults) {
+                assertEquals(expectedFiles.length, scanResults.length);
+                scanResults.forEach(function(result) {
+                  // Verify that the scanner only returns files.
+                  assertTrue(result.isFile, result.fullPath + ' is not a file');
+                  assertTrue(expectedFiles.indexOf(result.fullPath) != -1,
+                      result.fullPath + ' not found in control set');
+                });
+              }),
+      callback);
 }
 
-function testMultipleDirectories(errorIf) {
+function testMultipleDirectories(callback) {
   var filenames = [
       'foo',
       'bar',
       [
           'foo.0',
           'bar.0.jpg'
       ],
       [
           'foo.1',
           'bar.1.jpg',
       ]
   ];
   // Expected file paths from the scan.  We're scanning the two subdirectories
   // only.
   var expectedFiles = [
       '/testMultipleDirectories/foo.0/bar.0.jpg',
       '/testMultipleDirectories/foo.1/bar.1.jpg'
   ];
 
   var getDirectory = function(root, dirname) {
     return new Promise(function(resolve, reject) {
       root.getDirectory(
           dirname, {create: false}, resolve, reject);
     });
   };
-  makeTestFilesystemRoot('testMultipleDirectories')
-      .then(populateDir.bind(null, filenames))
-      .then(
-          /**
-           * Scans the directories.
-           * @param {!DirectoryEntry} root
-           */
-          function(root) {
-            return Promise.all(['foo.0', 'foo.1'].map(
-                getDirectory.bind(null, root))).then(
-                    function(directories) {
-                      var scanner = new MediaScanner(directories);
-                      return scanner.getFiles();
-                    });
-          })
-      .then(
-          /**
-           * Verifies the results of the media scan.
-           * @param {!Array.<!FileEntry>} scanResults
-           */
-          function(scanResults) {
-            assertEquals(expectedFiles.length, scanResults.length);
-            scanResults.forEach(function(result) {
-              // Verify that the scanner only returns files.
-              assertTrue(result.isFile, result.fullPath + ' is not a file');
-              assertTrue(expectedFiles.indexOf(result.fullPath) != -1,
-                  result.fullPath + ' not found in control set');
-            });
-            // Signal test completion with no errors.
-            errorIf(false);
-          })
-      .catch(
-          function(e) {
-            // Catch failures and print them.
-            console.error(e);
-            errorIf(e);
-          });
+
+  reportPromise(
+      makeTestFilesystemRoot('testMultipleDirectories')
+          .then(populateDir.bind(null, filenames))
+          .then(
+              /**
+               * Scans the directories.
+               * @param {!DirectoryEntry} root
+               */
+              function(root) {
+                return Promise.all(['foo.0', 'foo.1'].map(
+                    getDirectory.bind(null, root))).then(
+                        function(directories) {
+                          var scanner = new MediaScanner(directories);
+                          return scanner.getFiles();
+                        });
+              })
+          .then(
+              /**
+               * Verifies the results of the media scan.
+               * @param {!Array.<!FileEntry>} scanResults
+               */
+              function(scanResults) {
+                assertEquals(expectedFiles.length, scanResults.length);
+                scanResults.forEach(function(result) {
+                  // Verify that the scanner only returns files.
+                  assertTrue(result.isFile, result.fullPath + ' is not a file');
+                  assertTrue(expectedFiles.indexOf(result.fullPath) != -1,
+                      result.fullPath + ' not found in control set');
+                });
+              }),
+      callback);
 }