Fix thread leak in ImsManagerTest

Add methods to stop the default executor's HandlerThread during testing

Bug: 156287358
Test: atest FrameworksTelephonyTests
Change-Id: Iea3553c9564a45b85596aacd5012e26b712a832d

1 files changed, 6 insertions, 1 deletions

diff --git a/src/java/com/android/ims/ImsManager.java b/src/java/com/android/ims/ImsManager.java
index 48459496..966814fd 100644
--- a/src/java/com/android/ims/ImsManager.java
+++ b/src/java/com/android/ims/ImsManager.java
@@ -210,7 +210,8 @@ public class ImsManager implements IFeatureConnector {
         void executeRunnable(Runnable runnable);
     }
 
-    private static class ImsExecutorFactory implements ExecutorFactory {
+    @VisibleForTesting
+    public static class ImsExecutorFactory implements ExecutorFactory {
 
         private final HandlerThread mThreadHandler;
         private final Handler mHandler;
@@ -225,6 +226,10 @@ public class ImsManager implements IFeatureConnector {
         public void executeRunnable(Runnable runnable) {
             mHandler.post(runnable);
         }
+
+        public void destroy() {
+            mThreadHandler.quit();
+        }
     }
 
     // Replaced with single-threaded executor for testing.