Add command-line non-client support to unit test lib.

client/testing/unittest/unittest.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 /**
  * A simple unit test library.
  */
 #library("unittest");
 
 #import("dart:dom");
 
-#source("unittestsuite.dart");
+#source("shared.dart");
+
+// TODO(rnystrom): Get rid of this if we get canonical closures for methods.
+EventListener _onErrorClosure;
+
+_initializePlatform() {

[Siggi Cherem (dart-lang), 2011/10/28 23:35:13]

suggestion - what if we create an abstraction for each platform instead?

Basically I'm thinking having a top-level symbol [_platform] which implements a
[Platform] interface. Then in shared.dart every platform-specific call is of the
form:

  _platform.foo()

rather than 
  _fooPlatform()

We would still do the trick you use here to combine the code for each entry
point, but I think the code in shared.dart will be easier to
understand/maintain.

[Bob Nystrom, 2011/10/28 23:57:48]

When would that field get initialized? Would the Platform just be a const class?
 That feels like more work than I want to do to just swap a couple of functions
out. I do like that your suggestion reads better.

Maybe I should do _platformRunTests() instead of _runTestsPlatform()?

[Siggi Cherem (dart-lang), 2011/10/29 00:13:19]

Another idea ---

if you'd like to avoid adding classes, a third option would be to make a library
with visible top-level functions, but import it with a prefix:

--- file browser_platform.dart:
init() {
...
}

--- file shared.dart:
_runTest() {
  platform.init();
  platform.runAsync(() {...});
}

--- file unittest.dart:
#import('browser_platform', prefix: 'platform');
#source('shared.dart');

[Siggi Cherem (dart-lang), 2011/10/29 00:13:19]

Basically yes, it would be a const class - here is more precisely what I meant.

--- file shared.dart:
interface Platform {
  init(); // == _initializePlatform
  initTests(); // == _runTestPlatform
  runAsync(void callback()); // == _doLaterPlatform
  completeTests(int passed, int failed, int errors);
}

...
_runTest() {
  _platform.initTests();
  _platform.runAsync(() { _nextBatch(); });
}

...
--- file unittest.dart:

#source('shared.dart');
#source('browser_platform.dart');
Platform _platform = new BrowserPlatform();

--- file browser_platform.dart ?:
class BrowserPlatform implements Platform {
 ...
}

+  _onErrorClosure = (e) { _onError(e); };
+}
+
+_doLaterPlatform(void callback()) {
+  window.setTimeout(callback, 0);
+}
+
+void _onError(e) {
+ if (_currentTest < _tests.length) {
+    final testCase = _tests[_currentTest];
+    // TODO(vsm): figure out how to expose the stack trace here
+    // Currently e.message works in dartium, but not in dartc.
+    testCase.recordError('(DOM callback has errors) Caught ${e}', '');
+    _state = _UNCAUGHT_ERROR;
+    if (testCase.callbacks > 0) {
+      _currentTest++;
+      _nextBatch();
+    }
+  }
+}
+
+/** Runs all queued tests, one at a time. */
+_runTestsPlatform() {
+  window.postMessage('unittest-suite-start', '*');
+
+  // Listen for uncaught errors.
+  window.onerror = _onErrorClosure;
+}
+
+_completeTestsPlatform(int testsPassed, int testsFailed, int testsErrors) {
+  window.onerror = null;
+
+  if (_isLayoutTest && testsPassed == _tests.length) {
+    document.body.innerHTML = "PASS";
+  } else {
+    var newBody = new StringBuffer();
+    newBody.add("<table class='unittest-table'><tbody>");
+    newBody.add(testsPassed == _tests.length
+        ? "<tr><td colspan='3' class='unittest-pass'>PASS</td></tr>"
+        : "<tr><td colspan='3' class='unittest-fail'>FAIL</td></tr>");
+
+    for (final test in _tests) {
+      newBody.add(_toHtml(test));
+    }
+
+    if (testsPassed == _tests.length) {
+      newBody.add("<tr><td colspan='3' class='unittest-pass'>All "
+          + testsPassed + " tests passed</td></tr>");
+    } else {
+      newBody.add("""
+          <tr><td colspan='3'>Total
+            <span class='unittest-pass'>${testsPassed} passed</span>,
+            <span class='unittest-fail'>${testsFailed} failed</span>
+            <span class='unittest-error'>${testsErrors} errors</span>
+          </td></tr>""");
+    }
+    newBody.add("</tbody></table>");
+    document.body.innerHTML = newBody.toString();
+  }
+
+  window.postMessage('unittest-suite-done', '*');
+}
+
+String _toHtml(TestCase test) {
+  if (!test.isComplete) {
+    return '''
+        <tr>
+          <td>${test.id}</td>
+          <td class="unittest-error">NO STATUS</td>
+          <td>Test did not complete</td>
+        </tr>''';
+  }
+
+  var html = '''
+      <tr>
+        <td>${test.id}</td>
+        <td class="unittest-${test.result}">${test.result.toUpperCase()}</td>
+        <td>Expectation: ${test.description}. ${test.message}</td>
+      </tr>''';
+
+  if (test.stackTrace != null) {
+    html +=
+        '<tr><td></td><td colspan="2"><pre>${test.stackTrace}</pre></td></tr>';
+  }
+
+  return html;
+}