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

Unified Diff: test/runner/browser/runner_test.dart

Issue 983573002: Add support for running browser tests to the test runner. (Closed) Base URL: git@github.com:dart-lang/unittest@master
Patch Set: Created 5 years, 10 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
Index: test/runner/browser/runner_test.dart
diff --git a/test/runner/browser/runner_test.dart b/test/runner/browser/runner_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..1fbaf7ad9a42a5a31e83fcb60cdbc11086bb31b5
--- /dev/null
+++ b/test/runner/browser/runner_test.dart
@@ -0,0 +1,161 @@
+// Copyright (c) 2015, 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.
+
+import 'dart:io';
+
+import 'package:path/path.dart' as p;
+import 'package:unittest/src/util/exit_codes.dart' as exit_codes;
+import 'package:unittest/unittest.dart';
+
+import '../../io.dart';
+
+String _sandbox;
+
+final _success = """
+import 'dart:async';
+
+import 'package:unittest/unittest.dart';
+
+void main() {
+ test("success", () {});
+}
+""";
+
+void main() {
+ setUp(() {
+ _sandbox = Directory.systemTemp.createTempSync('unittest_').path;
+ });
+
+ tearDown(() {
+ new Directory(_sandbox).deleteSync(recursive: true);
+ });
+
+ group("fails gracefully if", () {
+ test("a test file fails to compile", () {
+ var testPath = p.join(_sandbox, "test.dart");
+ new File(testPath).writeAsStringSync("invalid Dart file");
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+
+ expect(result.stdout,
+ contains("Expected a declaration, but got 'invalid'"));
+ expect(result.stderr, equals(
+ 'Failed to load "${p.relative(testPath, from: _sandbox)}": dart2js '
+ 'failed.\n'));
+ expect(result.exitCode, equals(exit_codes.data));
+ });
+
+ test("a test file throws", () {
+ var testPath = p.join(_sandbox, "test.dart");
+ new File(testPath).writeAsStringSync("void main() => throw 'oh no';");
+
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+ expect(result.stderr, startsWith(
+ 'Failed to load "${p.relative(testPath, from: _sandbox)}": oh no\n'));
+ expect(result.exitCode, equals(exit_codes.data));
+ });
+
+ test("a test file doesn't have a main defined", () {
+ var testPath = p.join(_sandbox, "test.dart");
+ new File(testPath).writeAsStringSync("void foo() {}");
+
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+ expect(result.stderr, startsWith(
+ 'Failed to load "${p.relative(testPath, from: _sandbox)}": No '
+ 'top-level main() function defined.\n'));
+ expect(result.exitCode, equals(exit_codes.data));
+ });
+
+ test("a test file has a non-function main", () {
+ var testPath = p.join(_sandbox, "test.dart");
+ new File(testPath).writeAsStringSync("int main;");
+
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+ expect(result.stderr, startsWith(
+ 'Failed to load "${p.relative(testPath, from: _sandbox)}": Top-level '
+ 'main getter is not a function.\n'));
+ expect(result.exitCode, equals(exit_codes.data));
+ });
+
+ test("a test file has a main with arguments", () {
+ var testPath = p.join(_sandbox, "test.dart");
+ new File(testPath).writeAsStringSync("void main(arg) {}");
+
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+ expect(result.stderr, startsWith(
+ 'Failed to load "${p.relative(testPath, from: _sandbox)}": Top-level '
+ 'main() function takes arguments.\n'));
+ expect(result.exitCode, equals(exit_codes.data));
+ });
+
+ // TODO(nweiz): test what happens when a test file is unreadable once issue
+ // 15078 is fixed.
+ });
+
+ group("runs successful tests", () {
+ test("on the browser only", () {
+ new File(p.join(_sandbox, "test.dart")).writeAsStringSync(_success);
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+ expect(result.exitCode, equals(0));
+ });
+
+ test("on the browser and the VM", () {
+ new File(p.join(_sandbox, "test.dart")).writeAsStringSync(_success);
+ var result = _runUnittest(["-p", "chrome", "-p", "vm", "test.dart"]);
+ expect(result.exitCode, equals(0));
+ });
+ });
+
+ group("runs failing tests", () {
+ test("on the browser only", () {
+ new File(p.join(_sandbox, "test.dart")).writeAsStringSync("""
+import 'dart:async';
+
+import 'package:unittest/unittest.dart';
+
+void main() {
+ test("failure", () => throw new TestFailure("oh no"));
+}
+""");
+ var result = _runUnittest(["-p", "chrome", "test.dart"]);
+ expect(result.exitCode, equals(1));
+ });
+
+ test("that fail only on the browser", () {
+ new File(p.join(_sandbox, "test.dart")).writeAsStringSync("""
+import 'dart:async';
+
+import 'package:path/path.dart' as p;
+import 'package:unittest/unittest.dart';
+
+void main() {
+ test("test", () {
+ if (p.style == p.Style.url) throw new TestFailure("oh no");
+ });
+}
+""");
+ var result = _runUnittest(["-p", "chrome", "-p", "vm", "test.dart"]);
+ expect(result.exitCode, equals(1));
+ });
+
+ test("that fail only on the VM", () {
+ new File(p.join(_sandbox, "test.dart")).writeAsStringSync("""
+import 'dart:async';
+
+import 'package:path/path.dart' as p;
+import 'package:unittest/unittest.dart';
+
+void main() {
+ test("test", () {
+ if (p.style != p.Style.url) throw new TestFailure("oh no");
+ });
+}
+""");
+ var result = _runUnittest(["-p", "chrome", "-p", "vm", "test.dart"]);
+ expect(result.exitCode, equals(1));
+ });
+ });
+}
+
+ProcessResult _runUnittest(List<String> args) =>
+ runUnittest(args, workingDirectory: _sandbox);

Powered by Google App Engine
This is Rietveld 408576698