#466 Appropriately fail test case when catching expected exception.

1 files changed, 13 insertions, 0 deletions

diff --git a/sdl_android/src/androidTest/java/com/smartdevicelink/test/proxy/SdlProxyBaseTests.java b/sdl_android/src/androidTest/java/com/smartdevicelink/test/proxy/SdlProxyBaseTests.java
index f4f6bd519..282c01357 100644
--- a/sdl_android/src/androidTest/java/com/smartdevicelink/test/proxy/SdlProxyBaseTests.java
+++ b/sdl_android/src/androidTest/java/com/smartdevicelink/test/proxy/SdlProxyBaseTests.java
@@ -76,6 +76,8 @@ import com.smartdevicelink.proxy.rpc.enums.Language;
 import com.smartdevicelink.proxy.rpc.enums.SdlDisconnectedReason;
 import com.smartdevicelink.transport.BaseTransportConfig;
 
+import junit.framework.Assert;
+
 import java.util.Vector;
 
 public class SdlProxyBaseTests extends AndroidTestCase{
@@ -113,6 +115,17 @@ public class SdlProxyBaseTests extends AndroidTestCase{
             proxy = builder.build();
         } catch (Exception e) {
             Log.v(TAG, "Exception in testNullSdlProxyConfigurationResources, testing null SdlProxyConfigurationResources");
+            Assert.fail("Exception in testNullSdlProxyConfigurationResources, testing null SdlProxyConfigurationResources");
+        }
+
+        //Construct with a non-null SdlProxyConfigurationResources and a null TelephonyManager
+        config.setTelephonyManager(null);
+        builder.setSdlProxyConfigurationResources(config);
+        try {
+            proxy = builder.build();
+        } catch (Exception e) {
+            Log.v(TAG, "Exception in testNullSdlProxyConfigurationResources, testing null TelephonyManager");
+            Assert.fail("Exception in testNullSdlProxyConfigurationResources, testing null TelephonyManager");
         }
     }