Skip to content
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

Add failing test for TernaryToElvisOperatorFixer #5595

Closed
wants to merge 1 commit into from

Conversation

jrmajor
Copy link
Contributor

@jrmajor jrmajor commented Apr 5, 2021

Original bug report: #5533 (comment)

I don't know how to fix this, but I've created a test that reproduces the #5533 issue. It does not fail, but times out and is marked as risky:

There was 1 risky test:

1) PhpCsFixer\Tests\Fixer\Operator\TernaryToElvisOperatorFixerTest::test80DoNotFix with data set #0 ('<?php\n\nfunction test(#[Test...) {}\n')
Execution aborted after 10 seconds

@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.851% when pulling d497213 on jrmajor:test-5533 into 9bf2760 on FriendsOfPHP:2.18.

@kubawerlos
Copy link
Contributor

Replaced by #5596

Thanks @jrmajor, I've cherry picked your test to that PR.

@kubawerlos kubawerlos closed this Apr 5, 2021
@jrmajor jrmajor deleted the test-5533 branch April 5, 2021 21:46
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.

None yet

3 participants