-
Notifications
You must be signed in to change notification settings - Fork 0
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
Roadmap for 2.0 #10
Milestone
Comments
dnaber-de
added a commit
that referenced
this issue
Oct 30, 2022
dnaber-de
added a commit
that referenced
this issue
Oct 30, 2022
dnaber-de
added a commit
that referenced
this issue
Oct 30, 2022
dnaber-de
added a commit
that referenced
this issue
Oct 31, 2022
* Rename setters: ::defineDbName() → ::useDbName() and thus * Declare constants and globals on WpTestsStarter::bootstrap() and not immediately on the setters call * set all unnecessary public methods from WpTestsStarter to private
dnaber-de
added a commit
that referenced
this issue
Oct 31, 2022
dnaber-de
added a commit
that referenced
this issue
Oct 31, 2022
dnaber-de
added a commit
that referenced
this issue
Oct 31, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Simplify bootstrapping/setup
Right now you have to define half a dozen constants (most of them DB credentials) in the phpunit.xml file just to work. I'd like to have a DB URL here like
mysql://user:password@host/db_name?table_prefix=wp_
as single, yet human readable configuration. It should be read by default from an environment variable.Ideally we ship a default boostrap file with a common set up to ease the set up of tests with WP Tests Starter
API Updates
SaltGeneratorInterface
it's simply over the topWpTestsStarter
, there are simply to much of them that are not needed to be publicCommon
toHelper
WpTestsStarter::createDummyConfigFile()
to::writeConfigFile()
WpTestsStarter::bootstrap()
. Thus::defineDbName()
should be renamed to::useDbName()
and so onWpTestsStarter
public methodsWpTestsStarter::addLivePlugin(callable $plugin)
to directly run onmuplugins_loaded
WpTestsStarter::addFilter()
and::addAction()
to register listener before WP is loadedWpTestsStarter
into extra classes following single responsibility principleWpTestsStarter::useMuPluginDir(string $muPluginDir)
WpTestsStarter::addNetworkActivePlugin(string $plugin)
wordpress/wordpress
repo withroots/wordpress
or any other plain WP core library and a custom boot loaderOthers
wordpress/wordpress
so we don't need to maintain a fork anymore (which is abandoned anyway)Inpsyde\WpTestsStarter
and move everything tosrc/
The text was updated successfully, but these errors were encountered: