---

jean-philippe-martin · garrettjonesgoogle · commit 56e9d92e431f · 2017-02-22T15:43:27.000-08:00
yaml --- r: 7847 b: refs/heads/tswast-patch-1 c: 0fb03eb h: refs/heads/master i: 7845: 65223b7 7843: 1fc6a70 7839: 1add59f
diff --git a/[refs] b/[refs]
@@ -57,5 +57,5 @@ refs/tags/v0.18.0: 9d193c4c4b9d1c6f21515dd8e50836b9194ec9bb
 refs/tags/v0.19.0: e67b56e4d8dad5f9a7b38c9b2107c23c828f2ed5
 refs/tags/v0.20.0: 839f7fb7156535146aa1cb2c5aadd8d375d854e8
 refs/tags/v0.20.1: 370471f437f1f4f68a11e068df5cd6bf39edb1fa
-refs/heads/tswast-patch-1: 235874dc168f0e66c57d5ac4c3004b2b24bf82a4
+refs/heads/tswast-patch-1: 0fb03eb5f6cf62556518566a4b0b27dc516ee7de
 refs/heads/pubsub-streaming-pull: 19262b752ee874eb2ca3b950eb2aef44d5a5267b
diff --git a/branches/tswast-patch-1/google-cloud-contrib/google-cloud-nio/src/main/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemProvider.java b/branches/tswast-patch-1/google-cloud-contrib/google-cloud-nio/src/main/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemProvider.java
@@ -598,7 +598,7 @@ public DirectoryStream<Path> newDirectoryStream(Path dir, final Filter<? super P
     final CloudStoragePath cloudPath = CloudStorageUtil.checkPath(dir);
     checkNotNull(filter);
     initStorage();
-    String prefix = cloudPath.toString();
+    String prefix = cloudPath.toRealPath().toString();
     final Iterator<Blob> blobIterator = storage.list(cloudPath.bucket(),
         Storage.BlobListOption.prefix(prefix), Storage.BlobListOption.currentDirectory(),
         Storage.BlobListOption.fields()).iterateAll();
diff --git a/branches/tswast-patch-1/google-cloud-contrib/google-cloud-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemTest.java b/branches/tswast-patch-1/google-cloud-contrib/google-cloud-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemTest.java
@@ -158,6 +158,13 @@ public void testListFiles() throws IOException {
         got.add(path);
       }
       assertThat(got).containsExactlyElementsIn(goodPaths);
+
+      // Must also work with relative path
+      got.clear();
+      for (Path path : Files.newDirectoryStream(fs.getPath("dir/"))) {
+        got.add(path);
+      }
+      assertThat(got).containsExactlyElementsIn(goodPaths);
     }
   }
 
diff --git a/branches/tswast-patch-1/google-cloud-contrib/google-cloud-nio/src/test/java/com/google/cloud/storage/contrib/nio/it/ITGcsNio.java b/branches/tswast-patch-1/google-cloud-contrib/google-cloud-nio/src/test/java/com/google/cloud/storage/contrib/nio/it/ITGcsNio.java
@@ -347,6 +347,13 @@ public void testListFiles() throws IOException {
       }
 
       List<Path> got = new ArrayList<>();
+      for (Path path : Files.newDirectoryStream(fs.getPath("/dir/"))) {
+        got.add(path);
+      }
+      assertThat(got).containsExactlyElementsIn(goodPaths);
+
+      // Must also work with relative path
+      got.clear();
       for (Path path : Files.newDirectoryStream(fs.getPath("dir/"))) {
         got.add(path);
       }

Original file line number	Diff line number	Diff line change
`@@ -158,6 +158,13 @@ public void testListFiles() throws IOException {`
`158`	`158`	`got.add(path);`
`159`	`159`	`}`
`160`	`160`	`assertThat(got).containsExactlyElementsIn(goodPaths);`
	`161`	`+`
	`162`	`+ // Must also work with relative path`
	`163`	`+ got.clear();`
	`164`	`+ for (Path path : Files.newDirectoryStream(fs.getPath("dir/"))) {`
	`165`	`+ got.add(path);`
	`166`	`+ }`
	`167`	`+ assertThat(got).containsExactlyElementsIn(goodPaths);`
`161`	`168`	`}`
`162`	`169`	`}`
`163`	`170`
Original file line number	Diff line number	Diff line change
`@@ -347,6 +347,13 @@ public void testListFiles() throws IOException {`
`347`	`347`	`}`
`348`	`348`
`349`	`349`	`List<Path> got = new ArrayList<>();`
	`350`	`+ for (Path path : Files.newDirectoryStream(fs.getPath("/dir/"))) {`
	`351`	`+ got.add(path);`
	`352`	`+ }`
	`353`	`+ assertThat(got).containsExactlyElementsIn(goodPaths);`
	`354`	`+`
	`355`	`+ // Must also work with relative path`
	`356`	`+ got.clear();`
`350`	`357`	`for (Path path : Files.newDirectoryStream(fs.getPath("dir/"))) {`
`351`	`358`	`got.add(path);`
`352`	`359`	`}`