search

pestphp / drift

[WIP] Command-line tool to migrate your PHPUnit tests to Pest
https://pestphp.com/docs/guides/drift
MIT License
28 stars 7 forks source link

Add Rector for 'depends' #15

Closed owenvoke closed 4 years ago

owenvoke commented 4 years ago

This adds support for converting the @depends PHPUnit doc tag to Pest's ->depends() chainable method.

Closes https://github.com/pestphp/drift/issues/14

owenvoke commented 4 years ago

The failures in the test suite appear to be unrelated to the code I've added. 🤔 Any thoughts @olivernybroe?

olivernybroe commented 4 years ago

@owenvoke So the test which fails is because we simply haven't implemented it yet :)

This way we don't forget that this the issue is there.

The warning in the tests is because we simply haven't added any tests for checking if the removal of the php unit class is done correctly

owenvoke commented 4 years ago

Thanks! From looking at Drift, Rector is so neat. 😀

olivernybroe commented 4 years ago

@owenvoke Yeah, it's really awesome to work with! :)

And the creator is active and willing to answer support questions and even help with the code.