Playwright and phpunit tests [TMZ-733]

[gsarig]

✨ PR Description

Purpose: Add PHPUnit and Playwright test infrastructure to enable automated testing for the Hello Elementor theme.

Main changes:

• Added PHPUnit configuration and basic test structure for PHP testing
• Implemented Playwright framework with TypeScript support for UI/E2E testing
• Created CI workflows for test automation on GitHub Actions
• Added utility scripts and configuration files for local test environment setup

Generated by LinearB AI and added by gitStream.
_{AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀}

[gitstream-cm]

✨ PR Review

The PR introduces comprehensive Playwright and PHPUnit testing infrastructure with extensive test utilities, configuration files, and page object models. The code appears well-structured but contains a few potential issues that should be addressed.

3 issues detected:

🐞 Bug - Incorrect grammar in error message reduces code professionalism and clarity.

Details: Line 6 contains a grammatical error in the error message that could confuse users trying to set up the testing environment.
File: tests/bootstrap.php (6-6)

🐞 Bug - Using for-in loop on arrays iterates over indices as strings, potentially causing incorrect array access.

Details: The deleteMedia method uses for (const id in ids) which iterates over array indices as strings, but then uses ids[id] assuming numeric indexing.
File: tests/playwright/assets/api-requests.ts (70-70)

🐞 Bug - The assignment `retry = retry++` doesn't increment the variable due to operator precedence, potentially causing an infinite loop.

Details: Line 1239 contains retry = retry++ which assigns the pre-increment value back to retry, effectively making it an infinite loop since retry never actually increments.
File: tests/playwright/pages/editor-page.ts (1239-1239)

Generated by LinearB AI and added by gitStream.
_{AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀}

[gitstream-cm]

🐞 Bug - Grammar Error: Change "Installing composer are required" to "Installing composer is required" or "Composer dependencies are required".

Suggested change

	die('Installing composer are required for running the tests.');
	die('Installing composer is required for running the tests.');

[gitstream-cm]

✨ PR Review

LGTM

Generated by LinearB AI and added by gitStream.
_{AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀}