-
Notifications
You must be signed in to change notification settings - Fork 119
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
Use local tooling to run PHPUnit tests instead of wp-dev-lib #1124
Changes from 14 commits
93171e6
576f807
6f59bdf
b966f74
da94955
3002b98
4f3072d
a773fef
9479513
1a8614f
275a83f
c7b8cfc
d0c1ed9
16cf3e0
19c87c4
0fcf906
5d65bf8
b42cbbc
cf3d3f1
ac4f172
5d0686f
aaa4054
023b673
942dc4e
e44f942
dc5483c
18da2c6
5e86ca8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,34 +4,29 @@ | |
"homepage": "https://wordpress.org/plugins/stream/", | ||
"type": "wordpress-plugin", | ||
"license": "GPL-2.0-or-later", | ||
"repositories": [ | ||
{ | ||
"type": "git", | ||
"url": "https://github.com/WordPress/wordpress-develop.git" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We use https://github.com/johnpbloch/wordpress now which doesn't require any build steps and is much faster to install, and can be used in combination of |
||
} | ||
], | ||
"require": { | ||
"composer/installers": "~1.0" | ||
}, | ||
"require-dev": { | ||
"johnpbloch/wordpress": "^5.4", | ||
"php-coveralls/php-coveralls": "^2.1", | ||
"phpunit/phpunit": "^5.7", | ||
"wordpress/wordpress": "^5.2", | ||
"wp-cli/wp-cli-bundle": "^2.2", | ||
"wp-coding-standards/wpcs": "^2.2", | ||
"wpsh/local": "^0.2.3", | ||
"xwp/wp-dev-lib": "^1.5" | ||
"wp-phpunit/wp-phpunit": "^5.4", | ||
"wpsh/local": "^0.2.3" | ||
}, | ||
"config": { | ||
"process-timeout": 600, | ||
"sort-packages": true, | ||
"platform": { | ||
"php": "5.6" | ||
"php": "5.6.20" | ||
} | ||
}, | ||
"extra": { | ||
"wordpress-install-dir": "local/public" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Previously we were using the WP core files provisioned by the |
||
}, | ||
"scripts": { | ||
"pre-install-cmd": [ | ||
"Composer\\Config::disableProcessTimeout" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Replaced with a 600 second timeout for all commands as a sane default. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @kasparsd I don't know the original purpose of this timeout. |
||
], | ||
"release": [ | ||
"composer install --no-dev --prefer-dist --optimize-autoloader" | ||
], | ||
|
@@ -45,6 +40,9 @@ | |
"test": [ | ||
"phpunit --coverage-text" | ||
], | ||
"test-multisite": [ | ||
"WP_MULTISITE=1 phpunit --coverage-text" | ||
], | ||
"test-report": [ | ||
"php-coveralls --verbose" | ||
] | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We now have
npm run phpunit-multisite
for this.