Add integration tests #393
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vcanales
requested review from
pbking,
jeffikus,
matiasbenedetto,
mikachan,
madhusudhand,
jffng and
a team
mikachan
reviewed
Jun 8, 2023
I've added testing instructions to the description.
|
|
I added a GitHub actions, which you can see working on this very PR. This required me to add back the changes made for #379 to get tests passing :) |
vcanales
changed the title
Data url obtained from the official docs, here: https://codex.wordpress.org/Theme_Unit_Test
Starting with the export_child_theme method.
Refactors test installation script for it to be able to run in a CI context without requiring user input try adding test action add mysql setup fix wp config creation command overwrite if it exists create wpconfig before install attempt move musql config to test step fix multiple command syntax add missing command add option to recreate db from cli Helps us skip the prompt for user input move cli options to the end add host to command call refactor to add options at the beginning try different order add missing steps use config create instead of core create skip check on config creation change to 127.0.0.1 as default host install composer dependencies refactor composer action install dev dependencies refactor Lint action silence stdout from test installation script pretty print tests do not profile composer
Required to get sample test passing
This allows us to share dependencies.
|
Added a command to run tests in watch mode. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
To facilitate future improvements to the codebase, this proposes configuring PHPUnit to be able to write tests for the plugin.
This was created using
wp scaffold plugin-test, plus some modifications to addwp-cliand theme data to the test site.A first test is included at
tests/test-theme-create-theme.php.Testing
To run this suite, a local MySQL installation is required. After that is taken care of, the following commands should get everything setup: