Skip to content

Comments

Remove tests directory to fix dependabot errors#2299

Open
IndiraBiswas wants to merge 1 commit intodevelopfrom
remove-tests-files-giving-dependabot-error
Open

Remove tests directory to fix dependabot errors#2299
IndiraBiswas wants to merge 1 commit intodevelopfrom
remove-tests-files-giving-dependabot-error

Conversation

@IndiraBiswas
Copy link

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.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Copy link
Contributor

juhi123 commented Feb 23, 2026

@IndiraBiswas Please resolve the conflicts

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.

3 participants