Unify wordpress - #4261
Unify wordpress#4261
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: 0e34e1f | Docs | View more details | Give us feedback! |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The shared fixture preserves the previous AppSec setup while adding focused automated-user-event coverage.
Review effort: Balanced
Findings: None
What changed in this PR
Unifies AppSec WordPress tests around the shared WordPress 6.1 fixture.
Changes:
- Adds shared AppSec trigger scripts and Apache configuration.
- Reuses the shared fixture from
WordPressTests. - Aligns database configuration and removes redundant AppSec files.
| File | Description |
|---|---|
tests/Frameworks/WordPress/Version_6_1/wp-config.php |
Makes the AppSec mock optional. |
tests/Frameworks/WordPress/Version_6_1/signup_trigger.php |
Adds signup-event trigger. |
tests/Frameworks/WordPress/Version_6_1/login_trigger.php |
Adds authentication trigger. |
tests/Frameworks/WordPress/Version_6_1/index.php |
Makes the AppSec mock optional. |
tests/Frameworks/WordPress/Version_6_1/hello.php |
Adds response-header test endpoint. |
tests/Frameworks/WordPress/Version_6_1/docker-init.sh |
Prepares the shared fixture for AppSec Apache. |
tests/Frameworks/WordPress/Version_6_1/behind_auth_trigger.php |
Adds authenticated-user trigger. |
tests/Frameworks/WordPress/Version_6_1/.htaccess |
Adds WordPress routing rules. |
appsec/tests/integration/src/test/www/wordpress/public/wp-config.php |
Removes duplicate configuration. |
appsec/tests/integration/src/test/www/wordpress/public/index.php |
Removes duplicate entry point. |
appsec/tests/integration/src/test/www/wordpress/docker-init.sh |
Removes superseded initialization. |
appsec/tests/integration/src/test/groovy/com/datadog/appsec/php/integration/WordPressTests.groovy |
Uses the shared fixture and aligned database settings. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Description
This PR unifies WordPress 6.1 test fixtures as the fourth step of #4149:
appsec/tests/integration/src/test/www/wordpress61/copytests/Frameworks/WordPress/Version_6_1/:docker-init.shso the AppSec container can bootstrap the app end-to-end:composer install, SQLite DB setup, and adatabase.local.phpoverride of the tracer's MySQL global configtest_web_wordpress_61tracer jobs and all AppSec integration test jobs passNext steps
Not doing