Add path.joinAll.

pkg/path/test/path_posix_test.dart

Index: pkg/path/test/path_posix_test.dart
diff --git a/pkg/path/test/path_posix_test.dart b/pkg/path/test/path_posix_test.dart
index e1c39421dd7adcfd5919e2819ca2f71cb5f6b824..c84da81566b74c4e541253c034016dfc705ace88 100644
--- a/pkg/path/test/path_posix_test.dart
+++ b/pkg/path/test/path_posix_test.dart
@@ -161,6 +161,30 @@ main() {
     });
   });
 
+  group('joinAll', () {
+    test('allows more than eight parts', () {
+      expect(builder.joinAll(['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i']),
+          'a/b/c/d/e/f/g/h/i');
+    });
+
+    test('does not add separator if a part ends in one', () {
+      expect(builder.joinAll(['a/', 'b', 'c/', 'd']), 'a/b/c/d');
+      expect(builder.joinAll(['a\\', 'b']), r'a\/b');
+    });
+
+    test('ignores parts before an absolute path', () {
+      expect(builder.joinAll(['a', '/', 'b', 'c']), '/b/c');
+      expect(builder.joinAll(['a', '/b', '/c', 'd']), '/c/d');
+      expect(builder.joinAll(['a', r'c:\b', 'c', 'd']), r'a/c:\b/c/d');
+      expect(builder.joinAll(['a', r'\\b', 'c', 'd']), r'a/\\b/c/d');
+    });
+
+    test('disallows intermediate nulls', () {

[Bob Nystrom, 2013/02/21 22:03:27]

I think it should disallow trailing nulls too.

[nweiz, 2013/02/21 22:19:25]

I actually meant to remove this test. I don't think it should explicitly handle
nulls at all, any more than any other method that accepts a conceptually
non-nullable argument.

+      expect(() => builder.joinAll(['a', null, 'b']), throwsArgumentError);
+      expect(() => builder.joinAll([null, 'a']), throwsArgumentError);
+    });
+  });
+
   group('split', () {
     test('simple cases', () {
       expect(builder.split(''), []);