Skip to content

PHPUnit 11#4269

Merged
fabpot merged 1 commit into
twigphp:4.xfrom
derrabus:improvement/phpunit-11
Sep 4, 2024
Merged

PHPUnit 11#4269
fabpot merged 1 commit into
twigphp:4.xfrom
derrabus:improvement/phpunit-11

Conversation

@derrabus derrabus force-pushed the improvement/phpunit-11 branch from d5d2825 to 50e5270 Compare September 3, 2024 13:55
Comment thread .gitignore
/doc/_build/vendor
/doc/_build/output
/composer.lock
/phpunit.phar

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This will allow developer to place a PHAR distribution of PHPUnit in the main directory which is picked up and indexed by PHPStorm for code completion.

Comment thread tests/ErrorTest.php Outdated
@derrabus derrabus force-pushed the improvement/phpunit-11 branch from 711396a to d216c22 Compare September 3, 2024 14:25
Comment thread .github/workflows/ci.yml Outdated
Comment thread phpunit.xml.dist
Comment thread phpunit.xml.dist Outdated
Comment thread tests/ErrorTest.php Outdated
@derrabus derrabus force-pushed the improvement/phpunit-11 branch 3 times, most recently from cae01e1 to 61eb1d2 Compare September 3, 2024 15:12
@fabpot fabpot closed this in e829b26 Sep 3, 2024
@fabpot fabpot reopened this Sep 3, 2024
@derrabus derrabus force-pushed the improvement/phpunit-11 branch 2 times, most recently from cc10de6 to 6451425 Compare September 3, 2024 21:17
fabpot added a commit that referenced this pull request Sep 4, 2024
…rabus)

This PR was merged into the 4.x branch.

Discussion
----------

Switch to static data provides for integration tests

This change should make our integration test case compatible with PHPUnit 10 and 11. Merging the PR will hopefully give us a green build on #4269. 🤞🏻

Commits
-------

9282dbc Switch to static data provides for integration tests
@derrabus derrabus force-pushed the improvement/phpunit-11 branch 3 times, most recently from 03b003a to be5913d Compare September 4, 2024 08:34
@derrabus

derrabus commented Sep 4, 2024

Copy link
Copy Markdown
Contributor Author

I don't understand the intl-extra failure. They look like they're failing because of data that could differ between ICU versions, but I'm not aware I have changed anything that could lead to different ICU data being loaded.

Also, I get a different failure localy, but that one is at least consistent between 4.x and this branch. 😓

@derrabus

derrabus commented Sep 4, 2024

Copy link
Copy Markdown
Contributor Author

Apart from that, the test suite is green. 🎉

@derrabus derrabus changed the title WIP: PHPUnit 11 PHPUnit 11 Sep 4, 2024
@stof

stof commented Sep 4, 2024

Copy link
Copy Markdown
Member

Could it be that the LANG env variable set in the phpunit configuration does not work anymore to define the default ext-intl locale ?

@fabpot fabpot force-pushed the improvement/phpunit-11 branch from 7350aea to f2053bb Compare September 4, 2024 10:11
@fabpot

fabpot commented Sep 4, 2024

Copy link
Copy Markdown
Contributor

Thank you @derrabus.

@fabpot fabpot merged commit 2a8cf65 into twigphp:4.x Sep 4, 2024
@derrabus derrabus deleted the improvement/phpunit-11 branch September 4, 2024 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants