Commitb9f485f

Spece

committed

#2079 Satisfy.

1 parent271328f commitb9f485fCopy full SHA for b9f485f

File tree

+12

-3

lines changed

+12

-3

lines changed

Lines changed: 12 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,13 @@`
`45`	`45`
`46`	`46`	`@pytest.fixture`
`47`	`47`	`defpermission_error_tmpdir(tmp_path):`
`48`		`-"""Fixture to test permissions errors in situations where they are not overcome."""`
	`48`	`+"""Fixture to test permissions errors when they cannot be bypassed.`
	`49`	`+`
	`50`	`+ On Unix-like systems the setup relies on running as a non-root user so that`
	`51`	+ removing a read-only directory raises ``PermissionError``. When the tests
	`52`	`+ execute with root privileges the expected error will not occur, so we skip`
	`53`	`+ them explicitly to avoid false failures.`
	`54`	`+ """`
`49`	`55`	`td=tmp_path/"testdir"`
`50`	`56`	`td.mkdir()`
`51`	`57`	`(td/"x").touch()`
`@@ -54,10 +60,13 @@ def permission_error_tmpdir(tmp_path):`
`54`	`60`	`(td/"x").chmod(stat.S_IRUSR)`
`55`	`61`
`56`	`62`	`# Set up PermissionError on Unix, where non-root users can't delete files in`
`57`		`-# read-only directories.(Tests that rely on this and assert that rmtree raises`
`58`		`-#PermissionError will fail if they are run as root.)`
	`63`	`+# read-only directories.Skip when running as root because the permission`
	`64`	`+#change would not trigger the expected failure.`
`59`	`65`	`td.chmod(stat.S_IRUSR\|stat.S_IXUSR)`
`60`	`66`
	`67`	`+ifhasattr(os,"geteuid")andos.geteuid()==0:`
	`68`	`+pytest.skip("permission_error_tmpdir requires non-root privileges to raise PermissionError")`
	`69`	`+`
`61`	`70`	`yieldtd`
`62`	`71`
`63`	`72`

Comments

(0)