1 files changed, 42 insertions, 0 deletions
diff --git a/test/unit/gluster/test_gfapi.py b/test/unit/gluster/test_gfapi.py
index 1608332..8451b51 100644
--- a/test/unit/gluster/test_gfapi.py
+++ b/test/unit/gluster/test_gfapi.py
@@ -180,6 +180,18 @@ class TestFile(unittest.TestCase):
             b = self.fd.read(5)
             self.assertEqual(b, 0)
 
+    def test_read_buflen_negative(self):
+        _mock_fgetsize = Mock(return_value=12345)
+
+        def _mock_glfs_read(fd, rbuf, buflen, flags):
+            self.assertEqual(buflen, 12345)
+            return buflen
+
+        for buflen in (-1,-2,-999):
+            with patch("glusterfs.gfapi.api.glfs_read", _mock_glfs_read):
+                with patch("glusterfs.gfapi.File.fgetsize", _mock_fgetsize):
+                    b = self.fd.read(buflen)
+
     def test_write_success(self):
         mock_glfs_write = Mock()
         mock_glfs_write.return_value = 5
@@ -791,6 +803,36 @@ class TestVolume(unittest.TestCase):
             mock_unlink.assert_called_once_with("dir1/file")
             mock_rmdir.assert_called_with("dir1")
 
+    def test_setfsuid_success(self):
+        mock_glfs_setfsuid = Mock()
+        mock_glfs_setfsuid.return_value = 0
+
+        with patch("glusterfs.gfapi.api.glfs_setfsuid", mock_glfs_setfsuid):
+            ret = self.vol.setfsuid(1000)
+            self.assertEquals(ret, 0)
+
+    def test_setfsuid_fail(self):
+        mock_glfs_setfsuid = Mock()
+        mock_glfs_setfsuid.return_value = -1
+
+        with patch("glusterfs.gfapi.api.glfs_setfsuid", mock_glfs_setfsuid):
+            self.assertRaises(OSError, self.vol.setfsuid, 1001)
+
+    def test_setfsgid_success(self):
+        mock_glfs_setfsgid = Mock()
+        mock_glfs_setfsgid.return_value = 0
+
+        with patch("glusterfs.gfapi.api.glfs_setfsgid", mock_glfs_setfsgid):
+            ret = self.vol.setfsgid(1000)
+            self.assertEquals(ret, 0)
+
+    def test_setfsgid_fail(self):
+        mock_glfs_setfsgid = Mock()
+        mock_glfs_setfsgid.return_value = -1
+
+        with patch("glusterfs.gfapi.api.glfs_setfsgid", mock_glfs_setfsgid):
+            self.assertRaises(OSError, self.vol.setfsgid, 1001)
+
     def test_setxattr_success(self):
         mock_glfs_setxattr = Mock()
         mock_glfs_setxattr.return_value = 0