This is not the correct way to do it. see this example

https://github.com/joomla/joomla-cms/blob/5.4-dev/build/build.php#L129-L130

Thanks for the guidance. I’ve updated the build cleanup rules to follow the existing pattern as suggested.

I think you will find it better to make the change here

https://github.com/joomla/joomla-cms/blob/5.4-dev/build/build.php#L169-L173

run_and_check('rm -rf libraries/vendor/symfony/*/Test');

That way it will also pick up libraries\vendor\symfony\service-contracts\Test

Thanks for the suggestion. I’ve updated the build cleanup to use a wildcard removal for Symfony Test directories, following the existing cleanup pattern.

Thanks for the guidance. I’ve updated the cleanup section so the Symfony */Test directories are removed before the */Tests rule, and I’ve removed the extra comment as suggested.
This now follows the existing ordering pattern and also correctly handles additional Symfony test folders like service-contracts/Test.

Please let me know if anything else needs adjusting.

I’ve applied your suggested change and committed the updated ordering. Thanks for the review!

sorry I must have made a mistake - please put the tab back and remove the empty line

Then it should be complete

Fixed formatting as requested (tab restored, empty line removed).
Ready for final review. Thanks!

I have tested this item ✅ successfully on 2798e8e

Thanks for testing and confirming. Ready for merge when convenient.

I have tested this item ✅ successfully on 2798e8e

I have tested this PR successfully
Joomla Version: 5.4.3-dev
PHP Version: 8.2.12
OS: Windows 11
Environment: XAMPP
Test Results
All Test directories were properly identified
The wildcard pattern approach works correctly
No errors during testing

I have tested this item ✅ successfully on 2798e8e

According to I study and anlyze the code according to that i will say given code is correct and i will propose that if you can add some comands in clean_checkout function so that it can remove libraries/vendor/symfony/http-client-contracts/Test.

Thanks for testing and the suggestion 👍

The directory libraries/vendor/symfony/http-client-contracts/Test is already covered by the existing wildcard cleanup:

libraries/vendor/symfony/*/Test

This was added intentionally to handle current and future Symfony packages consistently, so no additional command should be required.