diff --git a/borg/testsuite/xattr.py b/borg/testsuite/xattr.py index d73856953..df0130c90 100644 --- a/borg/testsuite/xattr.py +++ b/borg/testsuite/xattr.py @@ -17,17 +17,23 @@ class XattrTestCase(BaseTestCase): def tearDown(self): os.unlink(self.symlink) + def assert_equal_se(self, is_x, want_x): + # check 2 xattr lists for equality, but ignore security.selinux attr + is_x = set(is_x) - {'security.selinux'} + want_x = set(want_x) + self.assert_equal(is_x, want_x) + def test(self): - self.assert_equal(listxattr(self.tmpfile.name), []) - self.assert_equal(listxattr(self.tmpfile.fileno()), []) - self.assert_equal(listxattr(self.symlink), []) + self.assert_equal_se(listxattr(self.tmpfile.name), []) + self.assert_equal_se(listxattr(self.tmpfile.fileno()), []) + self.assert_equal_se(listxattr(self.symlink), []) setxattr(self.tmpfile.name, 'user.foo', b'bar') setxattr(self.tmpfile.fileno(), 'user.bar', b'foo') setxattr(self.tmpfile.name, 'user.empty', None) - self.assert_equal(set(listxattr(self.tmpfile.name)), set(['user.foo', 'user.bar', 'user.empty'])) - self.assert_equal(set(listxattr(self.tmpfile.fileno())), set(['user.foo', 'user.bar', 'user.empty'])) - self.assert_equal(set(listxattr(self.symlink)), set(['user.foo', 'user.bar', 'user.empty'])) - self.assert_equal(listxattr(self.symlink, follow_symlinks=False), []) + self.assert_equal_se(listxattr(self.tmpfile.name), ['user.foo', 'user.bar', 'user.empty']) + self.assert_equal_se(listxattr(self.tmpfile.fileno()), ['user.foo', 'user.bar', 'user.empty']) + self.assert_equal_se(listxattr(self.symlink), ['user.foo', 'user.bar', 'user.empty']) + self.assert_equal_se(listxattr(self.symlink, follow_symlinks=False), []) self.assert_equal(getxattr(self.tmpfile.name, 'user.foo'), b'bar') self.assert_equal(getxattr(self.tmpfile.fileno(), 'user.foo'), b'bar') self.assert_equal(getxattr(self.symlink, 'user.foo'), b'bar')