From a600a3d254bfe2b192fa5006f72142eebb6e5365 Mon Sep 17 00:00:00 2001 From: Adrien Ferrand Date: Tue, 4 Jun 2019 23:29:51 +0200 Subject: [PATCH] Fix comment, and trigger --- certbot/tests/compat/filesystem_test.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/certbot/tests/compat/filesystem_test.py b/certbot/tests/compat/filesystem_test.py index d59b6ac0b..5e7d1c799 100644 --- a/certbot/tests/compat/filesystem_test.py +++ b/certbot/tests/compat/filesystem_test.py @@ -14,11 +14,11 @@ from certbot.compat import filesystem from certbot.tests.util import TempDirTestCase -@unittest.skipIf(not POSIX_MODE, reason='Test specific to Windows security') +@unittest.skipIf(POSIX_MODE, reason='Test specific to Windows security') class WindowsChmodTests(TempDirTestCase): """Unit tests for Windows chmod function in filesystem module""" def test_symlink_resolution(self): - probe_path = _create_probe(self.tempdir) # This is 0o744 by default + probe_path = _create_probe(self.tempdir) link_path = os.path.join(self.tempdir, 'link') os.symlink(probe_path, link_path) @@ -26,7 +26,6 @@ class WindowsChmodTests(TempDirTestCase): ref_dacl_probe = _get_security_dacl(probe_path).GetSecurityDescriptorDacl() ref_dacl_link = _get_security_dacl(link_path).GetSecurityDescriptorDacl() - # Removing the rights for `all`, we have at least one ACL less than in the case of 0o744. filesystem.chmod(link_path, 0o700) # Assert the real file is impacted, not the link.