Unit/System Tests PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
1 Apr 2024

Alternatives for #43184

avatar bembelimen bembelimen - open - 1 Apr 2024
avatar bembelimen bembelimen - change - 1 Apr 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Apr 2024
Category Libraries
avatar bembelimen bembelimen - change - 1 Apr 2024
The description was changed
avatar bembelimen bembelimen - edited - 1 Apr 2024
avatar bembelimen bembelimen - change - 1 Apr 2024
Labels Added: PR-5.1-dev
avatar LadySolveig LadySolveig - test_item - 1 Apr 2024 - Tested successfully
avatar LadySolveig
LadySolveig - comment - 1 Apr 2024

I have tested this item ✅ successfully on e9e03f8


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

avatar richard67
richard67 - comment - 1 Apr 2024

@bembelimen It seems this PR breaks the unit tests for the constraint checker.

avatar joomla-cms-bot joomla-cms-bot - change - 1 Apr 2024
Category Libraries Libraries Unit Tests
avatar bembelimen bembelimen - change - 1 Apr 2024
Labels Added: Unit/System Tests
avatar laoneo
laoneo - comment - 2 Apr 2024

Why did you revert the final statement? Sounds logic to me instead of the long description that this class is internal only.

avatar bembelimen
bembelimen - comment - 2 Apr 2024

I'm not keen to rewrite and fix all the tests, when I declare them as final (plus, I don't like to close classes that way). Feel free to look into it, but I don't plan to do it before the RC.

avatar LadySolveig LadySolveig - close - 2 Apr 2024
avatar LadySolveig LadySolveig - merge - 2 Apr 2024
avatar LadySolveig LadySolveig - change - 2 Apr 2024
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-04-02 09:12:28
Closed_By LadySolveig
avatar LadySolveig
LadySolveig - comment - 2 Apr 2024

Thank you!

Add a Comment

Login with GitHub to post a comment