Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
View all resources
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  GitHub Stars
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

WordPress / two-factor Public

Notifications You must be signed in to change notification settings
Fork 177
Star 808

Code
Issues 42
Pull requests 25
Actions
Projects
Wiki
Security and quality
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Wiki
Security and quality
Insights

Fix Action unit testing#624

Merged

dd32 merged 51 commits into

WordPress:masterWordPress/two-factor:masterfrom

dd32:fix/builddd32/two-factor:fix/buildCopy head branch name to clipboard

Aug 22, 2024

Conversation Commits51 (51)Checks Files changed

Merged

Fix Action unit testing#624
dd32 merged 51 commits into
WordPress:masterfrom
dd32:fix/build

Commits

Commits on Aug 20, 2024

Use composer within the cli container.
dd32
committed
Update wp-env.
dd32
committed
Update the node requirement to 18.
dd32
committed
Update composer
dd32
committed
Update wp-env, again.
dd32
committed
Try running phpunit via the container.
dd32
committed
Try running phpunit via the container.
dd32
committed
PHP 7.2
dd32
committed
rm local wordpress checkout.
dd32
committed
rm wordpress
dd32
committed
update composer
dd32
committed
Import tests_add_filter() via the same method as Gutenberg: https://github.com/WordPress/gutenberg/blob/9c9f6535cdc3cd57a3b8583ea6292a2c90552ab3/phpunit/bootstrap.php#L41
dd32
committed
update composer
dd32
committed
update
dd32
committed
update packages
dd32
committed
Split test into multiple jobs
dd32
committed
Drop PHP 7.2 test for now, wp-env doesn't seem to install the proper phpunit for that
dd32
committed
disable wp_redirect()
dd32
committed
Fix email sending.
dd32
committed
Don't need to override wp_redirect, the PHP warnings don't cause a failure.
dd32
committed
Updates
dd32
committed
Restore PHP 7.2
dd32
committed
Add PHP 8.3
dd32
committed
Don't need to be compatible with PHP 5.6
dd32
committed
add trunk, include versions.
dd32
committed
PHP 7.2+ for compat
dd32
committed
Reverse order of PHP's
dd32
committed
Fix Build
dd32
committed
Revert the parse-gitignore package back to 0.5.1, v1 and v2 don't work the same way.
dd32
committed
Restore the phpunit-watcher
dd32
committed
Remove the old wp_deploy code.
dd32
committed
Gruntfile: Remove isTruthy.
dd32
committed
Run tests on the tests container, such as not to affect the WordPress env
dd32
committed
qrcode-generator doesn't need to be listed as a devDependency
dd32
committed
Try adding a WP 4.6 / PHP 5.6-7.1 job due to the plugin claiming to support it.
dd32
committed
Try a more recent older WP version.
dd32
committed
Drop PHP <7.2 tests, not possible with wp-env due to core support.
dd32
committed
Remove reference to /wordpress/
dd32
committed

Commits on Aug 21, 2024

Retry the wp-env bootup for unit testing, as this appears to fail sporadically
dd32
committed
Update composer versions to >=7.2.24 rather than 5.6, per #625
dd32
committed
Update github actions to newer node20 versions.
dd32
committed
Ensure that the environment is started with xdebug coverage support for testing.
dd32
committed
Lock to PHPUnit 9.6, this requires claiming a PHP 7.4 platform.
dd32
committed
Remove grunt deploy
dd32
committed
Don't need coverage support in Actions.
dd32
committed
Take {PHP_INT_MAX}.
dd32
committed
Correct the wp-env command for phpunit.
dd32
committed

Commits on Aug 22, 2024

set wp-env tests constants for the test environment, to ensure that emails can be sent without needing a filter override.
dd32
committed
Run tests with WP_DEBUG enabled, such as not to need to include unit-test customisations to plugin.
dd32
committed
Merge branch 'master' into fix/build
dd32
committed
Merge branch 'master' into fix/build
dd32
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.