demobox commented on this pull request.


>     }
 
    @Override
    public void testSetBlobAccess() throws Exception {
       throw new SkipException("filesystem does not support anonymous access");
    }
+
+   private void checkExtendedAttributesSupport() {
+      if (isMacOSX()) {
+         throw new SkipException("filesystem does not support extended 
attributes on Mac OSX");

If we end up using `SkipException` in lots of places, thoughts on creating a 
`TestUtils.skip` helper for readability? Then we could write:

```
if (isMacOSX()) {
   skip("filesystem does not support extended attributes on Mac OSX");
}
```
?

Thanks for taking care of this, @nacx!

-- 
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1150#pullrequestreview-71472165

Reply via email to