---

jean-philippe-martin · garrettjonesgoogle · commit caefa34ecb2e · 2017-02-22T15:43:27.000-08:00
yaml --- r: 5399 b: refs/heads/master c: 0fb03eb h: refs/heads/master i: 5397: f112231 5395: bc3e095 5391: 0ec83a6
diff --git a/[refs] b/[refs]
@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 235874dc168f0e66c57d5ac4c3004b2b24bf82a4
+refs/heads/master: 0fb03eb5f6cf62556518566a4b0b27dc516ee7de
 refs/heads/travis: dae77e558b884bc1b165155482d76c8e40b0fca4
 refs/heads/gh-pages: 229631582f8957646f81e92ae5a326504f48ee5b
 refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
diff --git a/trunk/google-cloud-contrib/google-cloud-nio/src/main/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemProvider.java b/trunk/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/trunk/google-cloud-contrib/google-cloud-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemTest.java b/trunk/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/trunk/google-cloud-contrib/google-cloud-nio/src/test/java/com/google/cloud/storage/contrib/nio/it/ITGcsNio.java b/trunk/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`	`}`