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

Side by Side Diff: lib/core/expect.dart

Issue 11275042: Renaming IndexOutOfRangeException to RangeError. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: ADded RangeError.value constructor. Made VM calls work. Created 8 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 /** 5 /**
6 * Expect is used for tests that do not want to make use of the 6 * Expect is used for tests that do not want to make use of the
7 * Dart unit test library - for example, the core language tests. 7 * Dart unit test library - for example, the core language tests.
8 * Third parties are discouraged from using this, and should use 8 * Third parties are discouraged from using this, and should use
9 * the expect() function in the unit test library instead for 9 * the expect() function in the unit test library instead for
10 * test assertions. 10 * test assertions.
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 * that the correct exception is being thrown. For example, to check 257 * that the correct exception is being thrown. For example, to check
258 * the type of the exception you could write this: 258 * the type of the exception you could write this:
259 * 259 *
260 * Expect.throws(myThrowingFunction, (e) => e is MyException); 260 * Expect.throws(myThrowingFunction, (e) => e is MyException);
261 */ 261 */
262 static void throws(void f(), 262 static void throws(void f(),
263 [_CheckExceptionFn check = null, 263 [_CheckExceptionFn check = null,
264 String reason = null]) { 264 String reason = null]) {
265 try { 265 try {
266 f(); 266 f();
267 } catch (e) { 267 } catch (e, s) {
268 if (check !== null) { 268 if (check !== null) {
269 Expect.isTrue(check(e)); 269 if (!check(e)) {
270 String msg = reason == null ? "" : reason;
floitsch 2012/11/01 12:11:39 why not just change "reason" to have a default val
Lasse Reichstein Nielsen 2012/11/01 13:39:09 I believe I have seen places where one check is ne
271 _fail("Expect.throws($msg): Unexpected '$e'\n$s");
272 }
270 } 273 }
271 return; 274 return;
272 } 275 }
273 String msg = _getMessage(reason); 276 String msg = reason == null ? "" : reason;
274 _fail('Expect.throws($msg) fails'); 277 _fail('Expect.throws($msg) fails');
275 } 278 }
276 279
277 static String _getMessage(String reason) 280 static String _getMessage(String reason)
278 => (reason === null) ? "" : ", '$reason'"; 281 => (reason === null) ? "" : ", '$reason'";
279 282
280 static void _fail(String message) { 283 static void _fail(String message) {
281 throw new ExpectException(message); 284 throw new ExpectException(message);
282 } 285 }
283 } 286 }
284 287
285 typedef bool _CheckExceptionFn(exception); 288 typedef bool _CheckExceptionFn(exception);
286 289
287 class ExpectException implements Exception { 290 class ExpectException implements Exception {
288 ExpectException(this.message); 291 ExpectException(this.message);
289 String toString() => message; 292 String toString() => message;
290 String message; 293 String message;
291 } 294 }
OLDNEW
« no previous file with comments | « lib/core/exceptions.dart ('k') | lib/core/list.dart » ('j') | lib/html/dart2js/html_dart2js.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698