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

Unified Diff: tests/lib/unittest/matchers_test.dart

Issue 10545167: Some unit test fixes: (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 8 years, 6 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: tests/lib/unittest/matchers_test.dart
===================================================================
--- tests/lib/unittest/matchers_test.dart (revision 0)
+++ tests/lib/unittest/matchers_test.dart (working copy)
@@ -2,47 +2,13 @@
// 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('unittestTest');
+#library('matcherTest');
+#import('../../../lib/unittest/unittest.dart');
+#source('testers.dart');
-#import('../../lib/unittest/unittest.dart');
-
-
doesNotThrow() {}
doesThrow() { throw 'X'; }
-int errorCount;
-String errorString;
-var testHandler;
-
-class MyFailureHandler extends DefaultFailureHandler {
- void fail(String reason) {
- ++errorCount;
- errorString = reason;
- }
-}
-
-void shouldFail(var value, Matcher matcher, expected) {
- errorCount = 0;
- errorString = '';
- configureExpectHandler(testHandler);
- expect(value, matcher);
- configureExpectHandler(null);
- expect(errorCount, equals(1));
- if (expected is String)
- expect(errorString, equalsIgnoringWhitespace(expected));
- else
- expect(errorString, expected);
-}
-
-void shouldPass(var value, Matcher matcher) {
- errorCount = 0;
- errorString = '';
- configureExpectHandler(testHandler);
- expect(value, matcher);
- configureExpectHandler(null);
- expect(errorCount, equals(0));
-}
-
class PrefixMatcher extends BaseMatcher {
final String _prefix;
const PrefixMatcher(this._prefix);
@@ -59,7 +25,7 @@
void main() {
- testHandler = new MyFailureHandler();
+ initTesters();
var a = new Map();
var b = new Map();
@@ -109,28 +75,12 @@
shouldPass(doesThrow, throws);
});
- test('throwsA', () {
- shouldPass(doesThrow, throwsA(equals('X')));
- shouldFail(doesThrow, throwsA(equals('Y')),
- "Expected: throws an exception which matches 'Y' "
- "but: exception does not match 'Y'");
- });
-
test('returnsNormally', () {
shouldPass(doesNotThrow, returnsNormally);
shouldFail(doesThrow, returnsNormally,
"Expected: return normally but: threw exception");
});
- /* Removing these temporarily while dart2js failure is investigated */
- /*
- test('isInstanceOf', () {
- shouldFail(0, new isInstanceOf<String>('String'),
- "Expected: an instance of String but: was <0>");
- shouldPass('cow', new isInstanceOf<String>('String'));
- });
- */
-
test('hasLength', () {
shouldPass(c, hasLength(0));
shouldPass(a, hasLength(0));

Powered by Google App Engine
This is Rietveld 408576698