8000 Skip test if running as superuser as it will always fail by mvrhov · Pull Request #2356 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

Skip test if running as superuser as it will always fail #2356

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Skip test if running as superuser as it will always fail #2356

wants to merge 1 commit into from

Conversation

mvrhov
Copy link
@mvrhov mvrhov commented Oct 8, 2011

The above test fails at least on Ubuntu if I run testsuite as root user

@fabpot
Copy link
Member
fabpot commented Oct 8, 2011

What is the error when it fails?

@mvrhov
Copy link
Author
mvrhov commented Oct 8, 2011

There was 1 failure:

  1. Symfony\Tests\Component\HttpFoundation\File\MimeTypeTest::testGuessWithNonReadablePath
    Expected exception Symfony\Component\HttpFoundation\File\Exception\AccessDeniedException

/srv/www/github/symfony2/tests/Symfony/Tests/Component/HttpFoundation/File/MimeType/MimeTypeTest.php:87

edit:
is_readable will return true even if I set the file permissions to 0000

If I run test suite as a normal user then the test will pass.

fabpot added a commit that referenced this pull request Oct 8, 2011
@fabpot
merged branch mvrhov/test_fixes (PR #2357)
35c5e1b 
Commits
-------

395f580 Rebuild resource files with genrb from ICU 4.2. as ICU 4.4 by default builds them in newer format. Also added all files necessary to build resources.dat with pkgdat because syntax for command is so cryptic
438581d Skip test if running as superuser as it will fail

Discussion
----------

Fixes failing tests

This one closes both the #2355 and #2356
@fabpot fabpot closed this Oct 8, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mvrhov @fabpot
0