Skip to content

Improve PHP 8.4 test bootstrap compatibility#6

Draft
digitalbase wants to merge 4 commits intorock-symphony:masterfrom
digitalbase:fix/php84-test-bootstrap-compat
Draft

Improve PHP 8.4 test bootstrap compatibility#6
digitalbase wants to merge 4 commits intorock-symphony:masterfrom
digitalbase:fix/php84-test-bootstrap-compat

Conversation

@digitalbase
Copy link

Summary

  • modernize test project setup by syncing submodule URLs, switching submodules to HTTPS, and updating prepare_tests.sh for current Symfony task/composer behavior
  • make installer and plugin bootstrap more resilient across Symfony variants used in test fixtures
  • reduce PHP 8.4 deprecation noise in test harness bootstrap and fix explicit plugin path setup in sfPropelDatabaseTest

Context

This branch focuses on unblocking test execution under local PHP 8.4. It does not fully resolve all remaining legacy vendor/runtime incompatibilities yet (notably old Propel/Phing parser/runtime issues), but it moves the suite past several bootstrap/setup blockers.

Validation

  • ran php test/bin/prove.php with local PHP 8.4 (/opt/homebrew/opt/php@8.4/bin/php)
  • confirmed progression from hard bootstrap/setup failures to downstream legacy compatibility failures

@digitalbase
Copy link
Author

Honestly i don't think this should be merged. Wanted to just preserve the state on the fixes it did getting ready for PHP8.4.

There are other issues in the propel/phing packages and we need to upgrade rock-symphony first too

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.

1 participant