diff options
author | Thiago da Silva <thiago@redhat.com> | 2014-01-11 13:07:17 -0500 |
---|---|---|
committer | Thiago da Silva <thiago@redhat.com> | 2014-01-14 16:12:20 -0500 |
commit | f23af7ead83d249d52700e69bc906a04ec0920f9 (patch) | |
tree | a2a6909d12df27a9228979012179e5aebcafff6f /gluster | |
parent | bd4e6c5dba99ec2dcb3675b4c1b3a9c4ae2ee9bf (diff) |
unit tests
added unit tests for existing methods
Change-Id: Iecd31bc1dea44ea38168120bd05ca505c4ff5c59
Signed-off-by: Thiago da Silva <thiago@redhat.com>
Diffstat (limited to 'gluster')
-rw-r--r-- | gluster/gfapi.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/gluster/gfapi.py b/gluster/gfapi.py index 0116e3c..01fdae7 100644 --- a/gluster/gfapi.py +++ b/gluster/gfapi.py @@ -99,7 +99,7 @@ class File(object): rbuf = ctypes.create_string_buffer(buflen) ret = api.glfs_read(self.fd, rbuf, buflen, flags) if ret > 0: - return rbuf.value[:rc] + return rbuf.value[:ret] elif ret < 0: err = ctypes.get_errno() raise OSError(err, os.strerror(err)) @@ -218,11 +218,10 @@ class Volume(object): def lstat(self, path): x = Stat() rc = api.glfs_lstat(self.fs, path, ctypes.byref(x)) - if rc >= 0: - return x - else: + if rc < 0: err = ctypes.get_errno() raise OSError(err, os.strerror(err)) + return x def mkdir(self, path): ret = api.glfs_mkdir(self.fs, path) |