format code, removed unused variables and deprecated usage

test/args_test.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // 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 args_test;
 
 import 'package:unittest/unittest.dart';
 import 'package:args/args.dart';
 import 'utils.dart';
 
@@ skipping 11 matching lines @@
       throwsIllegalArg(() => parser.addFlag('foo'));
     });
 
     test('throws ArgumentError if the abbreviation exists', () {
       var parser = new ArgParser();
       parser.addFlag('foo', abbr: 'f');
       throwsIllegalArg(() => parser.addFlag('flummox', abbr: 'f'));
     });
 
     test('throws ArgumentError if the abbreviation is longer '
-         'than one character', () {
+        'than one character', () {
       var parser = new ArgParser();
       throwsIllegalArg(() => parser.addFlag('flummox', abbr: 'flu'));
     });
 
     test('throws ArgumentError if a flag name is invalid', () {
       var parser = new ArgParser();
 
-      for(var name in _INVALID_OPTIONS) {
+      for (var name in _INVALID_OPTIONS) {
         var reason = '${Error.safeToString(name)} is not valid';
         throwsIllegalArg(() => parser.addFlag(name), reason: reason);
       }
     });
 
     test('accepts valid flag names', () {
       var parser = new ArgParser();
 
-      for(var name in _VALID_OPTIONS) {
+      for (var name in _VALID_OPTIONS) {
         var reason = '${Error.safeToString(name)} is valid';
         expect(() => parser.addFlag(name), returnsNormally, reason: reason);
       }
     });
   });
 
   group('ArgParser.addOption()', () {
     test('throws ArgumentError if the flag already exists', () {
       var parser = new ArgParser();
       parser.addFlag('foo');
       throwsIllegalArg(() => parser.addOption('foo'));
     });
 
     test('throws ArgumentError if the option already exists', () {
       var parser = new ArgParser();
       parser.addOption('foo');
       throwsIllegalArg(() => parser.addOption('foo'));
     });
 
     test('throws ArgumentError if the abbreviation exists', () {
       var parser = new ArgParser();
       parser.addFlag('foo', abbr: 'f');
       throwsIllegalArg(() => parser.addOption('flummox', abbr: 'f'));
     });
 
     test('throws ArgumentError if the abbreviation is longer '
-         'than one character', () {
+        'than one character', () {
       var parser = new ArgParser();
       throwsIllegalArg(() => parser.addOption('flummox', abbr: 'flu'));
     });
 
     test('throws ArgumentError if the abbreviation is empty', () {
       var parser = new ArgParser();
       throwsIllegalArg(() => parser.addOption('flummox', abbr: ''));
     });
 
     test('throws ArgumentError if the abbreviation is an invalid value', () {
       var parser = new ArgParser();
-      for(var name in _INVALID_OPTIONS.where((v) => v != null)) {
+      for (var name in _INVALID_OPTIONS.where((v) => v != null)) {
         throwsIllegalArg(() => parser.addOption('flummox', abbr: name));
       }
     });
 
     test('throws ArgumentError if the abbreviation is a dash', () {
       var parser = new ArgParser();
       throwsIllegalArg(() => parser.addOption('flummox', abbr: '-'));
     });
 
     test('allows explict null value for "abbr"', () {
       var parser = new ArgParser();
       expect(() => parser.addOption('flummox', abbr: null), returnsNormally);
     });
 
     test('throws ArgumentError if an option name is invalid', () {
       var parser = new ArgParser();
 
-      for(var name in _INVALID_OPTIONS) {
+      for (var name in _INVALID_OPTIONS) {
         var reason = '${Error.safeToString(name)} is not valid';
         throwsIllegalArg(() => parser.addOption(name), reason: reason);
       }
     });
 
     test('accepts valid option names', () {
       var parser = new ArgParser();
 
-      for(var name in _VALID_OPTIONS) {
+      for (var name in _VALID_OPTIONS) {
         var reason = '${Error.safeToString(name)} is valid';
         expect(() => parser.addOption(name), returnsNormally, reason: reason);
       }
     });
   });
 
   group('ArgParser.getDefault()', () {
     test('returns the default value for an option', () {
       var parser = new ArgParser();
       parser.addOption('mode', defaultsTo: 'debug');
@@ skipping 153 matching lines @@
       expect(parser.options['single-def'].getOrDefault('v'), equals('v'));
       expect(parser.options['multi-no'].getOrDefault(null), equals([]));
       expect(parser.options['multi-no'].getOrDefault(['v']), equals(['v']));
       expect(parser.options['multi-def'].getOrDefault(null), equals(['def']));
       expect(parser.options['multi-def'].getOrDefault(['v']), equals(['v']));
     });
   });
 }
 
 const _INVALID_OPTIONS = const [
- ' ', '', '-', '--', '--foo',
- ' with space',
- 'with\ttab',
- 'with\rcarriage\rreturn',
- 'with\nline\nfeed',
- "'singlequotes'",
- '"doublequotes"',
- 'back\\slash',
- 'forward/slash'
+  ' ',
+  '',
+  '-',
+  '--',
+  '--foo',
+  ' with space',
+  'with\ttab',
+  'with\rcarriage\rreturn',
+  'with\nline\nfeed',
+  "'singlequotes'",
+  '"doublequotes"',
+  'back\\slash',
+  'forward/slash'
 ];
 
 const _VALID_OPTIONS = const [
- 'a' // One character.
- 'contains-dash',
- 'contains_underscore',
- 'ends-with-dash-',
- 'contains--doubledash--',
- '1starts-with-number',
- 'contains-a-1number',
- 'ends-with-a-number8'
+  'a' // One character.
+      'contains-dash',

[Siggi Cherem (dart-lang), 2015/01/14 01:37:47]

oh, weird, can you file a bug in dart_style about this one?

[kevmoo, 2015/01/14 01:47:47]

https://github.com/dart-lang/dart_style/issues/138

+  'contains_underscore',
+  'ends-with-dash-',
+  'contains--doubledash--',
+  '1starts-with-number',
+  'contains-a-1number',
+  'ends-with-a-number8'
 ];