Skip to content

Automated browser tests#12

Merged
adriaandotcom merged 180 commits intosimpleanalytics:masterfrom
osbre:feat/automated-browser-tests
Apr 9, 2026
Merged

Automated browser tests#12
adriaandotcom merged 180 commits intosimpleanalytics:masterfrom
osbre:feat/automated-browser-tests

Conversation

@osbre
Copy link
Copy Markdown
Contributor

@osbre osbre commented Apr 14, 2025

No description provided.

@osbre osbre marked this pull request as draft April 14, 2025 03:29
@adriaandotcom adriaandotcom mentioned this pull request Jul 29, 2025
osbre and others added 18 commits April 8, 2026 12:08
docker-compose.yml mirrors the CI setup: MySQL 8 + FrankenPHP on a
shared named network. wp-cli runs in a container on that network so
DB_HOST=mysql works consistently. Makefile exposes three targets:
  make test-env   — first-time setup (downloads WP, starts stack)
  make test       — run the suite
  make test-reset — tear down and start completely fresh

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@osbre osbre marked this pull request as ready for review April 9, 2026 12:48
@osbre
Copy link
Copy Markdown
Contributor Author

osbre commented Apr 9, 2026

The timeouts were due to WP cron all along. Disabling it solved.

@osbre
Copy link
Copy Markdown
Contributor Author

osbre commented Apr 9, 2026

Now I either need to force-push this branch to join all the commits into one, or the reviewer must merge with a "squash and merge" option in the dropdown

@adriaandotcom
Copy link
Copy Markdown
Contributor

Let's do squash and merge.

@adriaandotcom
Copy link
Copy Markdown
Contributor

Can you fix the merge conflicts?

@osbre
Copy link
Copy Markdown
Contributor Author

osbre commented Apr 9, 2026

@adriaandotcom sure

@osbre
Copy link
Copy Markdown
Contributor Author

osbre commented Apr 9, 2026

@adriaandotcom I don't think there's any? At least on GitHub's Web UI. Let me know if there is something I can't see

image

@adriaandotcom
Copy link
Copy Markdown
Contributor

Hm, that's odd. I see this:

2026-04-09-15h24-screenshot@2x

@adriaandotcom adriaandotcom merged commit 9a53735 into simpleanalytics:master Apr 9, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants