Good that I decided to introduce unit tests to CORE because I found multiple issues in the Scene Manager which I was now able to fix. See 52c6d029d9, 25c4667a05 and 32b82c8f5b.
I somehow fucked up and thought that the 'core' variable in unitbase.gd is used for some other stuff and therefore assigned it a new variable called 'core_test'. Seems like I was wrong however and slipped up the code I wrote just days before. Anyway, fixed this slip up, have a nice day :)
A better version of the 'z_cleanup' test has been integrated into Bessere Tests in the latest commit (as of writing this commit message). Therefore this test is no longer required as Bessere Tests will warn us on shutdown for any leftover children or orphan nodes from now on.
Changes in this commit include:
- Add Core.cleanup() and CoreBaseModule._cleanup() functions
- All builtin modules now clean and free any nodes
- Reduce code duplication in src/core.gd by not writing the same code for all builtin modules but instead iterating through a array
The edl module will now only log the amount of headers (create_download() and start_download()) and it will no longer print the full request body into the console. It will now instead print the amount of characters in the request body (create_download()).
