Add tests

Author: romtsn

sentry-android-core/src/main/java/io/sentry/android/core/AppState.java

@@ -9,6 +9,7 @@
 import io.sentry.ISentryLifecycleToken;
 import io.sentry.NoOpLogger;
 import io.sentry.SentryLevel;
+import io.sentry.SentryOptions;
 import io.sentry.android.core.internal.util.AndroidThreadChecker;
 import io.sentry.util.AutoClosableReentrantLock;
 import java.io.Closeable;
@@ -36,11 +37,18 @@ private AppState() {}
 
   private volatile @Nullable Boolean inBackground = null;
 
+  @ApiStatus.Internal
   @TestOnly
-  void resetInstance() {
+  public void resetInstance() {
     instance = new AppState();
   }
 
+  @ApiStatus.Internal
+  @TestOnly
+  public LifecycleObserver getLifecycleObserver() {
+    return lifecycleObserver;
+  }
+
   public @Nullable Boolean isInBackground() {
     return inBackground;
   }
@@ -65,7 +73,8 @@ public void removeAppStateListener(final @NotNull AppStateListener listener) {
     }
   }
 
-  void registerLifecycleObserver(final @Nullable SentryAndroidOptions options) {
+  @ApiStatus.Internal
+  public void registerLifecycleObserver(final @Nullable SentryOptions options) {
     if (lifecycleObserver != null) {
       return;
     }
@@ -121,7 +130,8 @@ private void addObserverInternal(final @NotNull ILogger logger) {
     }
   }
 
-  void unregisterLifecycleObserver() {
+  @ApiStatus.Internal
+  public void unregisterLifecycleObserver() {
     if (lifecycleObserver == null) {
       return;
     }
@@ -155,7 +165,8 @@ public void close() throws IOException {
     unregisterLifecycleObserver();
   }
 
-  final class LifecycleObserver implements DefaultLifecycleObserver {
+  @ApiStatus.Internal
+  public final class LifecycleObserver implements DefaultLifecycleObserver {
     final List<AppStateListener> listeners =
         new CopyOnWriteArrayList<AppStateListener>() {
           @Override
@@ -171,6 +182,12 @@ public boolean add(AppStateListener appStateListener) {
           }
         };
 
+    @ApiStatus.Internal
+    @TestOnly
+    public List<AppStateListener> getListeners() {
+      return listeners;
+    }
+
     @Override
     public void onStart(@NonNull LifecycleOwner owner) {
       setInBackground(false);

sentry-android-core/src/main/java/io/sentry/android/core/internal/util/AndroidConnectionStatusProvider.java

@@ -460,6 +460,7 @@ public void close() {
                    connectivityManagerLock.acquire()) {
             connectivityManager = null;
           }
+          AppState.getInstance().removeAppStateListener(this);
         });
   }
 

sentry-android-core/src/test/java/io/sentry/android/core/SystemEventsBreadcrumbsIntegrationTest.kt

@@ -1,10 +1,13 @@
 package io.sentry.android.core
 
+import android.app.ActivityManager
+import android.app.ActivityManager.RunningAppProcessInfo
 import android.content.Context
 import android.content.Intent
 import android.os.BatteryManager
 import android.os.Build
 import android.os.Looper
+import androidx.test.core.app.ApplicationProvider
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import io.sentry.Breadcrumb
 import io.sentry.IScopes
@@ -33,6 +36,9 @@ import org.mockito.kotlin.verifyNoMoreInteractions
 import org.mockito.kotlin.whenever
 import org.robolectric.Shadows.shadowOf
 import org.robolectric.annotation.Config
+import org.robolectric.shadow.api.Shadow
+import org.robolectric.shadows.ShadowActivityManager
+import org.robolectric.shadows.ShadowBuild
 
 @RunWith(AndroidJUnit4::class)
 @Config(sdk = [Build.VERSION_CODES.TIRAMISU])
@@ -41,6 +47,7 @@ class SystemEventsBreadcrumbsIntegrationTest {
     val context = mock<Context>()
     var options = SentryAndroidOptions()
     val scopes = mock<IScopes>()
+    lateinit var shadowActivityManager: ShadowActivityManager
 
     fun getSut(
       enableSystemEventBreadcrumbs: Boolean = true,
@@ -64,6 +71,9 @@ class SystemEventsBreadcrumbsIntegrationTest {
   fun `set up`() {
     AppState.getInstance().resetInstance()
     AppState.getInstance().registerLifecycleObserver(fixture.options)
+    ShadowBuild.reset()
+    val activityManager = ApplicationProvider.getApplicationContext<Context>().getSystemService(Context.ACTIVITY_SERVICE) as ActivityManager?
+    fixture.shadowActivityManager = Shadow.extract(activityManager)
   }
 
   @AfterTest
@@ -501,4 +511,16 @@ class SystemEventsBreadcrumbsIntegrationTest {
 
     assertNull(sut.receiver)
   }
+
+  @Test
+  fun `when integration is registered in background, receiver is registered`() {
+    val process = RunningAppProcessInfo().apply { this.importance = RunningAppProcessInfo.IMPORTANCE_CACHED }
+    val processes = mutableListOf(process)
+    fixture.shadowActivityManager.setProcesses(processes)
+
+    val sut = fixture.getSut()
+    sut.register(fixture.scopes, fixture.options)
+
+    assertNull(sut.receiver)
+  }
 }