[IOT-2745] Static Analysis fix. 21/22821/2
authorJay Sharma <jay.sharma@samsung.com>
Fri, 13 Oct 2017 12:01:28 +0000 (17:31 +0530)
committerAshok Babu Channa <ashok.channa@samsung.com>
Mon, 16 Oct 2017 18:28:39 +0000 (18:28 +0000)
Java static analysis fix for resource-encapsulation:.
1. Updated exception handling for logging.
2. Updated sample applications and unit test cases.

Bug: https://jira.iotivity.org/browse/IOT-2745
Change-Id: I1aeb1b8b2b3338515cbc9733ef8fa032fd6f068e
Signed-off-by: Jay Sharma <jay.sharma@samsung.com>
service/resource-encapsulation/android/service/src/androidTest/java/org/iotivity/service/RemoteResourceObjectTest.java
service/resource-encapsulation/android/service/src/androidTest/java/org/iotivity/service/ResourceObjectTest.java
service/resource-encapsulation/android/service/src/androidTest/java/org/iotivity/service/TestBase.java
service/resource-encapsulation/android/service/src/main/java/org/iotivity/service/RcsValue.java
service/resource-encapsulation/examples/android/RESampleClientApp/app/src/main/java/org/iotivity/service/sample/client/MainActivity.java
service/resource-encapsulation/examples/android/RESampleServerApp/app/src/main/java/org/iotivity/service/sample/server/MainActivity.java

index d208bdc..b11de38 100644 (file)
@@ -1,5 +1,7 @@
 package org.iotivity.service;
 
+import android.util.Log;
+
 import static org.iotivity.service.client.RcsRemoteResourceObject.CacheState;
 import static org.iotivity.service.client.RcsRemoteResourceObject.OnCacheUpdatedListener;
 import static org.iotivity.service.client.RcsRemoteResourceObject.OnRemoteAttributesReceivedListener;
@@ -15,6 +17,8 @@ import org.mockito.Mockito;
 
 public class RemoteResourceObjectTest extends TestBase {
 
+    private final String TAG = RemoteResourceObjectTest.class.getName();
+
     private RcsResourceAttributes createAttrs() {
         RcsResourceAttributes attrs = new RcsResourceAttributes();
         attrs.put("key", new RcsValue(3));
@@ -78,6 +82,7 @@ public class RemoteResourceObjectTest extends TestBase {
             mClient.startMonitoring(listener);
             fail("No exception thrown");
         } catch (RcsIllegalStateException e) {
+            Log.e(TAG, "caught exception: " + e.toString());
         }
     }
 
@@ -104,6 +109,7 @@ public class RemoteResourceObjectTest extends TestBase {
             mClient.startCaching();
             fail("No exception thrown");
         } catch (RcsIllegalStateException e) {
+            Log.e(TAG, "caught exception: " + e.toString());
         }
     }
 
@@ -146,6 +152,7 @@ public class RemoteResourceObjectTest extends TestBase {
             mClient.getCachedAttributes();
             fail("No exception thrown");
         } catch (RcsIllegalStateException e) {
+            Log.e(TAG, "caught exception: " + e.toString());
         }
     }
 
@@ -169,6 +176,7 @@ public class RemoteResourceObjectTest extends TestBase {
             mClient.getCachedAttribute("some_key");
             fail("No exception thrown");
         } catch (RcsIllegalStateException e) {
+            Log.e(TAG, "caught exception: " + e.toString());
         }
     }
 
@@ -203,6 +211,7 @@ public class RemoteResourceObjectTest extends TestBase {
             mClient.getUri();
             fail("No exception thrown");
         } catch (RcsDestroyedObjectException e) {
+            Log.e(TAG, "caught exception: " + e.toString());
         }
     }
 }
index bb6a6d7..7eb5a5b 100644 (file)
@@ -1,5 +1,7 @@
 package org.iotivity.service;
 
+import android.util.Log;
+
 import static org.mockito.Matchers.any;
 import static org.mockito.Matchers.anyInt;
 import static org.mockito.Matchers.eq;
@@ -21,6 +23,7 @@ import org.mockito.Mockito;
 
 public class ResourceObjectTest extends TestBase {
     private static final String NEW_KEY = "new" + KEY;
+    private final String TAG = ResourceObjectTest.class.getName();
 
     private OnRemoteAttributesReceivedListener mOnRemoteAttributesReceivedListener;
     private SetRequestHandler                  mSetRequestHandler;
@@ -211,6 +214,7 @@ public class ResourceObjectTest extends TestBase {
             mServer.removeAttribute("");
             fail("No exception thrown");
         } catch (RcsDestroyedObjectException e) {
+            Log.e(TAG, "caught exception: " + e.toString());
         }
 
     }
index 1b78bd7..67448b0 100755 (executable)
@@ -7,6 +7,7 @@ import org.iotivity.service.client.RcsRemoteResourceObject;
 import org.iotivity.service.server.RcsResourceObject;
 
 import android.test.InstrumentationTestCase;
+import android.util.Log;
 
 public abstract class TestBase extends InstrumentationTestCase {
     protected static final String RESOURCEURI       = "/a/TemperatureSensor";
@@ -22,6 +23,8 @@ public abstract class TestBase extends InstrumentationTestCase {
     protected RcsResourceObject       mServer;
     protected RcsRemoteResourceObject mClient;
 
+    private final String TAG = TestBase.class.getName();
+
     private OnResourceDiscoveredListener mOnResourceDiscoveredListener = new OnResourceDiscoveredListener() {
         @Override
         public void onResourceDiscovered(
@@ -63,7 +66,7 @@ public abstract class TestBase extends InstrumentationTestCase {
 
                 discoveryTask.cancel();
             } catch (InterruptedException e) {
-                e.printStackTrace();
+                Log.e(TAG, "caught exception: " + e.toString());
             }
         }
     }
index 1106d9e..8d795b7 100644 (file)
@@ -593,6 +593,7 @@ public final class RcsValue {
         try {
             return (T) mObject;
         } catch (final ClassCastException e) {
+            e.printStackTrace();
             return null;
         }
     }
@@ -630,6 +631,7 @@ public final class RcsValue {
         try {
             return get();
         } catch (final ClassCastException e) {
+            e.printStackTrace();
             return defaultValue;
         }
     }
@@ -657,6 +659,7 @@ public final class RcsValue {
         try {
             return get();
         } catch (final ClassCastException e) {
+            e.printStackTrace();
             return defaultValue;
         }
     }
@@ -684,6 +687,7 @@ public final class RcsValue {
         try {
             return get();
         } catch (final ClassCastException e) {
+            e.printStackTrace();
             return defaultValue;
         }
     }
@@ -711,6 +715,7 @@ public final class RcsValue {
         try {
             return get();
         } catch (final ClassCastException e) {
+            e.printStackTrace();
             return defaultValue;
         }
     }
index 21b95af..b9eefeb 100644 (file)
@@ -84,8 +84,9 @@ public class MainActivity extends Activity {
     }
 
     private void configurePlatform() {
+        String ipAddress = "0.0.0.0";
         OcPlatform.Configure(new PlatformConfig(getApplicationContext(),
-                ServiceType.IN_PROC, ModeType.CLIENT_SERVER, "0.0.0.0", 0,
+                ServiceType.IN_PROC, ModeType.CLIENT_SERVER, ipAddress, 0,
                 QualityOfService.LOW));
 
         Log.i(LOG_TAG, "Configuration done Successfully");
index b83b8ef..fe7170f 100644 (file)
@@ -84,8 +84,9 @@ public class MainActivity extends Activity {
     }
 
     private void configurePlatform() {
+        String ipAddress = "0.0.0.0";
         OcPlatform.Configure(new PlatformConfig(getApplicationContext(),
-                ServiceType.IN_PROC, ModeType.CLIENT_SERVER, "0.0.0.0", 0,
+                ServiceType.IN_PROC, ModeType.CLIENT_SERVER, ipAddress, 0,
                 QualityOfService.LOW));
         Log.i(LOG_TAG, "Configuration done successfully");
     }