src/test/kotlin/platform/mixin/InvalidInjectorMethodSignatureInspectionTest.kt diff

← → edited src/test/kotlin/platform/mixin/InvalidInjectorMethodSignatureInspectionTest.kt

Ignore whitespaces Open in IDE

⁠ kyle wood: Fix up all duplicaate code warnings

Also messed up the ktlint configuration trying to upgrade it, but future
versions are still a broken mess. Sorry about that.

Copy Copy

/*
* Minecraft Dev for IntelliJ
*
* https://minecraftdev.org
*
* Copyright (c) 2023 minecraft-dev
*
* MIT License
*/

package com.demonwav.mcdev.platform.mixin

import com.demonwav.mcdev.framework.EdtInterceptor
import com.demonwav.mcdev.platform.mixin.inspection.injector.InvalidInjectorMethodSignatureInspection
import org.intellij.lang.annotations.Language
import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.Test
import org.junit.jupiter.api.extension.ExtendWith

@ExtendWith(EdtInterceptor::class)
@DisplayName("Invalid Injector Method Signature Inspection Test")
class InvalidInjectorMethodSignatureInspectionTest : BaseMixinTest() {

private fun doTest(@Language("JAVA") code: String) {
buildProject {
dir("test") {
java("TestMixin.java", code)
}
}

fixture.enableInspections(InvalidInjectorMethodSignatureInspection::class)
fixture.checkHighlighting(false, false, false)
}

@Test
@DisplayName("Redirect in constructor before superconstructor call")
fun redirectInConstructorBeforeSuperconstructorCall() {
doTest(
"""
package test;

import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Redirect;

@Mixin(MixedInOuter.class)
public class TestMixin {
@Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method1()Ljava/lang/String;"))
private String <error descr="Method must be static">redirectMethod1</error>() {
return null;
}

@Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method2()V"))
private void redirectMethod2() {
}
}
"""
)
}

@Test
@DisplayName("Inner Ctor @Inject Parameters")
fun innerCtorInjectParameters() {
doTest(
"""
package test;

import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Inject;
import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;

@Mixin(MixedInOuter.MixedInInner.class)
public class TestMixin {

@Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;)V", at = @At("RETURN"))
private void injectCtor(MixedInOuter outer, CallbackInfo ci) {
}

@Inject(method = "<init>", at = @At("RETURN"))
private void injectCtor(CallbackInfo ci) {
}

@Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor(MixedInOuter outer, String string, CallbackInfo ci) {
}

@Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(String string, CallbackInfo ci)</error> {
}
}
"""
)
}

@Test
@DisplayName("Static Inner Ctor @Inject Parameters")
fun staticInnerCtorInjectParameters() {
doTest(
"""
package test;

import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Inject;
import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;

@Mixin(MixedInOuter.MixedInStaticInner.class)
public class TestMixin {

@Inject(method = "<init>()V", at = @At("RETURN"))
private void injectCtorWrong<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, CallbackInfo ci)</error> {
}

@Inject(method = "<init>", at = @At("RETURN"))
private void injectCtor(CallbackInfo ci) {
}

@Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, String string, CallbackInfo ci)</error> {
}

@Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor(String string, CallbackInfo ci) {
}
}
"""
)
}
}

/*
* Minecraft Dev for IntelliJ
*
* https://minecraftdev.org
*
* Copyright (c) 2023 minecraft-dev
*
* MIT License
*/

package com.demonwav.mcdev.platform.mixin

import com.demonwav.mcdev.framework.EdtInterceptor
import com.demonwav.mcdev.platform.mixin.inspection.injector.InvalidInjectorMethodSignatureInspection
import org.intellij.lang.annotations.Language
import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.Test
import org.junit.jupiter.api.extension.ExtendWith

@ExtendWith(EdtInterceptor::class)
@DisplayName("Invalid Injector Method Signature Inspection Test")
class InvalidInjectorMethodSignatureInspectionTest : BaseMixinTest() {

private fun doTest(@Language("JAVA") code: String) {
buildProject {
dir("test") {
java("TestMixin.java", code)
}
}

fixture.enableInspections(InvalidInjectorMethodSignatureInspection::class)
fixture.checkHighlighting(false, false, false)
}

@Test
@DisplayName("Redirect in constructor before superconstructor call")
fun redirectInConstructorBeforeSuperconstructorCall() {
doTest(
"""
package test;

import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Redirect;

@Mixin(MixedInOuter.class)
public class TestMixin {
@Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method1()Ljava/lang/String;"))
private String <error descr="Method must be static">redirectMethod1</error>() {
return null;
}

@Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method2()V"))
private void redirectMethod2() {
}
}
""",
)
}

@Test
@DisplayName("Inner Ctor @Inject Parameters")
fun innerCtorInjectParameters() {
doTest(
"""
package test;

import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Inject;
import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;

@Mixin(MixedInOuter.MixedInInner.class)
public class TestMixin {

@Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;)V", at = @At("RETURN"))
private void injectCtor(MixedInOuter outer, CallbackInfo ci) {
}

@Inject(method = "<init>", at = @At("RETURN"))
private void injectCtor(CallbackInfo ci) {
}

@Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor(MixedInOuter outer, String string, CallbackInfo ci) {
}

@Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(String string, CallbackInfo ci)</error> {
}
}
""",
)
}

@Test
@DisplayName("Static Inner Ctor @Inject Parameters")
fun staticInnerCtorInjectParameters() {
doTest(
"""
package test;

import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Inject;
import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;

@Mixin(MixedInOuter.MixedInStaticInner.class)
public class TestMixin {

@Inject(method = "<init>()V", at = @At("RETURN"))
private void injectCtorWrong<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, CallbackInfo ci)</error> {
}

@Inject(method = "<init>", at = @At("RETURN"))
private void injectCtor(CallbackInfo ci) {
}

@Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, String string, CallbackInfo ci)</error> {
}

@Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN"))
private void injectCtor(String string, CallbackInfo ci) {
}
}
""",
)
}
}

/* * Minecraft Dev for IntelliJ * * https://minecraftdev.org * * Copyright (c) 2023 minecraft-dev * * MIT License */ package com.demonwav.mcdev.platform.mixin import com.demonwav.mcdev.framework.EdtInterceptor import com.demonwav.mcdev.platform.mixin.inspection.injector.InvalidInjectorMethodSignatureInspection import org.intellij.lang.annotations.Language import org.junit.jupiter.api.DisplayName import org.junit.jupiter.api.Test import org.junit.jupiter.api.extension.ExtendWith @ExtendWith(EdtInterceptor::class) @DisplayName("Invalid Injector Method Signature Inspection Test") class InvalidInjectorMethodSignatureInspectionTest : BaseMixinTest() { private fun doTest(@Language("JAVA") code: String) { buildProject { dir("test") { java("TestMixin.java", code) } } fixture.enableInspections(InvalidInjectorMethodSignatureInspection::class) fixture.checkHighlighting(false, false, false) } @Test @DisplayName("Redirect in constructor before superconstructor call") fun redirectInConstructorBeforeSuperconstructorCall() { doTest( """ package test; import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Redirect; @Mixin(MixedInOuter.class) public class TestMixin { @Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method1()Ljava/lang/String;")) private String <error descr="Method must be static">redirectMethod1</error>() { return null; } @Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method2()V")) private void redirectMethod2() { } } """ ) } @Test @DisplayName("Inner Ctor @Inject Parameters") fun innerCtorInjectParameters() { doTest( """ package test; import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Inject; import org.spongepowered.asm.mixin.injection.callback.CallbackInfo; @Mixin(MixedInOuter.MixedInInner.class) public class TestMixin { @Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;)V", at = @At("RETURN")) private void injectCtor(MixedInOuter outer, CallbackInfo ci) { } @Inject(method = "<init>", at = @At("RETURN")) private void injectCtor(CallbackInfo ci) { } @Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor(MixedInOuter outer, String string, CallbackInfo ci) { } @Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(String string, CallbackInfo ci)</error> { } } """ ) } @Test @DisplayName("Static Inner Ctor @Inject Parameters") fun staticInnerCtorInjectParameters() { doTest( """ package test; import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Inject; import org.spongepowered.asm.mixin.injection.callback.CallbackInfo; @Mixin(MixedInOuter.MixedInStaticInner.class) public class TestMixin { @Inject(method = "<init>()V", at = @At("RETURN")) private void injectCtorWrong<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, CallbackInfo ci)</error> { } @Inject(method = "<init>", at = @At("RETURN")) private void injectCtor(CallbackInfo ci) { } @Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, String string, CallbackInfo ci)</error> { } @Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor(String string, CallbackInfo ci) { } } """ ) } } /* * Minecraft Dev for IntelliJ * * https://minecraftdev.org * * Copyright (c) 2023 minecraft-dev * * MIT License */ package com.demonwav.mcdev.platform.mixin import com.demonwav.mcdev.framework.EdtInterceptor import com.demonwav.mcdev.platform.mixin.inspection.injector.InvalidInjectorMethodSignatureInspection import org.intellij.lang.annotations.Language import org.junit.jupiter.api.DisplayName import org.junit.jupiter.api.Test import org.junit.jupiter.api.extension.ExtendWith @ExtendWith(EdtInterceptor::class) @DisplayName("Invalid Injector Method Signature Inspection Test") class InvalidInjectorMethodSignatureInspectionTest : BaseMixinTest() { private fun doTest(@Language("JAVA") code: String) { buildProject { dir("test") { java("TestMixin.java", code) } } fixture.enableInspections(InvalidInjectorMethodSignatureInspection::class) fixture.checkHighlighting(false, false, false) } @Test @DisplayName("Redirect in constructor before superconstructor call") fun redirectInConstructorBeforeSuperconstructorCall() { doTest( """ package test; import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Redirect; @Mixin(MixedInOuter.class) public class TestMixin { @Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method1()Ljava/lang/String;")) private String <error descr="Method must be static">redirectMethod1</error>() { return null; } @Redirect(method = "<init>()V", at = @At(value = "INVOKE", target = "Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;method2()V")) private void redirectMethod2() { } } """, ) } @Test @DisplayName("Inner Ctor @Inject Parameters") fun innerCtorInjectParameters() { doTest( """ package test; import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Inject; import org.spongepowered.asm.mixin.injection.callback.CallbackInfo; @Mixin(MixedInOuter.MixedInInner.class) public class TestMixin { @Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;)V", at = @At("RETURN")) private void injectCtor(MixedInOuter outer, CallbackInfo ci) { } @Inject(method = "<init>", at = @At("RETURN")) private void injectCtor(CallbackInfo ci) { } @Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor(MixedInOuter outer, String string, CallbackInfo ci) { } @Inject(method = "<init>(Lcom/demonwav/mcdev/mixintestdata/invalidInjectorMethodSignatureInspection/MixedInOuter;Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(String string, CallbackInfo ci)</error> { } } """, ) } @Test @DisplayName("Static Inner Ctor @Inject Parameters") fun staticInnerCtorInjectParameters() { doTest( """ package test; import com.demonwav.mcdev.mixintestdata.invalidInjectorMethodSignatureInspection.MixedInOuter; import org.spongepowered.asm.mixin.Mixin; import org.spongepowered.asm.mixin.injection.At; import org.spongepowered.asm.mixin.injection.Inject; import org.spongepowered.asm.mixin.injection.callback.CallbackInfo; @Mixin(MixedInOuter.MixedInStaticInner.class) public class TestMixin { @Inject(method = "<init>()V", at = @At("RETURN")) private void injectCtorWrong<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, CallbackInfo ci)</error> { } @Inject(method = "<init>", at = @At("RETURN")) private void injectCtor(CallbackInfo ci) { } @Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor<error descr="Method parameters do not match expected parameters for Inject">(MixedInOuter outer, String string, CallbackInfo ci)</error> { } @Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN")) private void injectCtor(String string, CallbackInfo ci) { } } """, ) } }

Add build comment

Mute test

Server communication failure

Run Custom Build

TODO

Responsibility

Edit tags

Loading...

Are you sure?