Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

What? No PHPUnit? #124

Closed
ghost opened this issue Jun 17, 2015 · 3 comments
Closed

What? No PHPUnit? #124

ghost opened this issue Jun 17, 2015 · 3 comments

Comments

@ghost
Copy link

ghost commented Jun 17, 2015

Considering half the tests in Drupal 8 are now PHPUnit based its omission from the base install is a bit of an oversight I'd think. Otherwise works great.

@geerlingguy
Copy link
Owner

PHPUnit should be installed along with other composer dependencies as required per-project... how else would you like to see it installed?

@ghost
Copy link
Author

ghost commented Jun 17, 2015

Globally..
composer global require "phpunit/phpunit=4.7.*"

Also PHP Code Sniffer would be useful to have off the bat for code format compliance.

composer global require "squizlabs/php_codesniffer=*"

I ran each of these after I got it up and running, then added the composer bin directory to my path.

They're easy enough to setup though including them would further help people hit the ground running.

@geerlingguy
Copy link
Owner

I've updated the upstream geerlingguy.composer role (see geerlingguy/ansible-role-composer#2), so now you can add something like the following in your config.yml file (I'll add a commented example in the project's example file in a commit in a minute or two):

composer_global_packages:
  - { name: phpunit/phpunit, release: '4.7.*' }
  - { name: phpmd/phpmd, release: '2.2.*' }

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant