add service loader tests for profiler and profile converter

Author: lbloder

sentry-async-profiler/api/sentry-async-profiler.api

@@ -30,6 +30,11 @@ public final class io/sentry/asyncprofiler/provider/AsyncProfilerProfileConverte
 	public fun getProfileConverter ()Lio/sentry/IProfileConverter;
 }
 
+public final class io/sentry/asyncprofiler/provider/AsyncProfilerProfileConverterProvider$AsyncProfilerProfileConverter : io/sentry/IProfileConverter {
+	public fun <init> ()V
+	public fun convertFromFile (Ljava/nio/file/Path;)Lio/sentry/protocol/profiling/SentryProfile;
+}
+
 public final class io/sentry/asyncprofiler/vendor/asyncprofiler/convert/Arguments {
 	public field alloc Z
 	public field bci Z

sentry-async-profiler/src/main/java/io/sentry/asyncprofiler/provider/AsyncProfilerProfileConverterProvider.java

@@ -21,7 +21,7 @@ public final class AsyncProfilerProfileConverterProvider implements JavaProfileC
    * Internal implementation of IProfileConverter that delegates to
    * JfrAsyncProfilerToSentryProfileConverter.
    */
-  private static final class AsyncProfilerProfileConverter implements IProfileConverter {
+  public static final class AsyncProfilerProfileConverter implements IProfileConverter {
 
     @Override
     public @NotNull io.sentry.protocol.profiling.SentryProfile convertFromFile(

sentry-async-profiler/src/test/java/io/sentry/asyncprofiler/JavaContinuousProfilingServiceLoaderTest.kt

@@ -0,0 +1,24 @@
+package io.sentry.asyncprofiler
+
+import io.sentry.ILogger
+import io.sentry.asyncprofiler.profiling.JavaContinuousProfiler
+import io.sentry.asyncprofiler.provider.AsyncProfilerProfileConverterProvider
+import io.sentry.profiling.ProfilingServiceLoader
+import kotlin.test.Test
+import org.mockito.kotlin.mock
+
+class JavaContinuousProfilingServiceLoaderTest {
+  @Test
+  fun loadsAsyncProfilerProfileConverter() {
+    val service = ProfilingServiceLoader.loadProfileConverter()
+    assert(service is AsyncProfilerProfileConverterProvider.AsyncProfilerProfileConverter)
+  }
+
+  @Test
+  fun loadsJavaAsyncProfiler() {
+    val logger = mock<ILogger>()
+
+    val service = ProfilingServiceLoader.loadContinuousProfiler(logger, "", 10, null)
+    assert(service is JavaContinuousProfiler)
+  }
+}

sentry/src/test/java/io/sentry/profiling/ProfilingServiceLoaderTest.kt

@@ -0,0 +1,78 @@
+package io.sentry.profiling
+
+import io.sentry.IContinuousProfiler
+import io.sentry.ILogger
+import io.sentry.IProfileConverter
+import io.sentry.ISentryExecutorService
+import io.sentry.ProfileLifecycle
+import io.sentry.TracesSampler
+import io.sentry.protocol.SentryId
+import io.sentry.protocol.profiling.SentryProfile
+import java.nio.file.Path
+import kotlin.test.Test
+import org.mockito.kotlin.mock
+
+class ProfilingServiceLoaderTest {
+  @Test
+  fun loadsProfileConverterStub() {
+    val service = ProfilingServiceLoader.loadProfileConverter()
+    assert(service is ProfileConverterStub)
+  }
+
+  @Test
+  fun loadsProfilerStub() {
+    val logger = mock<ILogger>()
+
+    val service = ProfilingServiceLoader.loadContinuousProfiler(logger, "", 10, null)
+    assert(service is ContinuousProfilerStub)
+  }
+}
+
+class JavaProfileConverterProviderStub : JavaProfileConverterProvider {
+  override fun getProfileConverter(): IProfileConverter? {
+    return ProfileConverterStub()
+  }
+}
+
+class ProfileConverterStub() : IProfileConverter {
+  override fun convertFromFile(jfrFilePath: Path): SentryProfile {
+    TODO("Not yet implemented")
+  }
+}
+
+class JavaProfilerProviderStub : JavaContinuousProfilerProvider {
+  override fun getContinuousProfiler(
+    logger: ILogger?,
+    profilingTracesDirPath: String?,
+    profilingTracesHz: Int,
+    executorService: ISentryExecutorService?,
+  ): IContinuousProfiler {
+    return ContinuousProfilerStub()
+  }
+}
+
+class ContinuousProfilerStub() : IContinuousProfiler {
+  override fun isRunning(): Boolean {
+    TODO("Not yet implemented")
+  }
+
+  override fun startProfiler(profileLifecycle: ProfileLifecycle, tracesSampler: TracesSampler) {
+    TODO("Not yet implemented")
+  }
+
+  override fun stopProfiler(profileLifecycle: ProfileLifecycle) {
+    TODO("Not yet implemented")
+  }
+
+  override fun close(isTerminating: Boolean) {
+    TODO("Not yet implemented")
+  }
+
+  override fun reevaluateSampling() {
+    TODO("Not yet implemented")
+  }
+
+  override fun getProfilerId(): SentryId {
+    TODO("Not yet implemented")
+  }
+}

sentry/src/test/resources/META-INF/services/io.sentry.profiling.JavaContinuousProfilerProvider

@@ -0,0 +1 @@
+io.sentry.profiling.JavaProfilerProviderStub

sentry/src/test/resources/META-INF/services/io.sentry.profiling.JavaProfileConverterProvider

@@ -0,0 +1 @@
+io.sentry.profiling.JavaProfileConverterProviderStub