Composer Dependency Changed PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
27 Apr 2024

Pull Request for Issue #42528 .

Summary of Changes

Update PHPUnit to the next major version because the current one we are using is outdated

Testing Instructions

Test/Merge after #43381

Unit test should work as before.

avatar rdeutz rdeutz - open - 27 Apr 2024
avatar rdeutz rdeutz - change - 27 Apr 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Apr 2024
Category External Library Composer Change
avatar richard67
richard67 - comment - 27 Apr 2024

@rdeutz It seems you are updating a lot of composer dependencies, see the lock file. Can it be that you have just run a composer update instead of a composer update phpunit/phpunit?

avatar brianteeman
brianteeman - comment - 27 Apr 2024

@richard67 try that and you will see why

avatar brianteeman brianteeman - test_item - 27 Apr 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 27 Apr 2024

I have tested this item ✅ successfully on 4f5f61c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43393.

avatar richard67
richard67 - comment - 27 Apr 2024

I have tested this item ✅ successfully on 4f5f61c

@brianteeman You have noticed that both appveyor and drone are failing due to failing unit tests?

@rdeutz If that is expected because it needs the other PR, then your testing instructions should make that more clear.

avatar rdeutz rdeutz - change - 28 Apr 2024
The description was changed
avatar rdeutz rdeutz - edited - 28 Apr 2024
avatar rdeutz rdeutz - change - 28 Apr 2024
Labels Added: Composer Dependency Changed PR-5.2-dev
avatar brianteeman
brianteeman - comment - 28 Apr 2024

@richard67 not sure what happened but when you replied to my comment you actually edited it. So it looks like I have said what you said. I have deleted it. and paste below

The composer lock updates are correct and if you try to do a composer update phpunit (as I said) then you will see why

Tests failing does not necessarily mean that the PR is wrong. It can also mean that the tests are wrong. We know that there are multiple tests that are either wrong or flakey.

I am well aware that these are failing unit tests which are failing. Thats what I was referring to.


As you can see now from Roberts ammended update the original update was correct. You just need to try it for yourself and you will see that you cannot just update phpunit without also updating a large batch of dependencies.

avatar richard67
richard67 - comment - 28 Apr 2024

not sure what happened but when you replied to my comment you actually edited it.

@brianteeman No idea what happened either, but it was not by purpose.

As you can see now from Roberts ammended update the original update was correct.

I know it needs to update a bunch of them, but in the first attempt there were definitely some included which are no dependencies of PHP unit. Now after the recent 2 commits it seems to be right.

avatar rdeutz
rdeutz - comment - 29 Apr 2024

@rdeutz If that is expected because it needs the other PR, then your testing instructions should make that more clear.

updated

one fail is expected when #43381 is not merged the other ones not

re-did the composer update

avatar rdeutz rdeutz - change - 5 Jun 2024
Title
Update PHPUnit to 10
[5.2] Update PHPUnit to 10
avatar rdeutz rdeutz - edited - 5 Jun 2024
avatar rdeutz
rdeutz - comment - 30 Jul 2024

closed in favor of #43865

avatar richard67
richard67 - comment - 30 Jul 2024

closed in favor of #43865

@rdeutz Forgotten to close?

avatar rdeutz rdeutz - change - 30 Jul 2024
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2024-07-30 11:47:52
Closed_By rdeutz
avatar rdeutz
rdeutz - comment - 30 Jul 2024

yes

avatar rdeutz rdeutz - close - 30 Jul 2024

Add a Comment

Login with GitHub to post a comment