Browse Source

label platform code, exclude freebsd and unknown platform from coverage measurement

this coverage configuration is mostly for travis and there we only can test linux and darwin.
Thomas Waldmann 9 years ago
parent
commit
35280e9a65
3 changed files with 10 additions and 8 deletions
  1. 2 0
      .coveragerc
  2. 4 4
      borg/platform.py
  3. 4 4
      borg/xattr.py

+ 2 - 0
.coveragerc

@@ -10,6 +10,8 @@ omit =
 [report]
 exclude_lines =
     pragma: no cover
+    pragma: freebsd only
+    pragma: unknown platform only
     def __repr__
     raise AssertionError
     raise NotImplementedError

+ 4 - 4
borg/platform.py

@@ -1,12 +1,12 @@
 import sys
 
-if sys.platform.startswith('linux'):
+if sys.platform.startswith('linux'):  # pragma: linux only
     from .platform_linux import acl_get, acl_set, API_VERSION
-elif sys.platform.startswith('freebsd'):
+elif sys.platform.startswith('freebsd'):  # pragma: freebsd only
     from .platform_freebsd import acl_get, acl_set, API_VERSION
-elif sys.platform == 'darwin':
+elif sys.platform == 'darwin':  # pragma: darwin only
     from .platform_darwin import acl_get, acl_set, API_VERSION
-else:
+else:  # pragma: unknown platform only
     API_VERSION = 2
 
     def acl_get(path, item, st, numeric_owner=False):

+ 4 - 4
borg/xattr.py

@@ -36,7 +36,7 @@ def _check(rv, path=None):
         raise OSError(get_errno(), path)
     return rv
 
-if sys.platform.startswith('linux'):
+if sys.platform.startswith('linux'):  # pragma: linux only
     libc.llistxattr.argtypes = (c_char_p, c_char_p, c_size_t)
     libc.llistxattr.restype = c_ssize_t
     libc.flistxattr.argtypes = (c_int, c_char_p, c_size_t)
@@ -100,7 +100,7 @@ if sys.platform.startswith('linux'):
             func = libc.lsetxattr
         _check(func(path, name, value, len(value) if value else 0, 0), path)
 
-elif sys.platform == 'darwin':
+elif sys.platform == 'darwin':  # pragma: darwin only
     libc.listxattr.argtypes = (c_char_p, c_char_p, c_size_t, c_int)
     libc.listxattr.restype = c_ssize_t
     libc.flistxattr.argtypes = (c_int, c_char_p, c_size_t)
@@ -166,7 +166,7 @@ elif sys.platform == 'darwin':
             flags = XATTR_NOFOLLOW
         _check(func(path, name, value, len(value) if value else 0, 0, flags), path)
 
-elif sys.platform.startswith('freebsd'):
+elif sys.platform.startswith('freebsd'):  # pragma: freebsd only
     EXTATTR_NAMESPACE_USER = 0x0001
     libc.extattr_list_fd.argtypes = (c_int, c_int, c_char_p, c_size_t)
     libc.extattr_list_fd.restype = c_ssize_t
@@ -247,7 +247,7 @@ elif sys.platform.startswith('freebsd'):
             func = libc.extattr_set_link
         _check(func(path, EXTATTR_NAMESPACE_USER, name, value, len(value) if value else 0), path)
 
-else:
+else:  # pragma: unknown platform only
     def listxattr(path, *, follow_symlinks=True):
         return []