Arg parsing tests

test/cli_util_test.dart

Index: test/cli_util_test.dart
diff --git a/test/cli_util_test.dart b/test/cli_util_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..b7b78228a0bf5b43afdac3750ac0c1a2bbfbc316
--- /dev/null
+++ b/test/cli_util_test.dart
@@ -0,0 +1,21 @@
+// Copyright (c) 2015, 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.
+
+import 'package:cli_util/cli_util.dart';
+import 'package:unittest/unittest.dart';
+
+void defineTests() {
+  group('getSdkDir', () {
+    test('arg parsing', () {
+      expect(getSdkDir(['--dart-sdk', '/dart/sdk']).path, equals('/dart/sdk'));
+      expect(getSdkDir(['--dart-sdk=/dart/sdk']).path, equals('/dart/sdk'));
+    });
+  });
+}
+
+main() {
+  groupSep = ' | ';

[devoncarew, 2015/02/04 05:09:33]

Curious, why are you setting this?

[pquitslund, 2015/02/04 05:27:29]

Convention I picked up, I think, from the analyzer tests.  The default is a
space maybe?  Setting it like so yields results like this:

PASS: getSdkDir | arg parsing

Making it easy to see the hierarchy...  In the end though it's more habit than
anything.  Happy to revisit! :)

+
+  defineTests();
+}