Pest: Proposal: pest-plugin-init

[nunomaduro]

After merging @fetzi's work on plugins. Would be cool to develop a pest-plugin-init that would work the following way:

1. In applications without tests: ./vendor/bin/pest --init
2. Boostraps the tests folder
3. Create a configuration file

[owenvoke]

This would be nice. 👍 I'd be happy to take a look when the plugin PR is merged.

[AlexMartinFR]

Nice!
Could we get rid of the dashes to provide a cleaner CLI experience ./vendor/bin/pest init ?

Some more thought about CLI syntax here: #10

[nunomaduro]

Dashes are for options. So you are right

[fetzi]

@owenvoke are you still on it?

[owenvoke]

I was, but was struggling on a way to implement it with the current plugin API.

If it uses HandlesArguments, then it may be run after other plugins which is not desired as it basically just needs to be a separate command.

[fetzi]

Yes, maybe this one shouldn't be a plugin and should be included in the core. @nunomaduro what do you think?

[nunomaduro]

Should be a plugin that exists by default on the core. 😊

[owenvoke]

Ok, I think we may need to modify the plugin API to allow this then. 👍

[fetzi]

@owenvoke I think that handleArguments is the right place for that. It won't matter if another plugin gets executed beforehand.

[owenvoke]

@fetzi, or afterwards? 🤔 Just seems a bit weird for init to also run tests. But if so, I'm happy to just do that.

[nunomaduro]

@owenvoke Well, you can exit(0) when you are handling the arguments of the init right?

[fetzi]

The watch plugin will also not continue to execute the tests directly. So I think exit(0) is good for the init plugin.

[owenvoke]

Just to check, I assume that if that's at the end of the handleArguments() method, then the addOutput() wouldn't be called? Because it would just exit.

Ignore that, addOutput() is after the test suite runs so there would be no point in including that. 🤦 Thinking about this, do we have an easy way of adding output to the handleArguments() method? 👍

---

Ok, another issue, if running pest init in a new repo that doesn't contain a tests directory then it throws an error.

Anyway, I've pushed a first version here:
https://github.com/pestphp/pest-plugin-init/pull/1