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

Unified Diff: client/tests/client/html/DocumentFragmentTests.dart

Issue 9474030: Revert "Add some sweet XML classes." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 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
« no previous file with comments | « client/tests/client/client.status ('k') | client/tests/client/html/ElementTests.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/tests/client/html/DocumentFragmentTests.dart
diff --git a/client/tests/client/html/DocumentFragmentTests.dart b/client/tests/client/html/DocumentFragmentTests.dart
index 3425beda94e11ae62668165790da69d11b6f5d8a..8ca1b89dcc730d8c0b6b8145bb76a37b6c9645d1 100644
--- a/client/tests/client/html/DocumentFragmentTests.dart
+++ b/client/tests/client/html/DocumentFragmentTests.dart
@@ -16,6 +16,15 @@ testDocumentFragment() {
return out;
};
+ assertUnsupported(void fn()) {
+ try {
+ fn();
+ } catch (UnsupportedOperationException e) {
+ return;
+ }
+ Expect.fail('Expected UnsupportedOperationException');
+ };
+
assertConstError(void fn()) {
try {
fn();
@@ -37,47 +46,39 @@ testDocumentFragment() {
// Expect.isNull(style.getPropertyCSSValue('color'));
// Expect.isNull(style.getPropertyShorthand('color'));
// Expect.isFalse(style.isPropertyImplicit('color'));
- expectUnsupported(() => style.cssText = '* {color: blue}');
- expectUnsupported(() => style.removeProperty('color'));
- expectUnsupported(() => style.setProperty('color', 'blue'));
+ assertUnsupported(() => style.cssText = '* {color: blue}');
+ assertUnsupported(() => style.removeProperty('color'));
+ assertUnsupported(() => style.setProperty('color', 'blue'));
}
- group('constructors', () {
- test('0-argument makes an empty fragment', () {
- final fragment = new DocumentFragment();
- Expect.listEquals([], fragment.elements);
- });
-
- test('.html parses input as HTML', () {
- final fragment = new DocumentFragment.html('<a>foo</a>');
- Expect.isTrue(fragment.elements.first is AnchorElement);
- });
-
- test('.svg parses input as SVG', () {
- final fragment = new DocumentFragment.svg('<a>foo</a>');
- Expect.isTrue(fragment.elements.first is SVGAElement);
- });
- });
+ void expectEmptyRect(ClientRect rect) {
+ Expect.equals(0, rect.bottom);
+ Expect.equals(0, rect.top);
+ Expect.equals(0, rect.left);
+ Expect.equals(0, rect.right);
+ Expect.equals(0, rect.height);
+ Expect.equals(0, rect.width);
+ }
test('Unsupported operations throw errors', () {
var emptyFragment = new DocumentFragment();
- expectUnsupported(() => emptyFragment.attributes = {});
- expectUnsupported(() => emptyFragment.classes = []);
- expectUnsupported(() => emptyFragment.dataAttributes = {});
- expectUnsupported(() => emptyFragment.contentEditable = "true");
- expectUnsupported(() => emptyFragment.dir);
- expectUnsupported(() => emptyFragment.dir = "ltr");
- expectUnsupported(() => emptyFragment.draggable = true);
- expectUnsupported(() => emptyFragment.hidden = true);
- expectUnsupported(() => emptyFragment.id = "foo");
- expectUnsupported(() => emptyFragment.lang);
- expectUnsupported(() => emptyFragment.lang = "en");
- expectUnsupported(() => emptyFragment.scrollLeft = 10);
- expectUnsupported(() => emptyFragment.scrollTop = 10);
- expectUnsupported(() => emptyFragment.spellcheck = true);
- expectUnsupported(() => emptyFragment.tabIndex = 5);
- expectUnsupported(() => emptyFragment.title = "foo");
- expectUnsupported(() => emptyFragment.webkitdropzone = "foo");
+ assertUnsupported(() => emptyFragment.attributes = {});
+ assertUnsupported(() => emptyFragment.classes = []);
+ assertUnsupported(() => emptyFragment.dataAttributes = {});
+ assertUnsupported(() => emptyFragment.contentEditable = "true");
+ assertUnsupported(() => emptyFragment.dir);
+ assertUnsupported(() => emptyFragment.dir = "ltr");
+ assertUnsupported(() => emptyFragment.draggable = true);
+ assertUnsupported(() => emptyFragment.hidden = true);
+ assertUnsupported(() => emptyFragment.id = "foo");
+ assertUnsupported(() => emptyFragment.lang);
+ assertUnsupported(() => emptyFragment.lang = "en");
+ assertUnsupported(() => emptyFragment.scrollLeft = 10);
+ assertUnsupported(() => emptyFragment.scrollTop = 10);
+ assertUnsupported(() => emptyFragment.spellcheck = true);
+ assertUnsupported(() => emptyFragment.tabIndex = 5);
+ assertUnsupported(() => emptyFragment.title = "foo");
+ assertUnsupported(() => emptyFragment.webkitdropzone = "foo");
});
group('elements', () {
@@ -311,9 +312,9 @@ testDocumentFragment() {
test('setters throw errors', () {
var style = new DocumentFragment().style;
- expectUnsupported(() => style.cssText = '* {color: blue}');
- expectUnsupported(() => style.removeProperty('color'));
- expectUnsupported(() => style.setProperty('color', 'blue'));
+ assertUnsupported(() => style.cssText = '* {color: blue}');
+ assertUnsupported(() => style.removeProperty('color'));
+ assertUnsupported(() => style.setProperty('color', 'blue'));
});
// TODO(nweiz): re-enable when const is better supported in dartc and/or frog
« no previous file with comments | « client/tests/client/client.status ('k') | client/tests/client/html/ElementTests.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698