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

Unified Diff: pkg/dev_compiler/test/codegen/lib/html/event_test.dart

Issue 2413073002: Start cleaning up the HTML tests. (Closed)
Patch Set: Unfork expect.dart. Created 4 years, 2 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: pkg/dev_compiler/test/codegen/lib/html/event_test.dart
diff --git a/pkg/dev_compiler/test/codegen/lib/html/event_test.dart b/pkg/dev_compiler/test/codegen/lib/html/event_test.dart
index 14c92f1db563f2e468a96d654cc75c497ff850aa..9f82e25742564b89a3d4c18a8318a2cf2c6a4a15 100644
--- a/pkg/dev_compiler/test/codegen/lib/html/event_test.dart
+++ b/pkg/dev_compiler/test/codegen/lib/html/event_test.dart
@@ -2,14 +2,10 @@
// 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.
-library EventTest;
-import "package:expect/expect.dart";
-import 'package:unittest/unittest.dart';
-import 'package:unittest/html_config.dart';
import 'dart:html';
-// TODO(nweiz): Make this private to testEvents when Frog supports closures with
-// optional arguments.
+import 'package:minitest/minitest.dart';
+
eventTest(String name, Event eventFn(), void validate(Event),
[String type = 'foo']) {
test(name, () {
@@ -25,8 +21,6 @@ eventTest(String name, Event eventFn(), void validate(Event),
}
main() {
- useHtmlConfiguration();
-
// Issue 1005.
// eventTest('AnimationEvent', () => new AnimationEvent('foo', 'color', 0.5),
// (ev) {
@@ -101,7 +95,7 @@ main() {
eventTest('MouseEvent',
() => new MouseEvent('foo', view: window, detail: 1, screenX: 2,
- screenY: 3, clientX: 4, clientY: 5, button: 6,
+ screenY: 3, clientX: 4, clientY: 5, button: 6,
ctrlKey: true, altKey: true, shiftKey: true,
metaKey: true, relatedTarget: document.body),
(ev) {

Powered by Google App Engine
This is Rietveld 408576698