fixed tests

Author: stefanosiano

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

@@ -396,7 +396,7 @@ private void updateCache() {
       if (!isUpdatingCache.getAndSet(true)) {
         // Fallback: query current active network in the background
         if (options.getThreadChecker().isMainThread()) {
-          submitSafe(() -> updateCacheFromConnectivityManager());
+          submitSafe(() -> updateCacheFromConnectivityManager(), () -> isUpdatingCache.set(false));
         } else {
           updateCacheFromConnectivityManager();
         }
@@ -854,12 +854,20 @@ public static List<NetworkCallback> getChildCallbacks() {
   }
 
   private void submitSafe(@NotNull Runnable r) {
+    submitSafe(r, null);
+  }
+
+  private void submitSafe(final @NotNull Runnable r, final @Nullable Runnable onFinally) {
     try {
       options.getExecutorService().submit(r);
     } catch (Throwable e) {
       options
           .getLogger()
           .log(SentryLevel.ERROR, "AndroidConnectionStatusProvider submit failed", e);
+    } finally {
+      if (onFinally != null) {
+        onFinally.run();
+      }
     }
   }
 }

sentry-android-core/src/test/java/io/sentry/android/core/internal/util/AndroidConnectionStatusProviderTest.kt

@@ -106,6 +106,7 @@ class AndroidConnectionStatusProviderTest {
     options = SentryOptions()
     options.setLogger(logger)
     options.executorService = ImmediateExecutorService()
+    options.threadChecker = AndroidThreadChecker.getInstance()
 
     // Reset current time for each test to ensure cache isolation
     currentTime = 1000L
@@ -128,6 +129,7 @@ class AndroidConnectionStatusProviderTest {
 
   @AfterTest
   fun `tear down`() {
+    options.executorService = ImmediateExecutorService()
     // clear the cache and ensure proper cleanup
     connectionStatusProvider.close()
     contextUtilsStaticMock.close()