-
-
Notifications
You must be signed in to change notification settings - Fork 447
Description
ForgeGradle 7.0 and later need to have functional tests written for it to avoid regressions that may be caused by updates to the Minecraft Mavenizer, Slime Launcher or ForgeGradle itself.
This is especially important in terms of how ForgeGradle handles Mixin and its compatibility with projects. Earlier incidents (#1026 is not the only one) have proven that there are unbearably high expectations on ForgeGradle to always remain functional with regards to Mixin and its usage in projects and at runtime. Any potential regressions to Mixin functionality in ForgeGradle threaten to be a massive headache moving forward. This needs to be addressed and tests need to be made.
Note
Gradle's User Guide has a page on Testing Plugins. This will serve as a good starting point to write tests for all of our projects, including ForgeGradle, though all of our projects should get the same love with regards to tests and buildscript cleanliness.