Skip to content
This repository was archived by the owner on Aug 24, 2021. It is now read-only.

Conversation

@ericpoe
Copy link
Contributor

@ericpoe ericpoe commented Sep 7, 2017

This tests that the bug reported in #75135 is actually a bug.

This test was a collaborative effort by the Kansas City PHP User Group on our first meetup for PHPTestFest 2017. Woo!

@ramsey
Copy link
Contributor

ramsey commented Sep 7, 2017

I believe you need to add the --XFAIL-- section to this test: https://qa.php.net/phpt_details.php#xfail_section

p365labs pushed a commit to p365labs/phptestfest-php-src that referenced this pull request Nov 18, 2017
unlink($targetDir.DIRECTORY_SEPARATOR.'getBasename_test.txt');
rmdir($targetDir);
?>
--XFAIL--
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We won't accept XFAIL tests 😕

@petk
Copy link
Contributor

petk commented Feb 11, 2019

Hello, thank you for the pull request. It seems that the only proper way here is to fix the mentioned bug instead. XFAIL tests aren't being added in the php-src anymore unfortunately.

@petk petk closed this Feb 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants