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

Unified Diff: pkg/dev_compiler/test/codegen/lib/html/hidden_dom_1_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/hidden_dom_1_test.dart
diff --git a/pkg/dev_compiler/test/codegen/lib/html/hidden_dom_1_test.dart b/pkg/dev_compiler/test/codegen/lib/html/hidden_dom_1_test.dart
index 120136b5a3da6069887ad8d67cd68acb304979d3..390b5103a92da947087e1fea5e023e29d2f189cf 100644
--- a/pkg/dev_compiler/test/codegen/lib/html/hidden_dom_1_test.dart
+++ b/pkg/dev_compiler/test/codegen/lib/html/hidden_dom_1_test.dart
@@ -1,14 +1,11 @@
-library HiddenDom1Test;
-import 'package:unittest/unittest.dart';
-import 'package:unittest/html_config.dart';
import 'dart:html';
+import 'package:minitest/minitest.dart';
+
// Test that the dart:html API does not leak native jsdom methods:
// onfocus setter.
main() {
- useHtmlConfiguration();
-
test('test1', () {
document.body.children.add(new Element.html(r'''
<div id='div1'>
@@ -17,7 +14,7 @@ Hello World!
Element e = document.query('#div1');
expect(e, isNotNull);
- checkNoSuchMethod(() { confuse(e).onfocus = null; });
+ expect(() { confuse(e).onfocus = null; }, throwsNoSuchMethodError);
});
}
@@ -28,18 +25,5 @@ class Decoy {
confuse(x) => opaqueTrue() ? x : (opaqueTrue() ? new Object() : new Decoy());
-/** Returns [:true:], but in a way that confuses the compiler. */
+/** Returns `true`, but in a way that confuses the compiler. */
opaqueTrue() => true; // Expand as needed.
-
-checkNoSuchMethod(action()) {
- var ex = null;
- try {
- action();
- } catch (e) {
- ex = e;
- }
- if (ex == null)
- expect(false, isTrue, reason: 'Action should have thrown exception');
-
- expect(ex, isNoSuchMethodError);
-}

Powered by Google App Engine
This is Rietveld 408576698