Skip to content

Add wp-env integration tests for Handbook plugin#559

Closed
dd32 wants to merge 1 commit intoWordPress:trunkfrom
dd32:fix/claude/wp-env-tests-ci
Closed

Add wp-env integration tests for Handbook plugin#559
dd32 wants to merge 1 commit intoWordPress:trunkfrom
dd32:fix/claude/wp-env-tests-ci

Conversation

@dd32
Copy link
Member

@dd32 dd32 commented Mar 10, 2026

Summary

  • Adds wp-env-based CI job for Handbook plugin integration tests using Docker
  • Installs Yoast PHPUnit Polyfills in the wp-env test container (required by WP test suite)
  • Updates Handbook test bootstrap:
    • Adds wp-env path fallback (/wordpress-phpunit/)
    • Auto-detects polyfills path from the WP test directory
  • Adds void return types to setUp()/tearDown() for PHPUnit 9+ compatibility
  • Updates sidebar assertion for WordPress show_in_rest key

Test plan

  • All standalone PHP tests pass
  • Handbook wp-env integration tests pass (89 tests)

🤖 Generated with Claude Code

@github-actions
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props dd32.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Add a wp-env-based CI job for the Handbook plugin tests alongside
the existing standalone PHP tests.

- Add wp-env path fallback to Handbook bootstrap
- Add void return types to setUp/tearDown for PHPUnit 9+ compat
- Update sidebar assertion for WordPress show_in_rest key

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dd32 dd32 force-pushed the fix/claude/wp-env-tests-ci branch from e508fda to 8f1d21b Compare March 10, 2026 05:10
@bazza bazza closed this in 5cd00ba Mar 10, 2026
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.

1 participant