refactor(test): logging

Author: BFergerson

build.gradle.kts

@@ -65,10 +65,17 @@ dependencies {
     implementation("eu.geekplace.javapinning:java-pinning-core:1.2.0")
 
     testImplementation("org.junit.jupiter:junit-jupiter-engine:$jupiterVersion")
+    testImplementation("org.slf4j:slf4j-simple:$slf4jVersion")
 
     detektPlugins("io.gitlab.arturbosch.detekt:detekt-formatting:1.23.0")
 }
 
+//remove slf4j-nop from test classpath (use slf4j-simple instead)
+tasks.withType<Test> {
+    useJUnitPlatform()
+    classpath = classpath.filter { !it.toString().contains("slf4j-nop") }
+}
+
 configure<app.cash.licensee.LicenseeExtension> {
     ignoreDependencies("plus.sourceplus", "protocol")
     allow("Apache-2.0")

src/test/kotlin/integration/BatchLiveInstrumentCLI.kt

@@ -47,6 +47,7 @@ class BatchLiveInstrumentCLI : CLIIntegrationTest() {
             addedLiveBps.add(addedLiveBp)
             assertNotNull(addedLiveBp.id)
             interceptor.clear()
+            log.info("Added live breakpoint: ${addedLiveBp.id}")
         }
 
         //get live bps

src/test/kotlin/integration/CLIIntegrationTest.kt

@@ -19,6 +19,8 @@ package integration
 import io.vertx.core.json.Json
 import io.vertx.core.json.JsonArray
 import org.junit.jupiter.api.BeforeAll
+import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import spp.cli.Main
 import spp.cli.util.ExitManager
 import spp.protocol.instrument.LiveBreakpoint
@@ -40,6 +42,8 @@ open class CLIIntegrationTest {
         }
     }
 
+    val log: Logger by lazy { LoggerFactory.getLogger(this::class.java.name) }
+
     class Interceptor(out: OutputStream) : PrintStream(out, true) {
         val output = StringBuilder()
 

src/test/resources/simplelogger.properties

@@ -0,0 +1,3 @@
+org.slf4j.simpleLogger.showDateTime=true
+org.slf4j.simpleLogger.dateTimeFormat=yyyy-MM-dd HH:mm:ss.SS
+org.slf4j.simpleLogger.showThreadName=true