commit 6c85560b30a69472e1bad02fced11a4b53aa7a77
author Jack Yu <jackyu@google.com> Mon May 22 10:05:56 2017 -0700
committer Jack Yu <jackyu@google.com> Mon May 22 16:31:56 2017 -0700
tree 4aa9cf141dec9d5a102bb79748d4d404ba0ad7c1
parent 1b9718ab5e1764de2afe9286fbe03fb5a5e875b0

Fixed a resource leak in unit tests

Test: Telephony unit tests
bug: 29579544
Merged-In: Ib0f10385b92d3effa76095f81c7bedc703df208e
Change-Id: Ib0f10385b92d3effa76095f81c7bedc703df208e

tests/telephonytests/src/com/android/internal/telephony/CarrierActionAgentTest.java

diff --git a/tests/telephonytests/src/com/android/internal/telephony/CarrierActionAgentTest.java b/tests/telephonytests/src/com/android/internal/telephony/CarrierActionAgentTest.java
index cc72463..711c84e 100644
--- a/tests/telephonytests/src/com/android/internal/telephony/CarrierActionAgentTest.java
+++ b/tests/telephonytests/src/com/android/internal/telephony/CarrierActionAgentTest.java
@@ -45,6 +45,7 @@
     private FakeContentProvider mFakeContentProvider;
     private static int DATA_CARRIER_ACTION_EVENT = 0;
     private static int RADIO_CARRIER_ACTION_EVENT = 1;
+    private CarrierActionAgentHandler mCarrierActionAgentHandler;
     @Mock
     private Handler mDataActionHandler;
     @Mock
@@ -107,7 +108,8 @@
         mFakeContentProvider = new FakeContentProvider();
         mFakeContentResolver.addProvider(Settings.AUTHORITY, mFakeContentProvider);
         doReturn(mFakeContentResolver).when(mContext).getContentResolver();
-        new CarrierActionAgentHandler(getClass().getSimpleName()).start();
+        mCarrierActionAgentHandler = new CarrierActionAgentHandler(getClass().getSimpleName());
+        mCarrierActionAgentHandler.start();
         waitUntilReady();
         logd("CarrierActionAgentTest -Setup!");
     }
@@ -131,6 +133,7 @@
     @After
     public void tearDown() throws Exception {
         Settings.Global.putInt(mFakeContentResolver, Settings.Global.AIRPLANE_MODE_ON, 0);
+        mCarrierActionAgentHandler.quit();
         super.tearDown();
     }
 }