[ci] add short class fixture names check to avoid node class names

[TomasVotruba]

• move internal /bin files to /scripts, to separate from public /bin API
• decouple script helpers
• [ci] add short class fixture names check to avoid node class names

[samsonasik]

this fixture test is needed, I can create separate PR for update fixture with new class name to not conflict with Node name :)

[TomasVotruba]

Same here. We already have this case covered. I remove these fixtures on purpose, to make tests more readable and maintainable.

[samsonasik]

no, loop of multi properties is special :), because it already too early imported on class level, I will find a referenced PR :)

[samsonasik]

Here the issue and PR that it fixed:

• AnnotationToAttributeRector only replace the first annotation when using use statement aliases rector#9114
• [Php80] Handle aliased with multi properties on AnnotationToAttributeRector #6838

the fixture is needed to avoid the patch removed in the future.

[TomasVotruba]

@samsonasik It should at least re-use existing configuration. No need to add new configuration item, morever of non-existing class. That makes test less readable.

[samsonasik]

sure, I will add to use existing config 👍

[samsonasik]

see PR:

• Add back removed fixture aliased multi properties on AnnotationToAttributeRector #7549

[samsonasik]

this fixture test is needed, I can create separate PR for update fixture with new class name to not conflict with Node name :)

[TomasVotruba]

This is on purpose, not to create fixture per every possible item, but make them readable.
Don't add anything here.

[github-actions]

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.