[Messaging-sms] applied HQ patch with minor adjustment
authorAndrzej Krolikowski <a.krolikowsk@samsung.com>
Thu, 26 Sep 2013 11:16:24 +0000 (13:16 +0200)
committerAndrzej Krolikowski <a.krolikowsk@samsung.com>
Thu, 26 Sep 2013 11:16:24 +0000 (13:16 +0200)
Change-Id: I4a309886cc3c7f67b45f176d7eb820292820d4e7

13 files changed:
tct-messaging-sms-tizen-tests/messaging/ConversationArraySuccessCallback_onsuccess_checking.html
tct-messaging-sms-tizen-tests/messaging/MessageArraySuccessCallback_onsuccess_checking.html
tct-messaging-sms-tizen-tests/messaging/MessageConversationArraySuccessCallback_onsuccess_exist.html
tct-messaging-sms-tizen-tests/messaging/MessageService_loadBodySMS.html
tct-messaging-sms-tizen-tests/messaging/MessageService_sms_sendMessage.html
tct-messaging-sms-tizen-tests/messaging/MessageService_sms_sendMessage_without_errorCallback.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_findMessages.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_findMessages_errorCallback_TypeMismatch_1.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_removeConversations_forsms.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_sms_findConversations_with_limit.html
tct-messaging-sms-tizen-tests/messaging/MessageStorage_sms_findMessages_from_with_errorCallback.html
tct-messaging-sms-tizen-tests/messaging/Message_sms_from_attribute.html
tct-messaging-sms-tizen-tests/messaging/support/messaging_common.js

index c520622e62ca741c76ab98176b90842299475632..5179c9035f34a488e21a1c5647656858b092814c 100644 (file)
@@ -52,7 +52,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("ConversationArraySuccessCallback_onsuccess_checking", {timeout: 90000}),
-    serviceError, serviceSuccess, addSuccess, findAllConversationsError, findAllConversationsSuccess, service;
+    serviceError, serviceSuccess, addSuccess, filter, findAllConversationsError, findAllConversationsSuccess, service;
 
 t.step(function () {
     findAllConversationsSuccess = t.step_func(function (conversations) {
@@ -65,7 +65,8 @@ t.step(function () {
     });
 
     addSuccess = t.step_func(function () {
-        service.messageStorage.findConversations(typeFilter, findAllConversationsSuccess, findAllConversationsError);
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+        service.messageStorage.findConversations(filter, findAllConversationsSuccess, findAllConversationsError);
     });
 
     serviceSuccess = t.step_func(function (services) {
index 20348926fa739551e39e68ed1e4610efc4fdc227..868789671ccb20b1431fd735c68b48255998d2eb 100644 (file)
@@ -52,8 +52,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageArraySuccessCallback_onsuccess_checking", {timeout: 90000}),
-    service, findMessageSuccess, findMessageError, addSuccess, serviceSuccess,
-    serviceError;
+    service, filter, findMessageSuccess, findMessageError, addSuccess, serviceSuccess, serviceError;
 
 t.step(function () {
     findMessageSuccess = t.step_func(function (messages) {
@@ -68,7 +67,8 @@ t.step(function () {
     });
 
     addSuccess = t.step_func(function () {
-        service.messageStorage.findMessages(typeFilter, findMessageSuccess, findMessageError);
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+        service.messageStorage.findMessages(filter, findMessageSuccess, findMessageError);
     });
 
     serviceSuccess = t.step_func(function (services) {
index c6d44710232891f457ee1f5cd355ed9c92b60ec2..e8d4ee66b32cd0e2785af5fa5f7c0aeeab750065 100644 (file)
@@ -48,21 +48,11 @@ Authors:
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== TEST_CRITERIA CBOA
 setup({timeout: 90000});
-var t = async_test("MessageConversationArraySuccessCallback_onsuccess_exist", {timeout:90000}), serviceSuccess, filter,
-    findConversationsSuccess, findConversationsError, msg, datetime, service, onReceiveMsg, messageStorage,
-    serviceError;
-t.step(function () {
-    //cleanup after TC
-    add_result_callback(function () {
-        try {
-            if (sendAndReceiveMessageListenerId !== null) {
-                messageStorage.removeChangeListener(sendAndReceiveMessageListenerId);
-            }
-        } catch (err) {
-            // do nothing in case removeChangeListener throw an exception
-        }
-    });
+var t = async_test("MessageConversationArraySuccessCallback_onsuccess_exist", {timeout:90000}),
+    serviceSuccess, serviceError, message, datetime, service, messageStorage, filter,
+    sendSuccess, sendError, findConversationsSuccess, findConversationsError;
 
+t.step(function () {
     findConversationsError = t.step_func(function (error) {
         assert_unreached("findConversations() error callback: name:" + error.name + ", msg:" + error.message);
     });
@@ -84,23 +74,27 @@ t.step(function () {
         t.done();
     });
 
-    onReceiveMsg = t.step_func(function (message) {
+    sendSuccess = t.step_func(function () {
         filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
         service.messageStorage.findConversations(filter, findConversationsSuccess, findConversationsError);
     });
 
+    sendError = t.step_func(function (error) {
+        assert_unreached("sendMessage() error callback: name:" + error.name + ", msg:" + error.message);
+    });
+
     serviceSuccess = t.step_func(function (services) {
         assert_greater_than(services.length, 0, "Received empty services array");
         service = services[0];
         messageStorage = service.messageStorage;
 
         datetime = new Date().getTime().toString();
-        msg = new tizen.Message("messaging.sms", {
+        message = new tizen.Message("messaging.sms", {
             plainBody: datetime,
             to: [ TEST_SMS_RECIPIENT ]
         });
 
-        sendAndReceiveMessage(t, service, msg, datetime, onReceiveMsg);
+        service.sendMessage(message, sendSuccess, sendError);
     });
 
     serviceError = t.step_func(function (error) {
index cf4d4eb1ea2459333ac861e294e15d70d433632a..acf4acca7a37e6cb654c958042464139dee4f8aa 100644 (file)
@@ -48,26 +48,16 @@ Authors:
 //==== SPEC Tizen Web API:Communication:Messaging:MessageService:loadMessageBody M
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== ONLOAD_DELAY 90
-//==== TEST_CRITERIA MOA MR
+//==== TEST_CRITERIA MOA
 setup({timeout: 90000});
 
-var t = async_test("MessageService_loadBodySMS", {timeout: 90000}), messageStorage,
-    msg, service, loadBodySuccess, loadBodyError, onReceiveMsg, serviceSuccess, serviceError, datetime,
-    returnedValue = null;
+var t = async_test("MessageService_loadBodySMS", {timeout: 90000}),
+    serviceSuccess, serviceError, service, messageStorage, message, datetime,
+    sendSuccess, sendError, loadBodySuccess, loadBodyError, returnedValue;
 
 t.step(function () {
-    //cleanup after TC
-    add_result_callback(function () {
-        try {
-            if (sendAndReceiveMessageListenerId !== null) {
-                messageStorage.removeChangeListener(sendAndReceiveMessageListenerId);
-            }
-        } catch (err) {
-            // do nothing in case removeChangeListener throw an exception
-        }
-    });
-
     loadBodySuccess = t.step_func(function () {
+        assert_equals(returnedValue, undefined, "Incorrect returned value");
         t.done();
     });
 
@@ -75,21 +65,24 @@ t.step(function () {
         assert_unreached("loadMessageBody() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
-    onReceiveMsg = t.step_func(function (message) {
+    sendSuccess = t.step_func(function () {
         returnedValue = service.loadMessageBody(message, loadBodySuccess, loadBodyError);
-        assert_equals(returnedValue, undefined, "Incorrect returned value");
+    });
+
+    sendError = t.step_func(function (error) {
+        assert_unreached("sendMessage() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
     serviceSuccess = t.step_func(function (services) {
         service = services[0];
         messageStorage = service.messageStorage;
         datetime = tizen.time.getCurrentDateTime().toString();
-        msg = new tizen.Message("messaging.sms", {
+        message = new tizen.Message("messaging.sms", {
             plainBody: datetime,
             to: [ TEST_SMS_RECIPIENT ]
         });
 
-        sendAndReceiveMessage(t, service, msg, datetime, onReceiveMsg);
+        service.sendMessage(message, sendSuccess, sendError);
     });
 
     serviceError = t.step_func(function (error) {
index 22ac78219c77b75f549512ddd2e9320311838e9f..824cb6d8ac2533990a2ff7947a996d5eee1a4e07 100644 (file)
@@ -52,28 +52,19 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageService_sms_sendMessage", {timeout: 90000}),
-    message, datetime, messageStorage,
-    service, serviceSuccess, serviceError,
-    onReceiveMsg;
+    message, datetime, messageStorage, sendSuccess, sendError,
+    service, serviceSuccess, serviceError;
 
 t.step(function () {
-    //cleanup after TC
-    add_result_callback(function () {
-        try {
-            if (sendAndReceiveMessageListenerId !== null) {
-                messageStorage.removeChangeListener(sendAndReceiveMessageListenerId);
-            }
-        } catch (err) {
-            // do nothing in case removeChangeListener throw an exception
-        }
+    sendSuccess = t.step_func(function (recipients) {
+        assert_type(recipients, "array", "recipients type is not array");
+        assert_equals(recipients.length, 1, "recipients.length should be 1");
+        assert_equals(recipients[0], TEST_SMS_RECIPIENT, "recipient in sent message is not the same as recipient set during message creation");
+        t.done();
     });
 
-    onReceiveMsg = t.step_func(function (receivedMessage) {
-        assert_equals(receivedMessage.body.plainBody, datetime, "plainBody values in sent message and received message are not equal.");
-
-        assert_greater_than_equal(receivedMessage.from.indexOf(TEST_SMS_RECIPIENT), 0, "Recipient differs from the one set in sent message");
-
-        t.done();
+    sendError = t.step_func(function (error) {
+        assert_unreached("sendMessage() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
     serviceSuccess = t.step_func(function (services) {
@@ -86,7 +77,7 @@ t.step(function () {
             to: [ TEST_SMS_RECIPIENT ]
         });
 
-        sendAndReceiveMessage(t, service, message, datetime, onReceiveMsg);
+        service.sendMessage(message, sendSuccess, sendError);
     });
 
     serviceError = t.step_func(function (error) {
index 6b8ccdf25c4c04074f76bb7d35a86ab5a85a0f81..f582c45585114dcb451a3446639976d6e4461a64 100644 (file)
@@ -47,7 +47,6 @@ t.step(function () {
     sendSuccess = t.step_func(function (recipients) {
         assert_type(recipients, "array", "recipients type is not array");
         assert_equals(recipients.length, 1, "recipients.length should be 1");
-        assert_equals(recipients[0], TEST_SMS_RECIPIENT, "recipient in sent message is not the same as recipient set during message creation");
         assert_greater_than_equal(recipients[0].indexOf(TEST_SMS_RECIPIENT), 0, "recipient in sent message is not the same as" +
             "recipient set during message creation");
 
index d1c893669358decd452d927d43dacb9510ac12e7..df615d2c2981b31dd9a79aca095a7d7b29ff17f2 100644 (file)
@@ -52,7 +52,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_findMessages", {timeout: 90000}),
-    smsService, findMessagesCallback, findError,
+    smsService, findMessagesCallback, findError, filter,
     getMessagingServicesCallback, addSuccess, serviceError;
 t.step(function () {
 
@@ -66,7 +66,8 @@ t.step(function () {
     });
 
     addSuccess = t.step_func(function () {
-        smsService.messageStorage.findMessages(typeFilter, findMessagesCallback, findError);
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+        smsService.messageStorage.findMessages(filter, findMessagesCallback, findError);
     });
 
     getMessagingServicesCallback = t.step_func(function (services) {
index 7c345f38e7ea5dfe3eed777140be5a0f2d03e58d..3b3a68f45df755df232c1dc22d0c56beda99cf56 100644 (file)
@@ -43,7 +43,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_findMessages_errorCallback_TypeMismatch_1", {timeout: 90000}),
-    serviceSuccess, serviceError, findSuccess;
+    serviceSuccess, serviceError, findSuccess, filter;
 t.step(function () {
 
     findSuccess = t.step_func(function () {
@@ -52,8 +52,11 @@ t.step(function () {
 
     serviceSuccess = t.step_func(function (services) {
         assert_greater_than(services.length, 0, "Received empty services array");
+
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+
         assert_throws({name: TYPE_MISMATCH_ERR}, function () {
-            services[0].messageStorage.findMessages(typeFilter, findSuccess, 1);
+            services[0].messageStorage.findMessages(filter, findSuccess, 1);
         });
 
         t.done();
index cabad921e875d46137db9ef8bd5380a5b2341908..5935ee809d55ea7a05666f0c598a64b66089db14 100644 (file)
@@ -51,7 +51,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_removeConversations_forsms", {timeout: 90000}),
-    service, serviceError, serviceSuccess, findError, findSuccess, removeError, removeSuccess,
+    service, serviceError, serviceSuccess, filter, findError, findSuccess, removeError, removeSuccess,
     onSuccess;
 
 t.step(function () {
@@ -72,7 +72,8 @@ t.step(function () {
     });
 
     onSuccess = t.step_func(function () {
-        service.messageStorage.findConversations(typeFilter, findSuccess, findError);
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+        service.messageStorage.findConversations(filter, findSuccess, findError);
     });
 
     serviceSuccess = t.step_func(function (services) {
index 5340f1fbab51c17cc63aed3f3afee495e65a4841..5daf4fdb09f703bc6af9ff87dd3b23af45551c9a 100644 (file)
@@ -43,7 +43,7 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_sms_findConversations_with_limit", {timeout: 90000}),
-    serviceError, serviceSuccess, service, onSuccess, limit = 1, findError, findSuccess;
+    serviceError, serviceSuccess, service, onSuccess, filter, limit = 1, findError, findSuccess;
 
 t.step(function () {
 
@@ -57,7 +57,8 @@ t.step(function () {
     });
 
     onSuccess = t.step_func(function () {
-        service.messageStorage.findConversations(typeFilter, findSuccess, findError, null, limit);
+        filter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
+        service.messageStorage.findConversations(filter, findSuccess, findError, null, limit);
     });
 
     serviceSuccess = t.step_func(function (services) {
index 18abf2e4aa3ea417b74209e359117fa70293b089..117d19bc996a94fb7e6d096682b0c5b2711c1eed 100644 (file)
@@ -39,96 +39,39 @@ Authors:
 setup({timeout: 90000});
 
 var t = async_test("MessageStorage_sms_findMessages_from_with_errorCallback", {timeout: 90000}),
-    getMessageServicesErrorCB, getMessageServicesSuccessCB, messageService, messageStorage,
-    message, messageReceived, findSuccessCB, findErrorCB, i, onMessagesAdded, plainBodyFilter,
-    datetime, filterSuccess = 0, watchId = null, callbackListener, fromAndPlainBodyFilter,
-    sendSuccess, sendError, returnedValue, fromFound;
+    serviceSuccess, serviceError, service, returnedValue, filter, findSuccessCB, findErrorCB,
+    fromFound, i;
+
 t.step(function () {
-    //cleanup after TC
-    add_result_callback(function () {
-        try {
-            if (watchId !== null) {
-                messageStorage.removeChangeListener(watchId);
-            }
-        } catch (err) {
-            // do nothing in case removeChangeListener throw an exception
-        }
-    });
+    findSuccessCB = t.step_func(function (messages) {
+        assert_equals(returnedValue, undefined, "Incorrect returned value");
 
-    findSuccessCB = t.step_func(function (msg) {
-        assert_greater_than(msg.length, 0, "No messages found");
-        for (i = 0; i < msg.length; i++) {
-            assert_not_equals(msg[i].from, null, "from is null");
-            fromFound = msg[i].from.indexOf(TEST_SMS_RECIPIENT);
+        for (i = 0; i < messages.length; i++) {
+            assert_not_equals(messages[i].from, null, "from is null");
+            fromFound = messages[i].from.indexOf(TEST_SMS_RECIPIENT);
             assert_greater_than_equal(fromFound, 0, "Recipient not found");
-
-            if (msg[i].id === messageReceived.id) {
-                filterSuccess++;
-            }
         }
 
-        assert_equals(filterSuccess, 1, "Incorrect number of found message");
         t.done();
     });
 
     findErrorCB = t.step_func(function (error) {
-        assert_unreached("findMessage() error callback: name:" + error.name + ", msg:" + error.message);
-    });
-
-    onMessagesAdded = t.step_func(function (messages) {
-        assert_equals(messages.length, 1, "Incorrect number of messages.");
-        messageReceived = messages[0];
-        assert_equals(messageReceived.body.plainBody, datetime, "Incorrect plainBody found");
-        assert_not_equals(messageReceived.id, message.id, "Incorrect id found - id of sent message");
-
-        messageStorage.removeChangeListener(watchId);
-        watchId = null;
-
-        returnedValue = messageStorage.findMessages(fromFilter, findSuccessCB, findErrorCB);
-        assert_equals(returnedValue, undefined, "Incorrect returned value");
-    });
-
-    callbackListener = {
-        messagesupdated: function (messages) {
-            assert_unreached("[callbackListener]  messagesupdated   " + messages.length + " message(s) updated. [0] = " + messages[0]);
-        },
-        messagesadded: onMessagesAdded,
-        messagesremoved: function (messages) {
-            assert_unreached("[callbackListener]  messagesremoved   " + messages.length + " message(s) removed. [0] = " + messages[0]);
-        }
-    };
-
-    sendSuccess = t.step_func(function () {
-        //just ignore this
+        assert_unreached("findMessages() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
-    sendError = t.step_func(function (error) {
-        assert_unreached("sendMessage() error callback: name:" + error.name + ", msg:" + error.message);
-    });
-
-    getMessageServicesSuccessCB = t.step_func(function (services) {
+    serviceSuccess = t.step_func(function (services) {
         assert_greater_than(services.length, 0, "Received empty services array");
+        service = services[0];
 
-        datetime = tizen.time.getCurrentDateTime().toString();
-        message = new tizen.Message("messaging.sms", {
-            plainBody: datetime,
-            to: [ TEST_SMS_RECIPIENT ]
-        });
-
-        messageService = services[0];
-        messageStorage = messageService.messageStorage;
-
-        plainBodyFilter = new tizen.AttributeFilter("body.plainBody", "EXACTLY", datetime);
-        fromAndPlainBodyFilter = new tizen.CompositeFilter("INTERSECTION", [fromFilter, plainBodyFilter]);
-        watchId = messageStorage.addMessagesChangeListener(callbackListener, fromAndPlainBodyFilter);
-        messageService.sendMessage(message, sendSuccess, sendError);
+        filter = new tizen.AttributeFilter("from", "CONTAINS", TEST_SMS_RECIPIENT);
+        returnedValue = service.messageStorage.findMessages(filter, findSuccessCB, findErrorCB);
     });
 
-    getMessageServicesErrorCB = t.step_func(function (error) {
+    serviceError = t.step_func(function (error) {
         assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
     });
 
-    tizen.messaging.getMessageServices("messaging.sms", getMessageServicesSuccessCB, getMessageServicesErrorCB);
+    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
 });
 
 </script>
index 7bea0be9245e21e9433f8f03cac839958f5011ef..149909c512d46d62f2dad29fb429d36fbbb676b7 100644 (file)
@@ -37,44 +37,14 @@ Authors:
 //==== SPEC_URL https://developer.tizen.org/help/topic/org.tizen.web.device.apireference/tizen/messaging.html
 //==== ONLOAD_DELAY 90
 //==== TEST_CRITERIA AE ADV ARO AT
-setup({timeout: 90000});
 
-var t = async_test("Message_sms_from_attribute", {timeout: 90000}), message,
-    service, onSuccess, currentDate = new Date(), serviceError, serviceSuccess,
-    plainBodyToSet = currentDate.getTime().toString(), originalFrom;
-t.step(function () {
-    message = new tizen.Message("messaging.sms");
+test(function () {
+    var message = new tizen.Message("messaging.sms");
 
     assert_true("from" in message, "No 'from' attribute in message");
     assert_equals(message.from, null, "'from' should be null by default");
     check_readonly(message, "from", null, "object", message.from + "dummyValue");
-
-    onSuccess = t.step_func(function (receivedMessage) {
-        assert_type(receivedMessage.from, "string", "from is not a string");
-        assert_greater_than(receivedMessage.from.indexOf(TEST_SMS_RECIPIENT), -1, "Incorrect from attribute");
-
-        originalFrom = receivedMessage.from;
-        receivedMessage.from = TEST_SMS_RECIPIENT_2;
-        assert_equals(receivedMessage.from, originalFrom, "from can be modified");
-
-        t.done();
-    });
-
-    serviceSuccess = t.step_func(function (services) {
-        assert_greater_than(services.length, 0, "Received empty services array");
-        service = services[0];
-
-        message = new tizen.Message("messaging.sms", smsMessageInitDict);
-        message.body.plainBody = plainBodyToSet;
-        sendAndReceiveMessage(t, service, message, plainBodyToSet, onSuccess);
-    });
-
-    serviceError = t.step_func(function (error) {
-        assert_unreached("getMessageServices() error callback: name:" + error.name + ", msg:" + error.message);
-    });
-
-    tizen.messaging.getMessageServices("messaging.sms", serviceSuccess, serviceError);
-});
+}, "Message_sms_from_attribute");
 
 </script>
 </body>
index f5a1e709bc5286ce3674b07897a2b222a9725b69..2dc5aeded233aac30e9a4f36ffcf46c2a3b4d45e 100644 (file)
@@ -25,10 +25,6 @@ var TEST_SMS_RECIPIENT_2 = ""; // this variable MUST be set before executing tes
 
 var sendMessageReturnValue;
 var smsService;
-var sendAndReceiveMessageListenerId = null;
-
-var typeFilter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
-var fromFilter = new tizen.AttributeFilter("from", "CONTAINS", TEST_SMS_RECIPIENT);
 
 var TYPE_MISMATCH_ERR  = 'TypeMismatchError';
 var INVALID_VALUES_ERR = 'InvalidValuesError';
@@ -44,58 +40,6 @@ var smsMessageInitDict = {
 };
 
 
-/*
-- send message SMS or MMS (msg)
-- wait for a message with body.plainBody equal to expectedPlainBody
-- call given callback (onReceive)
-- remember to remove sendAndReceiveMessageListenerId (if !== null)
-*/
-function sendAndReceiveMessage(t, service, msg, expectedPlainBody, onReceive) {
-  var callback;
-
-  function onReceiveInternal(messages) {
-      t.step(function () {
-          var i, found = -1;
-
-          for (i = 0 ; i < messages.length ; i++) {
-              if (messages[i].body.plainBody === expectedPlainBody && msg.id !== messages[i].id && messages[i].from !== null && messages[i].from !== undefined) {
-                  found = i;
-                  break;
-              }
-          }
-
-          if (found > -1) {
-              service.messageStorage.removeChangeListener(sendAndReceiveMessageListenerId);
-              sendAndReceiveMessageListenerId = null;
-              onReceive(messages[found]);
-          }
-      });
-  }
-
-  callback = {
-      messagesupdated: function () {},
-      messagesadded: onReceiveInternal,
-      messagesremoved: function () {}
-  };
-
-  function sendSuccess() {
-  }
-
-  function sendError(error) {
-      t.step(function () {
-          service.messageStorage.removeChangeListener(sendAndReceiveMessageListenerId);
-          sendAndReceiveMessageListenerId = null;
-          assert_unreached("Failed to send message: " + error.message);
-      });
-  }
-
-  t.step(function () {
-      sendAndReceiveMessageListenerId = service.messageStorage.addMessagesChangeListener(callback);
-
-      service.sendMessage(msg, sendSuccess, sendError);
-  });
-}
-
 function addDraft(t, service, initDict, onSuccess) {
     var msg, addSuccess, addError, serviceListSuccess, serviceListError;
 
@@ -111,13 +55,14 @@ function addDraft(t, service, initDict, onSuccess) {
     service.messageStorage.addDraftMessage(msg, addSuccess, addError);
 }
 
+
 function addSMSDraft(t, service, initDict, onSuccess) {
     addDraft(t, service, initDict, onSuccess);
 }
 
 
 function removeAllMessages(t, service, onSuccess) {
-    var msg, findSuccess, findError, removeSuccess, removeError, msgs;
+    var msg, findSuccess, findError, removeSuccess, removeError, msgs, typeFilter;
 
     removeError = t.step_func(function (error) {
         assert_unreached("failed to remove messages: " + error.message);
@@ -142,6 +87,8 @@ function removeAllMessages(t, service, onSuccess) {
     t.step(function () {
         assert_equals(service.type, "messaging.sms",
             "Fix your test: unknown service type: " + service.type);
+
+        typeFilter = new tizen.AttributeFilter("type", "EXACTLY", "messaging.sms");
         service.messageStorage.findMessages(typeFilter, findSuccess, findError);
     });
 }