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.

Sign up for GitHub

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

Jump to bottom

add e2e tests for WP multisite usage #1238

Merged
merged 34 commits into from
Jan 10, 2025
Merged

add e2e tests for WP multisite usage #1238

merged 34 commits into from
Jan 10, 2025

Conversation

diosmosis
Copy link
Member

Description:

As title.

Review

@diosmosis
add test for mark_matomo_installed + when updating run update in addi…
a668dc9 
…tion to install just in case there ends up being more work to do
@diosmosis diosmosis force-pushed the multisite-e2e branch 5 times, most recently from 5c05ce6 to 0498641 Compare January 10, 2025 00:55
@diosmosis diosmosis merged commit 4ea1eac into develop Jan 10, 2025
9 of 11 checks passed
@diosmosis diosmosis deleted the multisite-e2e branch January 10, 2025 03:48
diosmosis added a commit that referenced this pull request Jan 13, 2025
@diosmosis
Update live for upcoming 5.2.0 release (#1242)
1fb89e3 
* update matomo core to 5.2.0

* update expected screenshots

* update assets

* update expected screenshots + remove tag manager getting started page tests as the page was removed

* debug woocommerce setup failure

* update more expected screenshots

* debug woocommerce e2e issue

* woocommerce 9.5.1 moves the "skip guided tour" link in the setup

* update expected screenshots and fix selector for tag manager container dashboard page

* update expected screenshot + fix regex

* fix regex

* update expected screenshot

* add ability to show notifications for new features in matomo for wordpress (#1226)

* add new feature to display hard coded notifications for new features or features we want to promote

* add JS to dismiss whats new notification + start on notification dot css

* add filter to allow other parts of MWP to edit matomo submenu information

* revert menu changes and go to javascript approach so pages added outside of the MWP plugin can have notification dots as well

* in wordpress:download-test-screenshots use curl CLI command instead of PHP curl due to issue in github API

* try to fix e2e test failure

* finalize look of promo and marketplace changes

* allow whats new notifications to determine exactly when they should show rather than hardcoding is_super_admin()

* translate crash analytics promo text

* stub unit tests for new WhatsNewNotifications feature

* add first unit test for WhatsNewNotifications

* add tests for WhatsNewNotification::is_active

* add tests for register_hooks

* add tests for on_admin_notices hook

* finish unit tests for WhatsNewNotifications feature and get to pass

* add e2e test for notifications

* fix import

* debug failing test

* fix random new url path issue in test

* update existing screenshots and add pause

* update expected screenshot

* fill out PR TODO

* do not show promo notifications on get started page so users has a clear list of what to do

* update expected screenshot

* do not report "could not set" errors when scheduling cron events

* update core to 5.2.1 (#1236)

* install multisite environment alongside normal for easier manual testing (#1233)

* install multisite environment alongside normal for easier manual testing

* fix local-dev-entrypoint.sh warning

* try to fix file permission in build

* add setup finished marker file after both instances are installed

* increase env setup wait timeout

* debug script mods

* smarter waiting for the database

* fix failing command

* fix setup_finished file creation

* debug e2e test failure

* fix uninstall e2e test failure

* recognize new mariadb specific queries in the custom WordPress database adapter (#1237)

* recognize new mariadb specific queries in WordPressDbStatement

* stub test for change

* add test to ensure SET STATEMENT FOR MariaDB specific SQL works properly in MWP

* compatibility with the latest wpstatistics release (#1240)

* compatibility with the latest wpstatistics release

* update expected screenshot tests

* add e2e tests for WP multisite usage (#1238)

* stub new e2e tests

* correctly detect if Matomo is installed for a specific site in a multisite setup + when installing, force plugins to intstall by emptying PluginsInstalled config

* try installing matomo before running the sync scheduled tasks, in case a site has not been installed

* fill out multisite e2e tests

* use option instead of direct db query

* debug test failure

* override wordpress folder in multisite e2e tests

* update screenshots + fix screenshot name in test

* update screenshots, fix e2e test JS and use more complex detection of whether the current sites Matomo has been installed completely

* settings must be strings

* handle unset option

* fix test and ignore matomo/matomo.php file in check for config installed

* try to fix e2e test selector

* add normalization logic to network diagnostics page object

* update screenshots and wait for element to exist before clicking

* more waits

* even more waits

* fix wait when user country map is empty

* update expected screenshots

* update screenshots and debug random install failure

* more debugging of build

* try to workaround issue that occurs when installing multiple non-core plugins at the same time

* screenshot normalization

* reporting dashboard tests do not need to differentiate based on trunk/php version

* update expected screenshot

* remove some unneeded plugin manager calls

* update woocommerce tested up to version

* tweak comments

* add unit tests + docs

* remove accidentally committed debug code

* add test for mark_matomo_installed + when updating run update in addition to install just in case there ends up being more work to do

* retry requests in e2e tests

* minor code cleanup

* bump version for 5.2.0 release (#1241)

* Bump versions and update changelog.

* wait after a failed request in e2e tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@diosmosis