From: Tim Kourt <tim.a.kourt@intel.com>
Date: Sat, 23 May 2015 01:36:22 +0000 (-0700)
Subject: Changed behavior of the Android tests
X-Git-Tag: 1.2.0+RC1~1673
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=637aa0a6dbcc242df81d56bce2285b320d757edb;p=platform%2Fupstream%2Fiotivity.git

Changed behavior of the Android tests

Change-Id: I47df4d62e5b41c1b84de1f5d6fa71adacc94d295
Signed-off-by: Tim Kourt <tim.a.kourt@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/1104
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Erich Keane <erich.keane@intel.com>
---

diff --git a/android/android_api/base/src/androidTest/java/org/iotivity/base/SmokeTest.java b/android/android_api/base/src/androidTest/java/org/iotivity/base/SmokeTest.java
index 490496a..956cbf7 100644
--- a/android/android_api/base/src/androidTest/java/org/iotivity/base/SmokeTest.java
+++ b/android/android_api/base/src/androidTest/java/org/iotivity/base/SmokeTest.java
@@ -223,9 +223,15 @@ public class SmokeTest extends InstrumentationTestCase {
                 if (ex instanceof OcException) {
                     OcException ocEx = (OcException) ex;
                     ErrorCode errCode = ocEx.getErrorCode();
+                    if(ErrorCode.NO_RESOURCE != errCode){
+                        Log.e(TAG, ocEx.getMessage());
+                        assertTrue(false);
+                    }
+                }
+                else{
+                    Log.e(TAG, ex.getMessage());
+                    assertTrue(false);
                 }
-                Log.e(TAG, ex.toString());
-                assertTrue(false);
             }
         };
 
@@ -364,9 +370,15 @@ public class SmokeTest extends InstrumentationTestCase {
                 if (ex instanceof OcException) {
                     OcException ocEx = (OcException) ex;
                     ErrorCode errCode = ocEx.getErrorCode();
+                    if(ErrorCode.NO_RESOURCE != errCode){
+                        Log.e(TAG, ocEx.getMessage());
+                        assertTrue(false);
+                    }
+                }
+                else{
+                    Log.e(TAG, ex.getMessage());
+                    assertTrue(false);
                 }
-                Log.e(TAG, ex.toString());
-                assertTrue(false);
             }
         };
 
@@ -400,9 +412,15 @@ public class SmokeTest extends InstrumentationTestCase {
                                             if (ex instanceof OcException) {
                                                 OcException ocEx = (OcException) ex;
                                                 ErrorCode errCode = ocEx.getErrorCode();
+                                                if(ErrorCode.NO_RESOURCE != errCode){
+                                                    Log.e(TAG, ocEx.getMessage());
+                                                    assertTrue(false);
+                                                }
+                                            }
+                                            else{
+                                                Log.e(TAG, ex.getMessage());
+                                                assertTrue(false);
                                             }
-                                            Log.e(TAG, ex.toString());
-                                            assertTrue(false);
                                         }
                                     });
 
@@ -527,9 +545,15 @@ public class SmokeTest extends InstrumentationTestCase {
                 if (ex instanceof OcException) {
                     OcException ocEx = (OcException) ex;
                     ErrorCode errCode = ocEx.getErrorCode();
+                    if(ErrorCode.NO_RESOURCE != errCode){
+                        Log.e(TAG, ocEx.getMessage());
+                        assertTrue(false);
+                    }
+                }
+                else{
+                    Log.e(TAG, ex.getMessage());
+                    assertTrue(false);
                 }
-                Log.e(TAG, ex.toString());
-                assertTrue(false);
             }
         };
 
@@ -563,9 +587,15 @@ public class SmokeTest extends InstrumentationTestCase {
                                             if (ex instanceof OcException) {
                                                 OcException ocEx = (OcException) ex;
                                                 ErrorCode errCode = ocEx.getErrorCode();
+                                                if(ErrorCode.NO_RESOURCE != errCode){
+                                                    Log.e(TAG, ocEx.getMessage());
+                                                    assertTrue(false);
+                                                }
+                                            }
+                                            else{
+                                                Log.e(TAG, ex.getMessage());
+                                                assertTrue(false);
                                             }
-                                            Log.e(TAG, ex.toString());
-                                            assertTrue(false);
                                         }
                                     });
 
@@ -679,9 +709,15 @@ public class SmokeTest extends InstrumentationTestCase {
                 if (ex instanceof OcException) {
                     OcException ocEx = (OcException) ex;
                     ErrorCode errCode = ocEx.getErrorCode();
+                    if(ErrorCode.NO_RESOURCE != errCode){
+                        Log.e(TAG, ocEx.getMessage());
+                        assertTrue(false);
+                    }
+                }
+                else{
+                    Log.e(TAG, ex.getMessage());
+                    assertTrue(false);
                 }
-                Log.e(TAG, ex.toString());
-                assertTrue(false);
             }
         };
 
@@ -948,62 +984,62 @@ public class SmokeTest extends InstrumentationTestCase {
         }
     }
 
-    public void testRegisterDeviceInfoGetDeviceInfo() throws InterruptedException {
-        final String resourceType = "unit.test.resource" + new Date().getTime();
-        final CountDownLatch signal = new CountDownLatch(1);
-
-        OcPlatform.OnDeviceFoundListener deviceFoundListener = new OcPlatform.OnDeviceFoundListener() {
-            @Override
-            public void onDeviceFound(OcRepresentation ocRepresentation) {
-                try {
-                    Log.i(TAG, "Device Name: " + ocRepresentation.getValue("dn"));
-                } catch (OcException e) {
-                    Log.e(TAG, e.toString());
-                    assertTrue(false);
-                }
-                boolean hasDeviceNameAtr = ocRepresentation.hasAttribute("dn");
-                assertTrue(hasDeviceNameAtr);
-                boolean hasNonExistingAtr = ocRepresentation.hasAttribute("NonExisting");
-                assertFalse(hasNonExistingAtr);
-                Log.i(TAG, "URI: " + ocRepresentation.getUri());
-                signal.countDown();
-            }
-        };
-
-        OcDeviceInfo devInfo = new OcDeviceInfo();
-
-        devInfo.setContentType("myContentType");
-        devInfo.setDateOfManufacture("myDateOfManufacture");
-        devInfo.setDeviceName("myDeviceName");
-        devInfo.setDeviceUuid("myDeviceUUID");
-        devInfo.setFirmwareVersion("myFirmwareVersion");
-        devInfo.setHostName("myHostName");
-        devInfo.setManufacturerName("myManufacturerNa");
-        devInfo.setManufacturerUrl("myManufacturerUrl");
-        devInfo.setModelNumber("myModelNumber");
-        devInfo.setPlatformVersion("myPlatformVersion");
-        devInfo.setSupportUrl("mySupportUrl");
-        devInfo.setVersion("myVersion");
-
-        try {
-            //server
-            OcPlatform.registerDeviceInfo(devInfo);
-
-            //client
-            OcPlatform.getDeviceInfo(
-                    "",
-                    OcPlatform.WELL_KNOWN_QUERY + "/d",
-                    OcConnectivityType.IPV4,
-                    deviceFoundListener);
-
-            //wait for onDeviceFound event
-            assertTrue(signal.await(60, TimeUnit.SECONDS));
-
-        } catch (OcException e) {
-            Log.e(TAG, e.getMessage());
-            assertTrue(false);
-        }
-    }
+//    public void testRegisterDeviceInfoGetDeviceInfo() throws InterruptedException {
+//        final String resourceType = "unit.test.resource" + new Date().getTime();
+//        final CountDownLatch signal = new CountDownLatch(1);
+//
+//        OcPlatform.OnDeviceFoundListener deviceFoundListener = new OcPlatform.OnDeviceFoundListener() {
+//            @Override
+//            public void onDeviceFound(OcRepresentation ocRepresentation) {
+//                try {
+//                    Log.i(TAG, "Device Name: " + ocRepresentation.getValue("dn"));
+//                } catch (OcException e) {
+//                    Log.e(TAG, e.toString());
+//                    assertTrue(false);
+//                }
+//                boolean hasDeviceNameAtr = ocRepresentation.hasAttribute("dn");
+//                assertTrue(hasDeviceNameAtr);
+//                boolean hasNonExistingAtr = ocRepresentation.hasAttribute("NonExisting");
+//                assertFalse(hasNonExistingAtr);
+//                Log.i(TAG, "URI: " + ocRepresentation.getUri());
+//                signal.countDown();
+//            }
+//        };
+//
+//        OcDeviceInfo devInfo = new OcDeviceInfo();
+//
+//        devInfo.setContentType("myContentType");
+//        devInfo.setDateOfManufacture("myDateOfManufacture");
+//        devInfo.setDeviceName("myDeviceName");
+//        devInfo.setDeviceUuid("myDeviceUUID");
+//        devInfo.setFirmwareVersion("myFirmwareVersion");
+//        devInfo.setHostName("myHostName");
+//        devInfo.setManufacturerName("myManufacturerNa");
+//        devInfo.setManufacturerUrl("myManufacturerUrl");
+//        devInfo.setModelNumber("myModelNumber");
+//        devInfo.setPlatformVersion("myPlatformVersion");
+//        devInfo.setSupportUrl("mySupportUrl");
+//        devInfo.setVersion("myVersion");
+//
+//        try {
+//            //server
+//            OcPlatform.registerDeviceInfo(devInfo);
+//
+//            //client
+//            OcPlatform.getDeviceInfo(
+//                    "",
+//                    OcPlatform.MULTICAST_PREFIX + "/oc/d",
+//                    OcConnectivityType.IPV4,
+//                    deviceFoundListener);
+//
+//            //wait for onDeviceFound event
+//            assertTrue(signal.await(60, TimeUnit.SECONDS));
+//
+//        } catch (OcException e) {
+//            Log.e(TAG, e.getMessage());
+//            assertTrue(false);
+//        }
+//    }
 
     public void testBindUnbindResources() throws InterruptedException {
         final String resourceType = "unit.test.resource" + new Date().getTime();