dart:io | Add rename to Link

tests/standalone/io/link_async_test.dart

Index: tests/standalone/io/link_async_test.dart
diff --git a/tests/standalone/io/link_async_test.dart b/tests/standalone/io/link_async_test.dart
index 0cc4a7a739dc051fd9765073342b4443b871af61..ddee7822e5331319edb9ce3f386baa18146f7c70 100644
--- a/tests/standalone/io/link_async_test.dart
+++ b/tests/standalone/io/link_async_test.dart
@@ -164,6 +164,46 @@ Future testCreateLoopingLink() {
 }
 
 
+Future testRename() {
+  Future testRename(Path base, String target) {
+    Link link1;
+    Link link2;
+    return new Link.fromPath(base.append('c')).create(target)
+        .then((link) {
+          link1 = link;
+          Expect.isTrue(link1.existsSync());
+          return link1.rename(base.append('d').toNativePath());
+        })
+        .then((link) {
+          link2 = link;
+          Expect.isFalse(link1.existsSync());
+          Expect.isTrue(link2.existsSync());
+          return link2.delete();
+        })
+        .then((_) => Expect.isFalse(link2.existsSync()));
+  }
+
+  return new Directory('').createTemp()
+      .then((baseDir) {
+          Path base = new Path(baseDir.path);
+          var targetsFutures = [];
+          targetsFutures.add(new Directory.fromPath(base.append('a')).create());
+          if (Platform.isWindows) {
+            // Currently only links to directories are supported on Windows.
+            targetsFutures.add(
+                new Directory.fromPath(base.append('b')).create());
+          } else {
+            targetsFutures.add(new File.fromPath(base.append('b')).create());
+          }
+          return Future.wait(targetsFutures)
+              .then((targets) {
+                  return testRename(base, targets[0].path)
+                      .then((_) => testRename(base, targets[1].path))

[Bill Hesse, 2013/06/26 12:03:54]

Don't we indent zero on a .then?

And actually, the "return" should only be indented 2, right?

Maybe the first .then((targets) { could be on the previous line.

[Søren Gjesse, 2013/06/26 13:59:40]

Done.

+                      .then((_) => baseDir.delete(recursive: true));
+                });
+      });
+}
+
 Future testDirectoryListing(Path base, Directory baseDir) {
   Map makeExpected(bool recursive, bool followLinks) {
     Map expected = new Map();
@@ -219,5 +259,6 @@ main() {
   ReceivePort keepAlive = new ReceivePort();
   testCreate()
   .then((_) => testCreateLoopingLink())
+  .then((_) => testRename())
   .then((_) => keepAlive.close());
 }