---

jean-philippe-martin · mziccard · commit 63309c95a33d · 2016-06-30T09:13:05.000+02:00
yaml --- r: 3471 b: refs/heads/master c: 312565c h: refs/heads/master i: 3469: 563e074 3467: 51817ee 3463: d9bb731 3455: 79922e8
diff --git a/[refs] b/[refs]
@@ -1,5 +1,5 @@
 ---
-refs/heads/master: 29ea8b0df2b5efd4159cdae38b581497efe1be7d
+refs/heads/master: 312565c38c47fbf5f1b5e26a46be2546056773fd
 refs/heads/travis: e21ee7b88a5edc3f3d8c71f90c3fc32abf7e8dd6
 refs/heads/gh-pages: 4e0561bb4504bf647db669a14417b2b2c87ba45d
 refs/heads/pubsub-alpha: 1a0e970f265af871e02274085b9662b3fe29058b
diff --git a/trunk/gcloud-java-contrib/gcloud-java-nio/src/main/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystem.java b/trunk/gcloud-java-contrib/gcloud-java-nio/src/main/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystem.java
@@ -158,10 +158,12 @@ public CloudStoragePath getPath(String first, String... more) {
    * Does nothing.
    */
   @Override
-  public void close() {}
+  public void close() throws IOException {
+    // TODO(#809): Synchronously close all channels associated with this FileSystem instance.
+  }
 
   /**
-   * Returns {@code true}.
+   * Returns {@code true}, even if you previously called the {@link #close()} method.
    */
   @Override
   public boolean isOpen() {
diff --git a/trunk/gcloud-java-contrib/gcloud-java-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemProviderTest.java b/trunk/gcloud-java-contrib/gcloud-java-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStorageFileSystemProviderTest.java
@@ -391,7 +391,7 @@ public void testExists_trailingSlash() {
   }
 
   @Test
-  public void testExists_trailingSlash_disablePseudoDirectories() {
+  public void testExists_trailingSlash_disablePseudoDirectories() throws IOException  {
     try (CloudStorageFileSystem fs = forBucket("military", usePseudoDirectories(false))) {
       assertThat(Files.exists(fs.getPath("fashion/"))).isFalse();
     }
@@ -547,7 +547,7 @@ public void testIsDirectory_trailingSlash_alwaysTrue() {
   }
 
   @Test
-  public void testIsDirectory_trailingSlash_pseudoDirectoriesDisabled_false() {
+  public void testIsDirectory_trailingSlash_pseudoDirectoriesDisabled_false() throws IOException  {
     try (CloudStorageFileSystem fs = forBucket("doodle", usePseudoDirectories(false))) {
       assertThat(Files.isDirectory(fs.getPath("fundir/"))).isFalse();
     }
diff --git a/trunk/gcloud-java-contrib/gcloud-java-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStoragePathTest.java b/trunk/gcloud-java-contrib/gcloud-java-nio/src/test/java/com/google/cloud/storage/contrib/nio/CloudStoragePathTest.java

Original file line number	Diff line number	Diff line change
`@@ -391,7 +391,7 @@ public void testExists_trailingSlash() {`
`391`	`391`	`}`
`392`	`392`
`393`	`393`	`@Test`
`394`		`- public void testExists_trailingSlash_disablePseudoDirectories() {`
	`394`	`+ public void testExists_trailingSlash_disablePseudoDirectories() throws IOException {`
`395`	`395`	`try (CloudStorageFileSystem fs = forBucket("military", usePseudoDirectories(false))) {`
`396`	`396`	`assertThat(Files.exists(fs.getPath("fashion/"))).isFalse();`
`397`	`397`	`}`
`@@ -547,7 +547,7 @@ public void testIsDirectory_trailingSlash_alwaysTrue() {`
`547`	`547`	`}`
`548`	`548`
`549`	`549`	`@Test`
`550`		`- public void testIsDirectory_trailingSlash_pseudoDirectoriesDisabled_false() {`
	`550`	`+ public void testIsDirectory_trailingSlash_pseudoDirectoriesDisabled_false() throws IOException {`
`551`	`551`	`try (CloudStorageFileSystem fs = forBucket("doodle", usePseudoDirectories(false))) {`
`552`	`552`	`assertThat(Files.isDirectory(fs.getPath("fundir/"))).isFalse();`
`553`	`553`	`}`