-
Notifications
You must be signed in to change notification settings - Fork 283
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 support for Pest test generation #625
Conversation
If I were going to merge this, I'd want to see a separate class, rather than a bunch of |
@jasonmccreary I'll work on the separate classes later today or more likely tomorrow. Do you want to keep them completely separate or is an abstract class they extend fine? |
Not sure it's worth it, but if you want. |
@jasonmccreary went for duplication for now because there's so many small differences that an abstraction would be a bit hacky. Also added tests for everything that the the PHPUnit generator also tests. |
@jasonmccreary something I wondered while looking at the Edit: I might be blind but |
This is a pretty large changeset. I'll review it over the next few days. |
This makes the test framework configurable to be either PHPUnit or Pest and then generates tests accordingly, of course defaulting to PHPUnit. The implementation is mostly a bunch of
if
statements that alternate between 2 strings and for Pest this results in some cases for additional imports being added in the form ofuse function
imports.