mirror of
https://github.com/Bubberstation/Bubberstation.git
synced 2026-07-11 16:14:08 +01:00
e7a01b6b27
## About The Pull Request As per the title. The unit test was never skipping any typepaths, regardless of whether or not they were defined in the modular folders. This made it so that even TG clothing was being checked. The regex which was used to find the typepaths was also broken, as it was prematurely cutting off at the letter `s`, and was also matching with typepaths that weren't actual definitions/overwrites. It also stopped searching for typepaths in the file after the first one was found. ## Why It's Good For The Game Fixes a unit test. ## Proof Of Testing <img width="258" height="56" alt="image" src="https://github.com/user-attachments/assets/24b8f8d7-42d1-429d-9a48-30eb9ef95e6b" /> It compiled and run the test fine. Checking with the debugger confirmed that the `typepaths_to_check` list (renamed from `typepath_files`) was being properly filled with clothing from `modular_skyrat` and `modular_zubbers` folders. This screenshot shows the status of the `typepath_files` list before the changes: <img width="1703" height="332" alt="image" src="https://github.com/user-attachments/assets/4e400269-016f-4a89-81b0-887b8f44d200" /> As can be seen, it is empty, meaning the `if(subtype_string in typepath_files)` line is never going to fire, because there is no item in the list. This screenshot shows the status of the `typepaths_to_check` list after my changes: <img width="1600" height="548" alt="image" src="https://github.com/user-attachments/assets/f0bf1fed-193d-41c0-950e-3c4285a6db3f" /> as you can see, it has 599 entries, all of which were defined or overwritten in `modular_zubbers` or `modular_skyrat` files.