Convert asynchronous closure test in cr.ui framework to a browser test.

chrome/test/data/webui/webui_resource_test.js

 // Copyright 2013 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.
 
 /**
+ * List of test cases.
+ * @type {Array.<strring>}  List of function names for tests to run.

[arv (Not doing code reviews), 2013/06/20 17:45:15]

Do we want to hide these in an IIFE?

[kevers, 2013/06/20 18:22:12]

Done.

+ */
+var testCases = [];
+
+/**
+ * Indicates if all tests have run successfully.
+ * @type {boolean}
+ */
+var cleanTestRun = true;
+
+/**
+ * Armed during setup of a test to call the matching tear down code.
+ * @type {?Function}
+ */
+var pendingTearDown = null;
+
+/**
  * Tests that an observation matches the expected value.
  * @param {Object} expected The expected value.
  * @param {Object} observed The actual value.
  * @param {string=} opt_message Optional message to include with a test
  *     failure.
  */
 function assertEquals(expected, observed, opt_message) {
   if (observed !== expected) {
     var message = 'Assertion Failed\n  Observed: ' + observed +
         '\n  Expected: ' + expected;
@@ skipping 34 matching lines @@
   if (observed === reference) {
     var message = 'Assertion Failed\n  Observed: ' + observed +
         '\n  Reference: ' + reference;
     if (opt_message)
       message = message + '\n  ' + opt_message;
     throw new Error(message);
   }
 }
 
 /**
- * Verifies that a test evaluation results in an assertion failure.
+ * Verifies that a test evaluation results in an exception.
  * @param {!Function} f The test function.
  */
 function assertThrows(f) {
   var triggeredError = false;
   try {
     f();
   } catch(err) {
     triggeredError = true;
   }
   if (!triggeredError)
     throw new Error('Assertion Failed: throw expected.');
 }
 
 /**
+ * Verifies that the contents of the expected and observed arrays match.
+ * @param {Array} expected The expected result.
+ * @param {Array} observed The actual result.
+ */
+function assertArrayEquals(expected, observed) {
+  var v1 = Array.prototype.slice.call(expected);
+  var v2 = Array.prototype.slice.call(observed);
+  var equal = v1.length == v2.length;
+  if (equal) {
+    for (var i = 0; i < v1.length; i++) {
+      if (v1[i] !== v2[i]) {
+        equal = false;
+        break;
+      }
+    }
+  }
+  if (!equal) {
+    var message = 'Assertion Failed\n  Observed: ' + v1 +
+        '\n  Expected: ' + v2;
+    throw new Error(message);
+  }
+}
+
+/**
  * Runs all functions starting with test and reports success or
  * failure of the test suite.
  */
 function runTests() {
-  var tests = [];
-
-  if (window.setUp)
-    window.setUp();
-
-  var success = true;
   for (var name in window) {
     if (typeof window[name] == 'function' && /^test/.test(name))
-      tests.push(name);
+      testCases.push(name);
   }
-  if (!tests.length) {
+  if (!testCases.length) {
     console.error('\nFailed to find test cases.');
-    success = false;
+    cleanTestRun = false;
   }
-  for (var i = 0; i < tests.length; i++) {
-    try {
-      window[tests[i]]();
-    } catch (err) {
-      console.error('Failure in test ' + tests[i] + '\n' + err);
-      console.log(err.stack);
-      success = false;
-    }
-  }
-
-  if (window.tearDown)
-    window.tearDown();
-
-  endTests(success);
+  continueTesting();
 }
 
 /**
+ * Runs the next test in the queue.  Reports the test results if the queue is
+ * empty.
+ */
+function continueTesting() {

[arv (Not doing code reviews), 2013/06/20 17:45:15]

Does continueTesting need to be a global?

[kevers, 2013/06/20 18:22:12]

Mo longer a global.

+  if (pendingTearDown) {
+    pendingTearDown();
+    pendingTearDown = null;
+  }
+  if(testCases.length > 0) {

[arv (Not doing code reviews), 2013/06/20 17:45:15]

if (

[kevers, 2013/06/20 18:22:12]

Done.

+    var fn = testCases.pop();
+    var isAsyncTest = /Async$/.test(fn);

[arv (Not doing code reviews), 2013/06/20 17:45:15]

I still prefer fn.length instead

[kevers, 2013/06/20 18:22:12]

Done.

+    try {
+      if (window.setUp)
+        window.setUp();
+      pendingTearDown = window.tearDown;
+      window[fn](continueTesting);
+    } catch(err) {
+      console.error('Failure in test ' + fn + '\n' + err);
+      console.log(err.stack);
+      cleanTestRun = false;
+    }
+    // Asynchronous tests must manually call continueTesting when complete.
+    if (!isAsyncTest)
+      continueTesting();
+  } else {
+    endTests(cleanTestRun);
+  }
+  if (testCases.length) {
+    domAutomationController.setAutomationId(1);
+    domAutomationController.send('PENDING');
+  }
+}
+
+/**
  * Signals completion of a test.
  * @param {boolean} success Indicates if the test completed successfully.
  */
 function endTests(success) {
   domAutomationController.setAutomationId(1);
   domAutomationController.send(success ? 'SUCCESS' : 'FAILURE');
 }
 
 window.onerror = function() {
   endTests(false);
 };