Skip to content

Commit 0cc95b8

Browse files
committed
Remove lchown($file, -5); test
Some operating systems choose to return EINVAL for users (and groups) out of range (e.g. user ids must not be negative) instead of EPERM. This thus cannot be deterministically tested for all systems - and we are anyway already testing the branch with errno with the inexistent filename (ENOENT) - thus no problem removing it.
1 parent 1b5b923 commit 0cc95b8

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

ext/standard/tests/file/lchown_error.phpt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,6 @@ var_dump( lchown( 'foobar_lchown.txt', $uid ) );
4343
var_dump( lchown( new StdClass(), $uid ) );
4444
var_dump( lchown( array(), $uid ) );
4545

46-
// Bad user
47-
var_dump( lchown( $filename, -5 ) );
48-
4946
?>
5047
===DONE===
5148
--CLEAN--
@@ -72,7 +69,4 @@ bool(true)
7269

7370
Warning: lchown() expects parameter 1 to be a valid path, array given in %s on line %d
7471
bool(true)
75-
76-
Warning: lchown(): %r(Operation not permitted|Invalid argument)%r in %s on line %d
77-
bool(false)
7872
===DONE===

0 commit comments

Comments
 (0)