Clean and fix runGameTestClient task

Author: dogboy21

build.gradle

@@ -184,15 +184,6 @@ minecraft {
             }
         }
 
-        // internal run config to automatically run the client-side gametests with the runGameTestClient task
-        runGameTestClientInternal {
-            parent runs.testClient
-
-            lazyToken("minecraft_classpath", lazyTokens["minecraft_classpath"])
-
-            property 'advancedperipheralstest.tags', 'client'
-        }
-
         gameTestServer {
             def oldClasspath = lazyTokens["minecraft_classpath"]
             lazyToken("minecraft_classpath") {
@@ -677,6 +668,9 @@ publishing {
 }
 
 tasks.register('runGameTestClient', ClientJavaExec, { task ->
+    task.outputs.upToDateWhen { false }
+
     description "Runs client-side gametests with no mods"
-    setRunConfig(minecraft.runs.runGameTestClientInternal)
+    setRunConfig(minecraft.runs.testClient)
+    tags("client")
 })

buildSrc/src/main/kotlin/cc/tweaked/gradle/MinecraftExec.kt

@@ -14,6 +14,7 @@ import org.gradle.api.services.BuildServiceParameters
 import org.gradle.api.tasks.*
 import org.gradle.kotlin.dsl.getByName
 import org.gradle.language.base.plugins.LifecycleBasePlugin
+import org.gradle.process.CommandLineArgumentProvider
 import java.io.File
 import java.nio.file.Files
 import java.util.concurrent.TimeUnit
@@ -91,7 +92,11 @@ abstract class ClientJavaExec : JavaExec() {
      * Only run tests with the given tags.
      */
     fun tags(vararg tags: String) {
-        systemProperty("advancedperipheralstest.tags", tags.joinToString(","))
+        jvmArgumentProviders.add(
+            CommandLineArgumentProvider {
+                listOf("-Dadvancedperipheralstest.tags=${tags.joinToString(",")}")
+            }
+        )
     }
 
     /**