Skip to content
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

introduce wdio for e2e tests #902

Merged
merged 15 commits into from
Oct 25, 2023
Merged

introduce wdio for e2e tests #902

merged 15 commits into from
Oct 25, 2023

Conversation

diosmosis
Copy link
Member

@diosmosis diosmosis commented Oct 18, 2023

Description:

Refs MWP-166

Changes:

  • Introduce wdio and add one single e2e test for tracking.
  • Add package.json that was missing.
  • Add new WP CLI commands: install, sync (for users or sites), getting/setting global settings.
  • Fix issue in Installer.php: activate_$plugin hook can be invoked with the path to the plugin entrypoint (ie, matomo/matomo.php). This happens, for instance, when activating the plugin via wp-cli. Currently this change doesn't have an effect though, since the Installer class is only initiated in very specific circumstances.
  • In local dev environment, use wordpress.org API to get the latest version.
  • In local dev environment, allow using folder that doesn't match wordpress version name, eg, test.
  • In local dev environment, allow dropping the existing database if any, to start with a clean setup (for tests).
  • In local dev environment, make sure debug.log can be modified by a developer without them having to run a chmod command.
  • In local dev environment, use wp-cli to create application password to use for WP rest API.

Currently the tests pass locally. Another PR will be created to run the test in github actions and add some README contents.

Review

@diosmosis diosmosis added the Needs Review For pull requests that need a code review. label Oct 18, 2023
@diosmosis diosmosis requested a review from tsteur October 18, 2023 11:37
@diosmosis diosmosis mentioned this pull request Oct 24, 2023
11 tasks
@diosmosis diosmosis merged commit 14afc40 into develop Oct 25, 2023
6 of 10 checks passed
@diosmosis diosmosis deleted the wdio branch October 25, 2023 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review For pull requests that need a code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant