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

Unified Diff: tests/lib/args/args_test.dart

Issue 10850034: Rename BadNumberFormatException -> FormatException. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Rebase. Created 8 years, 4 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 | « tests/language/arithmetic_test.dart ('k') | tests/lib/math/math_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/lib/args/args_test.dart
diff --git a/tests/lib/args/args_test.dart b/tests/lib/args/args_test.dart
index f8cb3906ab1c31d98f2d96f49305d56a5bbc2e37..997cb5f5ce6eb91705aba06089cde1a50e9d1cd1 100644
--- a/tests/lib/args/args_test.dart
+++ b/tests/lib/args/args_test.dart
@@ -92,7 +92,7 @@ main() {
var parser = new ArgParser();
parser.addFlag('verbose');
- throwsBadFormat(parser, ['--verbose=true']);
+ throwsFormat(parser, ['--verbose=true']);
});
});
@@ -127,7 +127,7 @@ main() {
var parser = new ArgParser();
parser.addFlag('strum', negatable: false);
- throwsBadFormat(parser, ['--no-strum']);
+ throwsFormat(parser, ['--no-strum']);
});
});
@@ -248,14 +248,14 @@ main() {
test('throw if unknown', () {
var parser = new ArgParser();
- throwsBadFormat(parser, ['-f']);
+ throwsFormat(parser, ['-f']);
});
test('throw if the value is missing', () {
var parser = new ArgParser();
parser.addOption('file', abbr: 'f');
- throwsBadFormat(parser, ['-f']);
+ throwsFormat(parser, ['-f']);
});
test('throw if the value looks like an option', () {
@@ -263,16 +263,16 @@ main() {
parser.addOption('file', abbr: 'f');
parser.addOption('other');
- throwsBadFormat(parser, ['-f', '--other']);
- throwsBadFormat(parser, ['-f', '--unknown']);
- throwsBadFormat(parser, ['-f', '-abbr']);
+ throwsFormat(parser, ['-f', '--other']);
+ throwsFormat(parser, ['-f', '--unknown']);
+ throwsFormat(parser, ['-f', '-abbr']);
});
test('throw if the value is not allowed', () {
var parser = new ArgParser();
parser.addOption('mode', abbr: 'm', allowed: ['debug', 'release']);
- throwsBadFormat(parser, ['-mprofile']);
+ throwsFormat(parser, ['-mprofile']);
});
test('throw if any but the first is not a flag', () {
@@ -281,7 +281,7 @@ main() {
parser.addOption('banana', abbr: 'b'); // Takes an argument.
parser.addFlag('cherry', abbr: 'c');
- throwsBadFormat(parser, ['-abc']);
+ throwsFormat(parser, ['-abc']);
});
test('throw if it has a value but the option is a flag', () {
@@ -290,7 +290,7 @@ main() {
parser.addFlag('banana', abbr: 'b');
// The '?!' means this can only be understood as '--apple b?!c'.
- throwsBadFormat(parser, ['-ab?!c']);
+ throwsFormat(parser, ['-ab?!c']);
});
});
@@ -325,14 +325,14 @@ main() {
test('throw if unknown', () {
var parser = new ArgParser();
- throwsBadFormat(parser, ['--unknown']);
- throwsBadFormat(parser, ['--nobody']); // Starts with "no".
+ throwsFormat(parser, ['--unknown']);
+ throwsFormat(parser, ['--nobody']); // Starts with "no".
});
test('throw if the arg does not include a value', () {
var parser = new ArgParser();
parser.addOption('mode');
- throwsBadFormat(parser, ['--mode']);
+ throwsFormat(parser, ['--mode']);
});
test('throw if the value looks like an option', () {
@@ -340,9 +340,9 @@ main() {
parser.addOption('mode');
parser.addOption('other');
- throwsBadFormat(parser, ['--mode', '--other']);
- throwsBadFormat(parser, ['--mode', '--unknown']);
- throwsBadFormat(parser, ['--mode', '-abbr']);
+ throwsFormat(parser, ['--mode', '--other']);
+ throwsFormat(parser, ['--mode', '--unknown']);
+ throwsFormat(parser, ['--mode', '-abbr']);
});
test('do not throw if the value is in the allowed set', () {
@@ -355,7 +355,7 @@ main() {
test('throw if the value is not in the allowed set', () {
var parser = new ArgParser();
parser.addOption('mode', allowed: ['debug', 'release']);
- throwsBadFormat(parser, ['--mode=profile']);
+ throwsFormat(parser, ['--mode=profile']);
});
});
@@ -560,9 +560,8 @@ throwsIllegalArg(function) {
expect(function, throwsA(new isInstanceOf<IllegalArgumentException>()));
}
-throwsBadFormat(ArgParser parser, List<String> args) {
- expect(() => parser.parse(args),
- throwsA(new isInstanceOf<ArgFormatException>()));
+throwsFormat(ArgParser parser, List<String> args) {
+ expect(() => parser.parse(args), throwsFormatException);
}
validateUsage(ArgParser parser, String expected) {
« no previous file with comments | « tests/language/arithmetic_test.dart ('k') | tests/lib/math/math_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698