Add downloads.onCreated apitest and fix a few bugs

chrome/test/data/extensions/api_test/downloads/test.js

 // Copyright (c) 2011 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.
 
 // downloads api test
 // browser_tests.exe --gtest_filter=DownloadsApiTest.Downloads
 
 chrome.test.getConfig(function(testConfig) {
   function getURL(path) {
     return "http://localhost:" + testConfig.testServer.port + "/" + path;
   }
+
   var nextId = 0;
   function getNextId() {
     return nextId++;
   }
 
   // There can be only one assertThrows per test function.
   function assertThrows(func, arg, expected_message) {
     try {
       func(arg, function() {
         // Don't use chrome.test.callbackFail because it requires the
         // callback to be called.
         chrome.test.fail("Failed to throw exception (" +
                          expected_message + ")");
       });
     } catch (exception) {
       chrome.test.assertEq(expected_message, exception.message);
       chrome.test.succeed();
     }
   }
 
   chrome.test.runTests([
+    // TODO(benjhayden): Test onErased using remove().
     function downloadFilename() {
       chrome.experimental.downloads.download(
-        {"url": getURL("pass"), "filename": "pass"},
+        {"url": getURL("slow?0"), "filename": "foo"},

[Mihai Parparita -not on Chrome, 2011/10/18 19:59:37]

Can you add a comment why your'e using this URL (it's not obvious that a "slow"
name would be fast). Also, would /download-known-size be more appropriate?

[benjhayden, 2011/10/18 20:32:56]

Done.

/download-[un]known-size stalls the request until it receives a parallel request
for /download-finish. I could run an XHR, but it would be confusing unnecessary
overhead.

         chrome.test.callbackPass(function(id) {
           chrome.test.assertEq(getNextId(), id);
       }));
       // TODO(benjhayden): Test the filename using onChanged.
     },
+    function downloadOnCreated() {
+      function myListener(item) {
+        // Remove the listener so that we don't call chrome.test.succeed

[Mihai Parparita -not on Chrome, 2011/10/18 19:59:37]

You can use chrome.test.listenOnce instead.

[benjhayden, 2011/10/18 20:32:56]

Done.

+        // spuriously for following test functions. Avoid
+        // chrome.test.callbackPass for event handlers so that we can use our
+        // own explicit version of the pendingCallbacks counter if necessary.
+        chrome.experimental.downloads.onCreated.removeListener(myListener);
+        chrome.test.succeed();
+      };
+      chrome.experimental.downloads.onCreated.addListener(myListener);
+      chrome.experimental.downloads.download(
+        {"url": getURL("slow?0")},
+        function(id) {
+          chrome.test.assertEq(getNextId(), id);
+      });
+    },
     function downloadSubDirectoryFilename() {
       chrome.experimental.downloads.download(
-        {"url": getURL("pass"), "filename": "foo/pass"},
+        {"url": getURL("slow?0"), "filename": "foo/slow"},
         chrome.test.callbackPass(function(id) {
           chrome.test.assertEq(getNextId(), id);
       }));
       // TODO(benjhayden): Test the filename using onChanged.
     },
     function downloadInvalidFilename() {
       chrome.experimental.downloads.download(
-        {"url": getURL("pass"), "filename": "../../../../../etc/passwd"},
+        {"url": getURL("slow?0"), "filename": "../../../../../etc/passwd"},
         chrome.test.callbackFail("I'm afraid I can't do that."));
       // TODO(benjhayden): Give a better error message.
     },
     function downloadEmpty() {
       assertThrows(chrome.experimental.downloads.download, {},
                    ("Invalid value for argument 1. Property 'url': " +
                     "Property is required."));
     },
     function downloadInvalidSaveAs() {
       assertThrows(chrome.experimental.downloads.download,
-                   {"url": getURL("pass"), "saveAs": "GOAT"},
+                   {"url": getURL("slow?0"), "saveAs": "GOAT"},
                    ("Invalid value for argument 1. Property 'saveAs': " +
                     "Expected 'boolean' but got 'string'."));
     },
     function downloadInvalidHeadersOption() {
       assertThrows(chrome.experimental.downloads.download,
-                   {"url": getURL("pass"), "headers": "GOAT"},
+                   {"url": getURL("slow?0"), "headers": "GOAT"},
                    ("Invalid value for argument 1. Property 'headers': " +
                     "Expected 'array' but got 'string'."));
     },
     function downloadInvalidURL() {
       chrome.experimental.downloads.download(
         {"url": "foo bar"},
         chrome.test.callbackFail("Invalid URL"));
     },
     function downloadInvalidMethod() {
       assertThrows(chrome.experimental.downloads.download,
-                   {"url": getURL("pass"), "method": "GOAT"},
+                   {"url": getURL("slow?0"), "method": "GOAT"},
                    ("Invalid value for argument 1. Property 'method': " +
                     "Value must be one of: [GET, POST]."));
     },
     function downloadSimple() {
       chrome.experimental.downloads.download(
-        {"url": getURL("pass")},
+        {"url": getURL("slow?0")},
         chrome.test.callbackPass(function(id) {
           chrome.test.assertEq(getNextId(), id);
       }));
     },
     function downloadHeader() {
       chrome.experimental.downloads.download(
-        {"url": getURL("pass"),
+        {"url": getURL("slow?0"),
          "headers": [{"name": "Foo", "value": "bar"}]},
         chrome.test.callbackPass(function(id) {
           chrome.test.assertEq(getNextId(), id);
         }));
     },
     function downloadInterrupted() {
+      // TODO(benjhayden): Find a suitable URL and test that this id is
+      // eventually interrupted using onChanged.
       chrome.experimental.downloads.download(
-        {"url": getURL("compressedfiles/Picture_1.doc?L")},
+        {"url": getURL("slow?0")},
         chrome.test.callbackPass(function(id) {
           chrome.test.assertEq(getNextId(), id);
-          // TODO(benjhayden): Test that this id is eventually interrupted using
-          // onChanged.
       }));
     },
     function downloadInvalidHeader() {
       chrome.experimental.downloads.download(
-        {"url": getURL("pass"),
+        {"url": getURL("slow?0"),
          "headers": [{"name": "Cookie", "value": "fake"}]},
         chrome.test.callbackFail("I'm afraid I can't do that."));
       // TODO(benjhayden): Give a better error message.
     }
   ]);
 });