[Messaging-sms] clean up - part 2
authorAndrzej Krolikowski <a.krolikowsk@samsung.com>
Fri, 20 Sep 2013 12:33:42 +0000 (14:33 +0200)
committerGerrit Code Review <gerrit@gerrit.vlan144.tizendev.org>
Tue, 24 Sep 2013 14:46:08 +0000 (14:46 +0000)
Change-Id: I55f7052cdb497aa4bd832260483e3d46181e9ed1

22 files changed:
tct-messaging-sms-tizen-tests/messaging/ConversationArraySuccessCallback_onsuccess_checking.html
tct-messaging-sms-tizen-tests/messaging/MessageConversationArraySuccessCallback_onsuccess_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageService_id_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageService_loadBodySMS.html
tct-messaging-sms-tizen-tests/messaging/MessageService_name_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageService_sendMessage_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageService_sms_sendMessage.html
tct-messaging-sms-tizen-tests/messaging/MessageService_type_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_addCCLAddMRemoveCRemoveCCL.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_addConversationsChangeListener_withoutfilter.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_addMCLaddM.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_addMessagesChangeListener_withfilter.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_addMessagesChangeListener_withoutfilter.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_findMessages.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_findMessages_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeChangeListener_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeChangeListener_forconversations.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeChangeListener_forfolder.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeChangeListener_formessages.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeChangeListener_valid_argument.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeConversations_sms.html
tct-messaging-sms-tizen-tests/messaging/MessagesChangeCallback_messagesupdated_withoutfilter.html

index 9e7183b06093b70f2bb067e74542769222e8d328..c520622e62ca741c76ab98176b90842299475632 100644 (file)
@@ -43,14 +43,12 @@ Authors:
 <body>
 <div id="log"></div>
 <script type="text/javascript">
-
 //==== TEST: ConversationArraySuccessCallback_onsuccess_checking
 //==== LABEL check the function of findConversations with sms filter
 //==== SPEC Tizen Web API:Communication:Messaging:MessageConversationArraySuccessCallback:onsuccess M
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== ONLOAD_DELAY 90
 //==== TEST_CRITERIA CBOA
-
 setup({timeout: 90000});
 
 var t = async_test("ConversationArraySuccessCallback_onsuccess_checking", {timeout: 90000}),
index 31db7e8de75222e4ae93d0dee94087640f6474e6..967bb77f72b18bdda8df203a97a8c81e710e3d14 100644 (file)
@@ -49,7 +49,8 @@ Authors:
 //==== TEST_CRITERIA CBOA
 setup({timeout: 90000});
 var t = async_test("MessageConversationArraySuccessCallback_onsuccess_exist", {timeout:90000}), serviceSuccess, filter,
-    findConversationsSuccess, findConversationsError, msg, datetime, service, onReceiveMsg, messageStorage;
+    findConversationsSuccess, findConversationsError, msg, datetime, service, onReceiveMsg, messageStorage,
+    serviceError;
 t.step(function () {
     //cleanup after TC
     add_result_callback(function () {
@@ -101,7 +102,11 @@ t.step(function () {
         sendAndReceiveMessage(t, service, msg, datetime, onReceiveMsg);
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
 });
 
 </script>
index 96e54e8604c4184a88e976f719bf21c5f59236cb..2c800fe77accf2f7d1990ac819fa0575ec10ae3a 100644 (file)
@@ -54,6 +54,7 @@ var t = async_test("MessageService_id_exist", {timeout: 90000}), serviceSuccess,
 t.step(function () {
 
     serviceSuccess = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
         assert_true("id" in services[0], "Messaging.MessageService.id property exists");
         t.done();
     });
index 73472e25e811ab665639ce6abf928658c424f11b..cf4d4eb1ea2459333ac861e294e15d70d433632a 100644 (file)
@@ -39,6 +39,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -48,9 +49,9 @@ Authors:
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== ONLOAD_DELAY 90
 //==== TEST_CRITERIA MOA MR
-setup({ timeout: 90000 });
+setup({timeout: 90000});
 
-var t = async_test("MessageService_loadBodySMS", { timeout: 90000 }), messageStorage,
+var t = async_test("MessageService_loadBodySMS", {timeout: 90000}), messageStorage,
     msg, service, loadBodySuccess, loadBodyError, onReceiveMsg, serviceSuccess, serviceError, datetime,
     returnedValue = null;
 
index cb54664dba837dacc582ef0d071df3585b2a63a7..086c740dc8ab837d4563320f5b2b65bba1b78a30 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -54,11 +55,11 @@ var t = async_test("MessageService_name_exist", {timeout: 90000}), serviceErrorC
 t.step(function () {
 
     serviceErrorCB = t.step_func(function (error) {
-        assert_unreached("Unexpected serviceErrorCB: " + error.name + ":" + error.message);
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
     serviceListCB = t.step_func(function (services) {
-        assert_true(services.length > 0, "No messaging.sms service.");
+        assert_greater_than(services.length, 0, "Received empty services array");
         assert_true("name" in services[0], "Messaging.MessageService.name property exists");
         t.done();
     });
index 2b7ccefa3a632fea2621800bfa9768a7c45d6409..a7717225b41dd201c14a96bda8dc212233b4c02d 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -53,10 +54,10 @@ setup({timeout: 90000});
 var t = async_test("MessageService_sendMessage_exist", {timeout: 90000}), serviceErrorCB, serviceListCB;
 t.step(function () {
     serviceErrorCB = t.step_func(function (error) {
-        assert_unreached("Cannot get messaging service: " + error.message);
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
     serviceListCB = t.step_func(function (services) {
-        assert_true(services.length > 0, "No services.");
+        assert_greater_than(services.length, 0, "Received empty services array");
         assert_true("sendMessage" in services[0], "sendMessage does not exist.");
         t.done();
     });
index 413d2c992d794e4e42496eef30d1fc385dbb7115..22ac78219c77b75f549512ddd2e9320311838e9f 100644 (file)
@@ -39,6 +39,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -48,9 +49,9 @@ Authors:
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== ONLOAD_DELAY 90
 //==== TEST_CRITERIA MOA
-setup({ timeout: 90000 });
+setup({timeout: 90000});
 
-var t = async_test("MessageService_sms_sendMessage", { timeout: 90000 }),
+var t = async_test("MessageService_sms_sendMessage", {timeout: 90000}),
     message, datetime, messageStorage,
     service, serviceSuccess, serviceError,
     onReceiveMsg;
@@ -76,6 +77,7 @@ t.step(function () {
     });
 
     serviceSuccess = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
         messageStorage = service.messageStorage;
         datetime = tizen.time.getCurrentDateTime().toString();
@@ -88,11 +90,10 @@ t.step(function () {
     });
 
     serviceError = t.step_func(function (error) {
-        assert_unreached("Cannot find services: " + error.message);
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
     tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
-
 });
 
 </script>
index 9d352d286e37b9107243c25d8fa6162d649ac840..849418a21a9e8dfe6a78264e46c9e0cdb0a2c1f4 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -47,44 +48,23 @@ Authors:
 //==== SPEC Tizen Web API:Communication:Messaging:MessageService:type A
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== TEST_CRITERIA AE
-
-//  check if the attribute type in MessageService
 setup({timeout: 90000});
 
-var t = async_test("MessageService_type_exist", {timeout: 90000});
-// Define service error callback.
-function serviceErrorCB(error) {
-    t.step(function () {
-        assert_true(false, "Cannot get messaging service: " + error.message);
+var t = async_test("MessageService_type_exist", {timeout: 90000}),
+    serviceSuccess, serviceError;
+t.step(function () {
+    serviceSuccess = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
+        assert_true("type" in services[0], "type in MessageService is not present");
+        t.done();
     });
-    t.done();
-}
-// Define the success callback.
-function serviceListCB(services) {
-    if (services.length > 0) {
-        // SMS sending example
-        try {
-            t.step(function () {
-                assert_true("type" in services[0], "type in MessageService");
-            });
-            t.done();
-        } catch (e) {
-            t.step(function () {
-                assert_true(false, "Exception message: " + e.message);
-            });
-            t.done();
-        }
-    }
-}
 
-try {
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceErrorCB);
-} catch (e) {
-    t.step(function () {
-        assert_true(false, "Exception message: " + e.message);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
-    t.done();
-}
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
+});
 
 </script>
 </body>
index b6c0228c58adb0ddb19e8281162ccb06ce34d5eb..4dcbf6a9d5d34e45ecbf3d7321406b7383205437 100644 (file)
@@ -91,7 +91,7 @@ t.step(function () {
     });
 
     successRemoveConCallback = t.step_func(function () {
-        return t.done();
+        t.done();
     });
 
     conversationsFindConArrayCB = t.step_func(function (conversations) {
index 8fb3857d983be1150e6e6e4b36572bd4bbd3d39c..85465bf5602c533b667f446e591d97cd5ced5c9a 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -50,16 +51,17 @@ Authors:
 //==== TEST_CRITERIA MAST MMINA
 setup({timeout: 90000});
 
-var t = async_test("MessageStorage_addConversationsChangeListener_withoutfilter", {timeout: 90000}), service, msg, conversationChangeCB, serviceListCB;
+var t = async_test("MessageStorage_addConversationsChangeListener_withoutfilter", {timeout: 90000}),
+    service, msg, conversationChangeCB, serviceListCB, watchId, messageStorage;
 
 t.step(function () {
     conversationChangeCB = {
         conversationsadded: t.step_func(function (conversations) {
-            assert_true(conversations.length > 0, "add conversations");
+            assert_greater_than(conversations.length, 0, "Received empty add conversations array");
             t.done();
         }),
         conversationsupdated: t.step_func(function (conversations) {
-            assert_true(conversations.length > 0, "update conversations");
+            assert_greater_than(conversations.length, 0, "Received empty update conversations array");
             t.done();
         }),
         conversationsremoved: t.step_func(function (conversations) {
@@ -68,7 +70,9 @@ t.step(function () {
     };
 
     serviceListCB = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
+        messageStorage = service.messageStorage;
         watchId = service.messageStorage.addConversationsChangeListener(conversationChangeCB);
         msg = new tizen.Message("messaging.sms", {
             plainBody: "test.",
index cfc0ed0b6ba18ecaec1898b3b7d013195450ee89..e794e57cc387952586398ea7014e4e88ff12affc 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -51,7 +52,8 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_addMCLaddM", {timeout: 90000}),
-    service, listenerId, messageChangeCallback, sendSuccess, sendFailure, serviceListCB, msg;
+    service, listenerId, messageChangeCallback, sendSuccess, sendFailure, serviceListCB,
+    msg, serviceError, messageStorage;
 
 t.step(function () {
     messageChangeCallback = {
@@ -69,13 +71,14 @@ t.step(function () {
 
     sendSuccess = t.step_func(function () {});
 
-    sendFailure = t.step_func(function (err) {
-        assert_unreached("sendMessage() error: " + err.message);
+    sendFailure = t.step_func(function (error) {
+        assert_unreached("sendMessage() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
     serviceListCB = t.step_func(function (services) {
-        assert_true(services.length > 0, "No MessageService found.");
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
+        messageStorage = service.messageStorage;
         listenerId = service.messageStorage.addMessagesChangeListener(messageChangeCallback);
         msg = new tizen.Message("messaging.sms", {
             plainBody: "test.",
@@ -84,7 +87,11 @@ t.step(function () {
         service.sendMessage(msg, sendSuccess, sendFailure);
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 //cleanup after TC
index 94f1f6bc43ad26644e9012bf3536a1564ae8e492..f24b7e44e97abd56dd43725dcb3d1b228d4b7f34 100644 (file)
@@ -39,6 +39,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -52,7 +53,8 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_addMessagesChangeListener_withfilter", {timeout: 90000}),
-service, listenerId, messageStorage, message, messageChangeCallback, filter, serviceListCB;
+    service, listenerId, messageStorage, message, messageChangeCallback, filter, serviceListCB,
+    serviceError;
 
 t.step(function (){
     messageChangeCallback = {
@@ -70,7 +72,7 @@ t.step(function (){
     };
 
     serviceListCB = t.step_func(function serviceListCB(services) {
-        assert_true(services.length > 0, "There is no services");
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
         messageStorage = service.messageStorage;
         filter = new tizen.AttributeFilter("to", "CONTAINS", TEST_SMS_RECIPIENT);
@@ -79,7 +81,11 @@ t.step(function (){
         service.messageStorage.addDraftMessage(message);
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 //cleanup after TC
index aae78cbd030461c3f43a40ff1d1c08b83b5c4eee..0114d5a365e205059cd379f9c3cca9ded5eb9057 100644 (file)
@@ -52,7 +52,8 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_addMessagesChangeListener_withoutfilter", {timeout: 90000}),
-    service, listenerId, serviceListCB, message, messageStorage, messageChangeCallback;
+    service, listenerId, serviceListCB, message, messageStorage, messageChangeCallback,
+    serviceError;
 t.step(function () {
     messageChangeCallback = {
         messagesupdated : t.step_func(function (messages) {
@@ -73,15 +74,20 @@ t.step(function () {
     };
 
     serviceListCB = t.step_func(function (services) {
-        assert_true(services.length > 0, "There is no services");
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
         messageStorage = service.messageStorage;
         listenerId = messageStorage.addMessagesChangeListener(messageChangeCallback);
-        assert_type(listenerId, "number" , "Incorrect returned value from addMessagesChangeListener method.");
+        assert_type(listenerId, "long" , "Incorrect returned value from addMessagesChangeListener method.");
         message = new tizen.Message("messaging.sms", {plainBody : "test.",to : [ TEST_SMS_RECIPIENT ]});
         service.sendMessage(message);
     });
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 </script>
index cb25c476717222457c327566b44a985266f3fef6..d1c893669358decd452d927d43dacb9510ac12e7 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -50,34 +51,36 @@ Authors:
 //==== TEST_CRITERIA MOA
 setup({timeout: 90000});
 
-var this_test = async_test("MessageStorage_findMessages", {timeout: 90000});
-this_test.step(function () {
-    var self = this, smsService, findMessagesCallback, onError, typeFilter, getMessagingServicesCallback,
-    addSuccess;
+var t = async_test("MessageStorage_findMessages", {timeout: 90000}),
+    smsService, findMessagesCallback, findError,
+    getMessagingServicesCallback, addSuccess, serviceError;
+t.step(function () {
 
-    findMessagesCallback = self.step_func(function (messages) {
-        assert_true(messages.length > 0, "message fetched");
-        return self.done();
+    findMessagesCallback = t.step_func(function (messages) {
+        assert_greater_than(messages.length, 0, "Received empty messages array");
+        t.done();
     });
 
-    onError = self.step_func(function (err) {
-        assert_unreached("Unexpected errorCallback arrived: " + err.message);
+    findError = t.step_func(function (error) {
+        assert_unreached("findMessages() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
-    typeFilter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
-
-    addSuccess = this_test.step_func(function () {
-        smsService.messageStorage.findMessages(typeFilter, findMessagesCallback, onError);
+    addSuccess = t.step_func(function () {
+        smsService.messageStorage.findMessages(typeFilter, findMessagesCallback, findError);
     });
 
-    getMessagingServicesCallback = self.step_func(function (services) {
-        assert_true(services.length > 0, "Received empty services array");
-        smsService = services[0];   //save service.
+    getMessagingServicesCallback = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
+        smsService = services[0];
+
+        addSMSDraft(t, smsService, smsMessageInitDict, addSuccess);
+    });
 
-        addSMSDraft(this_test, smsService, smsMessageInitDict, addSuccess);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", getMessagingServicesCallback, onError);
+    tizen.messaging.getMessageServices("messaging.sms", getMessagingServicesCallback, serviceError);
 });
 
 </script>
index 169b5b3eadd658544d027fcaabafc5cf2aa25bea..1effbc0383f9783320c71558e0fb9adb0ba92e97 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -47,33 +48,24 @@ Authors:
 //==== SPEC Tizen Web API:Communication:Messaging:MessageStorage:findMessages M
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== TEST_CRITERIA ME
-
-//  check the method of findMessages in MessageStore
 setup({timeout: 90000});
 
-var t = async_test("MessageStorage_findMessages_exist", {timeout: 90000});
-function serviceListCB(services){
-    try {
-        t.step(function () {
-            assert_true("findMessages" in services[0].messageStorage, "Messaging.MessageStorage.findMessages method exists");
-        });
-        t.done();
-    } catch(e) {
-        t.step(function (){
-            assert_true(false, "there is invalid value");
-        });
+var t = async_test("MessageStorage_findMessages_exist", {timeout: 90000}),
+    serviceSuccess, serviceError;
+t.step(function () {
+
+    serviceSuccess = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
+        assert_true("findMessages" in services[0].messageStorage, "Messaging.MessageStorage.findMessages method does not exist");
         t.done();
-    }
-}
+    });
 
-try {
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
-} catch(e) {
-    t.step(function (){
-        assert_true(false, "Exception message: " + e.message);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
-    t.done();
-}
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
+});
 
 </script>
 </body>
index 21d319ec7199c53206924e7129af64f83a1da8d6..d00d856af37df289bb5cba4b545b39deb72b446f 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -49,16 +50,21 @@ Authors:
 //==== TEST_CRITERIA ME
 setup({timeout: 90000});
 
-var t = async_test("MessageStorage_removeChangeListener_exist", {timeout: 90000});
-function serviceListCB(services){
-    t.step(function () {
-        assert_true("removeChangeListener" in services[0].messageStorage, "Messaging.MessageStorage.removeChangeListener method exists");
-    });
-    t.done();
-}
+var t = async_test("MessageStorage_removeChangeListener_exist", {timeout: 90000}),
+    serviceSuccess, serviceError;
 
 t.step(function () {
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    serviceSuccess = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
+        assert_true("removeChangeListener" in services[0].messageStorage, "Messaging.MessageStorage.removeChangeListener method does not exist");
+        t.done();
+    });
+
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
 });
 
 </script>
index baa117db88fb04b938c8e0e5df1dceb6038ba886..1ae2fbb50d217e9db37bec62b44db9bb7b1baa73 100644 (file)
@@ -38,6 +38,7 @@ Authors:
 <script type="text/javascript" src="../resources/unitcommon.js"></script>
 <script type="text/javascript" src="support/messaging_common.js"></script>
 </head>
+
 <body>
 <div id="log"></div>
 <script type="text/javascript">
@@ -53,7 +54,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_removeChangeListener_forconversations", {timeout: 90000}),
-      watchId, conversationChangeCB, serviceListCB;
+      watchId, conversationChangeCB, serviceListCB, serviceError;
 
 t.step(function () {
     conversationChangeCB = {
@@ -66,13 +67,17 @@ t.step(function () {
     };
 
     serviceListCB = t.step_func(function (services) {
-        assert_true(services.length > 0, "No message services.");
+        assert_greater_than(services.length, 0, "Received empty services array");
         watchId = services[0].messageStorage.addConversationsChangeListener(conversationChangeCB);
         services[0].messageStorage.removeChangeListener(watchId);
         t.done();
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 </script>
index 6a046656c132ac8343df1182961cdeba566f61c6..2eb25fda817d361361490bc8a0e9ebf021fca637 100644 (file)
@@ -49,12 +49,10 @@ Authors:
 //==== SPEC Tizen Web API:Communication:Messaging:MessageStorage:removeChangeListener M
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== TEST_CRITERIA MMINA
-
-// Check removeChangeListener method with removing folders listener
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_removeChangeListener_forfolder", {timeout: 90000}),
-      watchId, folderChangeCB, serviceListCB;
+      watchId, folderChangeCB, serviceListCB, serviceError;
 
 t.step(function () {
     folderChangeCB = {
@@ -67,13 +65,17 @@ t.step(function () {
     };
 
     serviceListCB = t.step_func(function (services) {
-        assert_true(services.length > 0, "No message services.");
+        assert_greater_than(services.length, 0, "Received empty services array");
         watchId = services[0].messageStorage.addFoldersChangeListener(folderChangeCB);
         services[0].messageStorage.removeChangeListener(watchId);
         t.done();
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 </script>
index 5e2cb2b6c86b2e2adfcdad3f76281a86ccbc1f84..34bba6ffe16aabad1caec675ca1b559971e0b78f 100644 (file)
@@ -48,12 +48,10 @@ Authors:
 //==== SPEC Tizen Web API:Communication:Messaging:MessageStorage:removeChangeListener M
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== TEST_CRITERIA MMINA
-
-// Check removeChangeListener method with removing messages listener
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_removeChangeListener_formessages", {timeout: 90000}),
-      watchId, messageChangeCB, serviceListCB;
+      watchId, messageChangeCB, serviceListCB, serviceError;
 
 t.step(function () {
     messageChangeCB = {
@@ -72,7 +70,11 @@ t.step(function () {
         t.done();
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 </script>
index e9c3d3166e72f81e7ecc779168dafad32f60746a..a8535a6aca52e7269e472c7c30d07f2f168eb153 100644 (file)
@@ -47,15 +47,13 @@ Authors:
 //==== SPEC Tizen Web API:Communication:Messaging:MessageStorage:removeChangeListener M
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== TEST_CRITERIA MMINA
-
-//  check the function of removeChangeListener in MessageStore
 setup({timeout: 90000});
 
-var t =async_test("MessageStorage_removeChangeListener_valid_argument", {timeout: 90000});
-var watchId, service, serviceListCB;
+var t = async_test("MessageStorage_removeChangeListener_valid_argument", {timeout: 90000}),
+    watchId, service, serviceListCB, serviceError, folderChangeCB;
 
 t.step(function () {
-    var folderChangeCB = {
+    folderChangeCB = {
         foldersadded: function (folders) {
         },
         foldersupdated: function (folders) {
@@ -65,13 +63,18 @@ t.step(function () {
     };
 
     serviceListCB = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
         watchId = service.messageStorage.addFoldersChangeListener(folderChangeCB);
         service.messageStorage.removeChangeListener(watchId);
-        return t.done();
+        t.done();
+    });
+
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
 });
 
 </script>
index c99000553403509835d864305bf0ccfa354dc7f0..d524cc0cc7bc27b828647465c84b1c95b3b8249d 100644 (file)
@@ -49,43 +49,47 @@ Authors:
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== ONLOAD_DELAY 90
 //==== TEST_CRITERIA MOA
-        setup({timeout: 90000});
-        var t =async_test("MessageStorage_removeConversations_sms", {timeout: 90000});
-        var service, msg, filter, successCallback, errorCallback, conversationsArrayCB, queryErrorCB, messageSent, serviceListCB;
-
-        t.step(function () {
-            successCallback = t.step_func(function () {
-                return t.done();
-            });
-
-            errorCallback = t.step_func(function (error) {
-                assert_true(false, error.message);
-                t.done();
-            });
-
-            conversationsArrayCB = t.step_func(function (conversations) {
-                assert_true(conversations.length > 0, "find conversation");
-                service.messageStorage.removeConversations(conversations, successCallback, errorCallback);
-            });
-
-            queryErrorCB = t.step_func(function (error) {
-                assert_true(false, "Cannot query conversations" + error.message);
-                t.done();
-            });
-
-            messageSent = t.step_func(function (recipients) {
-                filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
-                service.messageStorage.findConversations(filter, conversationsArrayCB, queryErrorCB);
-            });
-
-            serviceListCB = t.step_func(function (services) {
-                service = services[0];
-                msg = new tizen.Message("messaging.sms", {plainBody: "test.", to: [TEST_SMS_RECIPIENT]});
-                service.sendMessage(msg, messageSent);
-            });
-
-            tizen.messaging.getMessageServices("messaging.sms", serviceListCB);
-        });
+setup({timeout: 90000});
+var t = async_test("MessageStorage_removeConversations_sms", {timeout: 90000}),
+    service, msg, filter, successCallback, errorCallback, conversationsArrayCB, queryErrorCB,
+    messageSent, serviceListCB, serviceError;
+
+t.step(function () {
+    successCallback = t.step_func(function () {
+        t.done();
+    });
+
+    errorCallback = t.step_func(function (error) {
+        assert_unreached("removeConversations() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    conversationsArrayCB = t.step_func(function (conversations) {
+        assert_greater_than(conversations.length, 0, "Received empty conversations array");
+        service.messageStorage.removeConversations(conversations, successCallback, errorCallback);
+    });
+
+    queryErrorCB = t.step_func(function (error) {
+        assert_unreached("findConversations() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    messageSent = t.step_func(function (recipients) {
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+        service.messageStorage.findConversations(filter, conversationsArrayCB, queryErrorCB);
+    });
+
+    serviceListCB = t.step_func(function (services) {
+        assert_greater_than(services.length, 0, "Received empty services array");
+        service = services[0];
+        msg = new tizen.Message("messaging.sms", {plainBody: "test.", to: [TEST_SMS_RECIPIENT]});
+        service.sendMessage(msg, messageSent);
+    });
+
+    serviceError = t.step_func(function (error) {
+        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
+    tizen.messaging.getMessageServices("messaging.sms", serviceListCB, serviceError);
+});
 
 </script>
 </body>
index 9bc661a29132bc0820222492a7e76da4487ab2a0..c98f9f41e368a0c95288e382cf77f223b6dd4b8d 100644 (file)
@@ -66,7 +66,7 @@ t.step(function () {
 
     messageChangeCB = {
         messagesupdated: t.step_func(function (messages) {
-            assert_true(messages.length > 0, "No messages updated!");
+            assert_greater_than(messages.length, 0, "No messages updated!");
             t.done();
         })
     };
@@ -78,7 +78,7 @@ t.step(function () {
     });
 
     messageSuccess = t.step_func(function (messages) {
-        assert_true(messages.length > 0, "No messages found.");
+        assert_greater_than(messages.length, 0, "No messages found.");
         messages[0].isRead = !messages[0].isRead;
         service.messageStorage.updateMessages(messages, updateSuccess, updateError);
     });
@@ -98,7 +98,7 @@ t.step(function () {
     });
 
     serviceSuccess = t.step_func(function (services) {
-        assert_true(services.length > 0, "No MessageService found");
+        assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
         msg = new tizen.Message("messaging.sms",
                 { plainBody : "test.", to : [ TEST_SMS_RECIPIENT_2] });