Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Try to resolve php unit test issues - WP-ENV 8.1.1 #2544

Merged
merged 1 commit into from Jul 28, 2023

Conversation

AnthonyLedesma
Copy link
Member

@AnthonyLedesma AnthonyLedesma commented Jul 28, 2023

Description

The only change here is to update wp-env to ^8.1.1

Confirmed that PHP Unit tests 8.1 + 8.1 are passing.

Types of changes

Package updates for test failures.

How has this been tested?

Pipelines.

Acceptance criteria

Should allow PHP Unit tests to pass for 8.1 + 8.2

Checklist:

  • My code is tested
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

@AnthonyLedesma AnthonyLedesma self-assigned this Jul 28, 2023
@AnthonyLedesma AnthonyLedesma merged commit ee35c4c into master Jul 28, 2023
104 of 152 checks passed
@AnthonyLedesma AnthonyLedesma deleted the try-fix-unit-wp-env branch July 28, 2023 17:26
@AnthonyLedesma
Copy link
Member Author

Looks like the wp-env change, had introduced a new bug when running yarn setup

$ wp-env run wordpress 'composer install -d /var/www/html/wp-content/plugins/coblocks/'
ℹ Starting 'composer install -d /var/www/html/wp-content/plugins/coblocks/' on the wordpress container. 

OCI runtime exec failed: exec failed: unable to start container process: exec: "composer install -d /var/www/html/wp-content/plugins/coblocks/": stat composer install -d /var/www/html/wp-content/plugins/coblocks/: no such file or directory: unknown
✖ Command failed with exit code 126
Command failed with exit code 126

@AnthonyLedesma AnthonyLedesma added this to the 3.1.0 milestone Aug 7, 2023
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.

None yet

1 participant