Remove tests directory to fix dependabot errors#2299
Open
IndiraBiswas wants to merge 1 commit intodevelopfrom
Open
Remove tests directory to fix dependabot errors#2299IndiraBiswas wants to merge 1 commit intodevelopfrom
IndiraBiswas wants to merge 1 commit intodevelopfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the entire tests directory from the repository to resolve Dependabot security vulnerabilities stemming from outdated Playwright and Codeception test dependencies. The tests are no longer actively maintained or executed, making removal the most practical solution.
Changes:
- Deletion of all Playwright E2E test files and configuration (tests/wp-e2e-playwright/)
- Deletion of all Codeception acceptance test files and configuration (tests/codeception/)
- Deletion of PHPUnit test bootstrap files (tests/bootstrap.php, tests/rt-wp-test-case.php)
Reviewed changes
Copilot reviewed 111 out of 117 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/wp-e2e-playwright/* | Complete removal of Playwright test suite including specs, utilities, config, and dependencies |
| tests/codeception/* | Complete removal of Codeception test suite including acceptance tests, support classes, and configuration |
| tests/bootstrap.php | Removal of PHPUnit bootstrap file |
| tests/rt-wp-test-case.php | Removal of PHPUnit test case class |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
juhi123
approved these changes
Feb 23, 2026
juhi123
approved these changes
Feb 23, 2026
Contributor
|
@IndiraBiswas Please resolve the conflicts |
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR removes the entire tests directory from the repository.
Why is this change necessary?
The tests folder contains outdated dependencies (specifically Playwright test dependencies) that have triggered multiple security vulnerabilities (1 critical, 3 high, 1 moderate) flagged by Dependabot. Since these tests are no longer actively maintained or executed, removing the folder is the safest and most efficient way to resolve the security alerts without impacting the core plugin functionality.