FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Automated browser tests by osbre · Pull Request #12 · simpleanalytics/wordpress-plugin · GitHub

Automated browser tests - #12

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

adriaandotcom merged 180 commits into
simpleanalytics:masterfrom
osbre:feat/automated-browser-tests

Conversation

osbre commented Apr 14, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

No description provided.

osbre marked this pull request as draft April 14, 2025 03:29
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 marked this pull request as ready for review April 9, 2026 12:48

osbre commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

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

osbre commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
Contributor

Let's do squash and merge.

Copy link
Copy Markdown
Contributor

Can you fix the merge conflicts?

osbre commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author

@adriaandotcom sure

osbre commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@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

Copy link
Copy Markdown
Contributor

Hm, that's odd. I see this:

adriaandotcom merged commit 9a53735 into simpleanalytics:master Apr 9, 2026
6 checks passed
This file contains hidden or 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
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


Back | FazBrowse Home | New Git URL