Merge pull request #117 from jayunit100/test4null

regression testable create method for gluster volume

Author: jayunit100

src/main/java/org/apache/hadoop/fs/glusterfs/GlusterVolume.java

@@ -69,10 +69,15 @@ public GlusterVolume(Configuration conf){
         this.setConf(conf);
     }
 
+    public URI getUriOrCreate(URI u) { 
+       if(NAME==null) 
+	       return URI.create("glusterfs:///");
+       return NAME;       
+    }
+
     public URI getUri() { 
-      if(NAME==null) return URI.create("glusterfs:///");
-      return NAME;       
-   }
+       return getUriOrCreate(NAME);
+    }
 
     public void initialize(URI uri, Configuration conf) throws IOException {
         /* we only really care about the URI up to the path, so strip other things off */

src/test/java/org/apache/hadoop/fs/test/GlusterVolumeTest.java

@@ -0,0 +1,23 @@
+package org.apache.hadoop.fs.test;
+
+import java.io.IOException;
+import junit.framework.Assert;
+import org.apache.hadoop.conf.Configuration;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.apache.hadoop.fs.glusterfs.*;
+/**
+ *  This class is for tests which are gluster specific.
+ */ 
+public class GlusterVolumeTest {
+   
+    Logger log = LoggerFactory.getLogger(GlusterVolumeTest.class);
+    static Configuration  config = null;
+ 
+    @org.junit.Test
+    public void testNullURI() { 
+	Assert.assertNotNull(new org.apache.hadoop.fs.glusterfs.GlusterVolume().getUriOrCreate(null));
+    }
+    
+}