Ensure use of PHPUnit installed by composer #58
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Changes tests scripts so that they explicitly use the
phpunit
command installed by composer.Motivation and Context
The travis scripts set the
PATH
to include the composer vendor directory, but there is no guarantee that a developer running the tests scripts will do the same or that they will not have a different version of PHPUnit installed elsewhere in theirPATH
. These changes ensure that thephpunit
that is installed in the vendor directory will be used.There may be a better way to do this, or we may not want to do it at all, so I'm open to discussion. I do think we should do something to make sure that it's easy for developers running these scripts on the command line to know the correct version of PHPUnit is being used.
Tests performed
Ran updated tests scripts on the command line.
Types of changes
Checklist: