Merge pull request #30 from jasontedor/attributes-uoe

Throw UOE when setting unsupported attributes

3 files changed, 4 insertions, 4 deletions

diff --git a/jimfs/src/main/java/com/google/common/jimfs/AttributeService.java b/jimfs/src/main/java/com/google/common/jimfs/AttributeService.java
index 636066e..9cbe221 100644
--- a/jimfs/src/main/java/com/google/common/jimfs/AttributeService.java
+++ b/jimfs/src/main/java/com/google/common/jimfs/AttributeService.java
@@ -264,7 +264,7 @@ final class AttributeService {
       }
     }
 
-    throw new IllegalArgumentException("cannot set attribute '" + view + ":" + attribute + "'");
+    throw new UnsupportedOperationException("cannot set attribute '" + view + ":" + attribute + "'");
   }
 
   /**
diff --git a/jimfs/src/test/java/com/google/common/jimfs/AttributeServiceTest.java b/jimfs/src/test/java/com/google/common/jimfs/AttributeServiceTest.java
index 8977eda..94b4912 100644
--- a/jimfs/src/test/java/com/google/common/jimfs/AttributeServiceTest.java
+++ b/jimfs/src/test/java/com/google/common/jimfs/AttributeServiceTest.java
@@ -162,14 +162,14 @@ public class AttributeServiceTest {
     try {
       service.setAttribute(file, "test:blah", "blah", false);
       fail();
-    } catch (IllegalArgumentException expected) {
+    } catch (UnsupportedOperationException expected) {
     }
 
     try {
       // baz is defined by "test", but basic doesn't inherit test
       service.setAttribute(file, "basic:baz", 5, false);
       fail();
-    } catch (IllegalArgumentException expected) {
+    } catch (UnsupportedOperationException expected) {
     }
 
     assertThat(file.getAttribute("test", "baz")).isEqualTo(1);
diff --git a/jimfs/src/test/java/com/google/common/jimfs/JimfsUnixLikeFileSystemTest.java b/jimfs/src/test/java/com/google/common/jimfs/JimfsUnixLikeFileSystemTest.java
index 72623bd..adb5fb8 100644
--- a/jimfs/src/test/java/com/google/common/jimfs/JimfsUnixLikeFileSystemTest.java
+++ b/jimfs/src/test/java/com/google/common/jimfs/JimfsUnixLikeFileSystemTest.java
@@ -684,7 +684,7 @@ public class JimfsUnixLikeFileSystemTest extends AbstractJimfsIntegrationTest {
     try {
       Files.createFile(path("/foo"), new BasicFileAttribute<>("basic:noSuchAttribute", "foo"));
       fail();
-    } catch (IllegalArgumentException expected) {
+    } catch (UnsupportedOperationException expected) {
     }
 
     assertThatPath("/foo").doesNotExist();