Skip to content

PHPC-2121: Skip failing test #1365

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.

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

Merged
merged 2 commits into from
Sep 16, 2022

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented Sep 15, 2022

PHPC-2121

In an effort to reach a green CI, I decided to skip this failing test since it's tracked as failing-on-waterfall.


As an aside, this sent me down a deep rabbit hole trying to figure out why I couldn't get the test to properly skip. The culprit is the SKIPIF caching introduced in PHP 8.1 (as noticed by @jmikola before - see #1301 and php/php-src#8076). Until we properly fix this, skip_if_not_clean has to be the last SKIPIF check. In a case of not optimising early enough, I added the skip_if_server_version check to the bottom, which caused run-tests.php to truncate its output due to nocache appearing before.

@alcaeus alcaeus requested a review from jmikola September 15, 2022 12:11
@alcaeus alcaeus self-assigned this Sep 15, 2022
Copy link
Member

@jmikola jmikola left a comment

Choose a reason for hiding this comment

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

Suggested TODO comment if you'd like to add that in. I recall doing this before in other tests.

Co-authored-by: Jeremy Mikola <jmikola@gmail.com>
@alcaeus alcaeus merged commit 04f504c into mongodb:master Sep 16, 2022
@alcaeus alcaeus deleted the feature/phpc-2121-skip-failing-test branch September 16, 2022 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants