Skip to content

Remove wp core init-tests command#885

Merged
scribu merged 1 commit into
masterfrom
no-init-tests
Nov 24, 2013
Merged

Remove wp core init-tests command#885
scribu merged 1 commit into
masterfrom
no-init-tests

Conversation

@scribu
Copy link
Copy Markdown
Member

@scribu scribu commented Nov 24, 2013

@scribu
Copy link
Copy Markdown
Member Author

scribu commented Nov 24, 2013

The workflow that wp core init-tests command encourages is the following:

These days, if you're serious about doing Core development, you'll check out http://develop.svn.wordpress.org which contains not only the test suite, but other tools, like Grunt tasks.

So, the test suite is both more visible and easier to set up than before, which eliminates this command's raison d'etre. Therefore, it should be removed from WP-CLI.

@scribu
Copy link
Copy Markdown
Member Author

scribu commented Nov 24, 2013

One might argue that the command could be adapted so that it doesn't download the test suite, but simply configures the one that's downloaded from develop.svn.wordpress.org. The steps performed would be:

  • create wp-tests-config.php
  • create an empty database

However, neither require any WP-CLI machinery, so they could be automated just as easily with a plain bash or PHP script. (Or, ideally, added as a Grunt task to Core.)

scribu pushed a commit that referenced this pull request Nov 24, 2013
Remove `wp core init-tests` command
@scribu scribu merged commit fadc6a6 into master Nov 24, 2013
@scribu scribu deleted the no-init-tests branch November 24, 2013 19:27
@ghost
Copy link
Copy Markdown

ghost commented Nov 24, 2013

FWIW, you may want to update http://wp-cli.org/blog/plugin-unit-tests.html since it references a now obsolete workflow. Not sure if you have a procedure in place for handling some of the old blog posts that now show irrelevant commands/info.

@scribu
Copy link
Copy Markdown
Member Author

scribu commented Nov 24, 2013

I'm probably going to write a new post and link to it from the old one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant