Skip to content

[3.x] fix unit tests MSSQL (Windows)#359

Merged
richard67 merged 1 commit intojoomla-framework:3.x-devfrom
heelc29:3.x/ci-fix-mssql-win
Feb 17, 2026
Merged

[3.x] fix unit tests MSSQL (Windows)#359
richard67 merged 1 commit intojoomla-framework:3.x-devfrom
heelc29:3.x/ci-fix-mssql-win

Conversation

@heelc29
Copy link
Contributor

@heelc29 heelc29 commented Feb 17, 2026

Summary of Changes

Currently, dependencies that may be incompatible with the php version are being installed during the MSSQL (win) tests.

In this particular case, doctrine/instantiator is required by phpunit, which causes the error for php 8.1 and 8.2.
Here is the relevant commit which requires a newer php version doctrine/instantiator@21dcb5f

The new syntax requires php 8.3 https://www.php.net/releases/8.3/en.php#typed_class_constants

private const SERIALIZATION_FORMAT_USE_UNSERIALIZER   = 'C';
private const SERIALIZATION_FORMAT_AVOID_UNSERIALIZER = 'O';
private const string SERIALIZATION_FORMAT_USE_UNSERIALIZER   = 'C';
private const string SERIALIZATION_FORMAT_AVOID_UNSERIALIZER = 'O';

So, as at the other tests, we should use composer update instead of composer install --ignore-platform-reqs.

Testing Instructions

unit tests

Documentation Changes Required

no

@richard67 richard67 merged commit 9eb4a44 into joomla-framework:3.x-dev Feb 17, 2026
33 checks passed
@richard67
Copy link
Contributor

Thanks.

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.

2 participants

Comments