[messaging email] method_returns_undefined
authorWitold Choinkowski <w.choinkowsk@samsung.com>
Thu, 26 Sep 2013 16:27:11 +0000 (18:27 +0200)
committerGerrit Code Review <gerrit@gerrit.vlan144.tizendev.org>
Mon, 30 Sep 2013 09:28:56 +0000 (09:28 +0000)
Change-Id: I49af46d1f77503f328c5d3d5449f34ea958f4bf6

24 files changed:
tct-messaging-email-tizen-tests/messaging/MessageService_email_loadMessageAttachment.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_loadMessageAttachment_without_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_loadMessageBody.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_loadMessageBody_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_sendMessage.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_sendMessage_without_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_sendMessage_without_successCallback.html
tct-messaging-email-tizen-tests/messaging/MessageService_sendMessage_email_without_callbacks.html
tct-messaging-email-tizen-tests/messaging/MessageService_syncFolder.html
tct-messaging-email-tizen-tests/messaging/MessageService_syncFolder_with_limit.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_addConversationsChangeListener.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addDraftMessage.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addDraftMessage_without_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addDraftMessage_without_successCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages_without_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_findFolders.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_findMessage_emailFilterWithoutErrorCb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeChangeListener.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeConversations.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeConversations_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeConversations_with_successCallback.html
tct-messaging-email-tizen-tests/messaging/Messaging_email_getMessageServices.html
tct-messaging-email-tizen-tests/messaging/Messaging_email_getMessageServices_with_errorCallback.html

index 3113eab4e734002d2a56ffb8de3a0ccb8c8afe19..e3dea8ffe05e04d0e98f3feee7b9ee7cd9109adf 100755 (executable)
@@ -39,7 +39,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageService_email_loadMessageAttachment", {timeout:180000}),
-    service, inboxFolder, message, attachment, returnedValue,
+    service, inboxFolder, message, attachment, returnedValue = null,
     serviceSuccess, serviceError, sendSuccess, sendError, syncSuccess, syncError,
     findSuccess, findError, foundMessage, loadAttachmentSuccess, loadAttachmentError,
     loadBodySuccess, loadBodyError, loadMessage, folderSuccess, folderError;
index b5370ba5130fcabb497c417e2c90a2ea72817eb6..d1476ef2e0d72a2f8b0b1eacaa59566997ba630e 100755 (executable)
@@ -41,7 +41,7 @@ setup({timeout: 180000});
 
 var t = async_test("MessageService_email_loadMessageAttachment_without_errorCallback", {timeout: 180000}),
     service, inboxFolder, serviceSuccess, serviceError, sendSuccess, sendError,
-    message, attachment, syncSuccess, syncError, returnedValue, findSuccess, findError, foundMessage,
+    message, attachment, syncSuccess, syncError, returnedValue = null, findSuccess, findError, foundMessage,
     loadAttachmentSuccess, loadBodySuccess, loadBodyError, loadMessage, folderSuccess, folderError;
 
 t.step(function () {
index f7efd7e3b4b19420b9304b6a4d57d626cedf45e8..23d373811f2bee71657003ebbc7e2f31e1288ff3 100755 (executable)
@@ -106,6 +106,8 @@ t.step(function () {
         assert_equals(msg.inResponseTo, myMessage.inResponseTo, "inResponseTo is not equal");
         assert_own_property(msg, "messageStatus", "Name messageStatus doesn't exist in provided object.");
         assert_equals(msg.messageStatus, myMessage.messageStatus, "messageStatus is not equal");
+
+        assert_equals(returnedValue, undefined, "Incorrect returned value");
         t.done();
     });
 
@@ -113,7 +115,6 @@ t.step(function () {
         assert_draft_message_equals(t, messages, message);
         myMessage = messages[0];
         returnedValue = service.loadMessageBody(myMessage, loadMessageBodySuccess);
-        assert_equals(returnedValue, undefined, "Incorrect returned value");
     });
 
     findMessagesError = t.step_func(function (error) {
index 1c3c79937c23d4f859eb548974b2c8344757dccd..967dfafc660703866c94420067afef46b7c3d669 100755 (executable)
@@ -104,6 +104,8 @@ t.step(function () {
         assert_equals(msg.inResponseTo, myMessage.inResponseTo, "inResponseTo is not equal");
         assert_own_property(msg, "messageStatus", "Name messageStatus doesn't exist in provided object.");
         assert_equals(msg.messageStatus, myMessage.messageStatus, "messageStatus is not equal");
+
+        assert_equals(returnedValue, undefined, "Incorrect returned value");
         t.done();
     });
 
@@ -115,7 +117,6 @@ t.step(function () {
         assert_draft_message_equals(t, messages, message);
         myMessage = messages[0];
         returnedValue = service.loadMessageBody(messages[0], loadMessageBodySuccess, loadMessageBodyError);
-        assert_equals(returnedValue, undefined, "Incorrect returned value");
     });
 
     findMessagesError = t.step_func(function (error) {
index eebf242b68241aad65955f17a3dc87ccef81a021..843a1fe06941a51c364a10517b621f61e369484b 100755 (executable)
@@ -55,7 +55,7 @@ Authors:
 setup({ timeout: 180000 });
 
 var t = async_test("MessageService_email_sendMessage", { timeout: 180000 }),
-    message, loadedMessage, resend=0, requestSending, sendMessageReturnValue, i, service, inboxFolder,
+    message, loadedMessage, resend=0, requestSending, sendMessageReturnValue = null, i, service, inboxFolder,
     serviceSuccess, serviceError, sendSuccess, sendError, syncSuccess, syncError,
     folderSuccess, folderError, findSuccess, findError, bodySuccess, bodyError,
     attachmentSuccess, attachmentError;
index 32883acd436f73a9226d65ffb811a04131557007..850a0a94b2f63e3b462ab774782f0482c26d4f63 100755 (executable)
@@ -39,7 +39,7 @@ Authors:
 //==== TEST_CRITERIA MR MOA
 setup({ timeout: 180000 });
 var t = async_test("MessageService_email_sendMessage_without_errorCallback", { timeout: 180000 }),
-    message, returnedValue, serviceSuccess, sendSuccess, serviceError;
+    message, returnedValue = null, serviceSuccess, sendSuccess, serviceError;
 
 t.step(function () {
 
index 8a15a58682c39201500fae5f61ad00f656f23fd8..a22f45f147db178e080a6b7bd21ef5af49dd0a12 100755 (executable)
@@ -39,7 +39,7 @@ Authors:
 //==== TEST_CRITERIA MR MMINA
 setup({ timeout: 180000 });
 var t = async_test("MessageService_email_sendMessage_without_successCallback", { timeout: 180000 }),
-    message, returnedValue, serviceSuccess, serviceError;
+    message, returnedValue = null, serviceSuccess, serviceError;
 
 t.step(function () {
     serviceSuccess = t.step_func(function (service) {
index 53ba5ad71c4156e72a93f184dda70be27a56fde5..08795561f2c11c330df5d4cbb2ea68a69b8689e0 100755 (executable)
@@ -43,7 +43,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageService_sendMessage_email_without_callbacks", {timeout: 180000}),
-    serviceSuccess, serviceError, message, returnedValue;
+    serviceSuccess, serviceError, message, returnedValue = null;
 
 t.step(function () {
     serviceSuccess = t.step_func(function (service) {
index 49aad5e49a44577b84fbef448c5fc7aa56a2ac71..66d3b6a7b293b233f2668063931367acd51acc88 100755 (executable)
@@ -41,7 +41,7 @@ setup({timeout: 180000});
 
 var t = async_test("MessageService_syncFolder", {timeout: 180000}),
     servicesSuccess, servicesError, service, filter, findFoldersError, findFoldersSuccess, i, inboxFolder,
-    findFoldersSuccess, returnedValue;
+    findFoldersSuccess, returnedValue = null;
 
 t.step(function () {
     findSuccess = t.step_func(function (folders) {
index 27b74d15dd9f8d2895bb9a4a4c9d869312250e23..4da4f08b7ef9a0761b953d00f362160f7731881e 100755 (executable)
@@ -41,7 +41,7 @@ setup({timeout: 180000});
 
 var t = async_test("MessageService_syncFolder_with_limit", {timeout: 180000}),
     serviceSuccess, serviceError, service, findFoldersSuccess, findFoldersError,
-    inboxFolder, syncId, syncFolderSuccess, syncFolderError;
+    inboxFolder, syncId = null, syncFolderSuccess, syncFolderError;
 
 t.step(function () {
     syncFolderSuccess = t.step_func(function () {
index 3e2e27075ca03054a4fd8b09a2e39a1d8202f56e..4f9464dff3e4e33b572a45a5f274009dbe3999fa 100755 (executable)
@@ -52,7 +52,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_addConversationsChangeListener", {timeout: 180000}),
-    serviceSuccess, serviceError, messageService, message, addDraftError, conversationChangeCB, returnedValue;
+    serviceSuccess, serviceError, messageService, message, addDraftError, conversationChangeCB, returnedValue = null;
 
 t.step(function () {
     add_result_callback(function () {
index 5003ae80d7ed8ea41b5a13a8fbb58807ae7d898f..e1197b3f9d2e43a93bf913d3be5616c0ffca4d4c 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 var t = async_test("MessageStorage_email_addDraftMessage", {timeout:180000}),
     serviceSuccess, serviceError, service, message,
-    addDraftSuccess, addDraftError, returnedValue, filter, findMessagesSuccess, findMessagesError;
+    addDraftSuccess, addDraftError, returnedValue = null, filter, findMessagesSuccess, findMessagesError;
 
 t.step(function () {
     findMessagesSuccess = t.step_func(function (messages) {
index 3c5f25db4a351b42aa6aa332705e947fa2a3f965..128727f84f2612758df24258ea46d8b055e54cca 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_email_addDraftMessage_without_errorCallback", {timeout:180000}),
-    serviceSuccess, serviceError, service, message, returnedValue,
+    serviceSuccess, serviceError, service, message, returnedValue = null,
     addDraftSuccess, filter, findMessagesSuccess, findMessagesError;
 
 t.step(function () {
index c92285747feed1b3a9310a0387c3c36370d9ad5c..e69845d402ccc69abc6eb7d754b228318834f56d 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_email_addDraftMessage_without_successCallback", {timeout:180000}),
-    serviceSuccess, serviceError, message, returnedValue;
+    serviceSuccess, serviceError, message, returnedValue = null;
 
 t.step(function () {
     serviceSuccess = t.step_func(function (service) {
index e95c9e17859e866380ad1c8b0705daf7f981173c..2de399684a1a28841f16e78682b8543b0375e9a6 100755 (executable)
@@ -44,7 +44,7 @@ var t = async_test("MessageStorage_email_updateMessages", {timeout:180000}),
     mutableMessage, constMessage,
     plainBodyToSetForMutableMesage = MESSAGE_BODY_PLAIN + "_mutable",
     plainBodyToSetForConstMessage = MESSAGE_BODY_PLAIN + "_const",
-    addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue,
+    addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue = null,
     updateSuccess, updateError, mutableMessagefilter, constMessagefilter,
     findMutableMessagesSuccess, findMutableMessagesError, foundMutableMessage, mutable = "MUTABLE",
     findConstMessagesSuccess, findConstMessagesError, foundConstMessage, consts = "CONST",
index 850f83ccf5f2ada8ee1d1a3da7b2003b32d276a8..8fe3f78e80fd424f046d21469e8c940a19f3e494 100755 (executable)
@@ -45,7 +45,7 @@ t.step(function () {
         mutableMessage, constMessage,
         plainBodyToSetForMutableMesage = MESSAGE_BODY_PLAIN + "_mutable",
         plainBodyToSetForConstMessage = MESSAGE_BODY_PLAIN + "_const",
-        addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue,
+        addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue = null,
         updateSuccess, mutableMessagefilter, constMessagefilter,
         findMutableMessagesSuccess, findMutableMessagesError, foundMutableMessage, mutable = "MUTABLE",
         findConstMessagesSuccess, findConstMessagesError, foundConstMessage, consts = "CONST",
index 200d2130253f68bbebc61a81f84e97624605b671..12d8c8d3965896fa655c701ee972a29c8417a182 100755 (executable)
@@ -41,7 +41,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_findFolders", {timeout: 180000}),
-    serviceError, serviceSuccess, filter, foldersSuccess, returnedValue;
+    serviceError, serviceSuccess, filter, foldersSuccess, returnedValue = null;
 
 t.step(function () {
     foldersSuccess = t.step_func(function () {
index 3939f6218c2c7bbb4eaee00c16859c13bcaf9066..d9e7e02100adecc146a5b63bcecbaec14744e5c9 100755 (executable)
@@ -52,18 +52,20 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_findMessage_emailFilterWithoutErrorCb", {timeout: 180000}),
-    serviceError, serviceSuccess, service, message,
+    serviceError, serviceSuccess, service, message, returnedValue = null,
     addDraftSuccess, addDraftError, filter, findSuccess;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_draft_message_equals(t, messages, message);
+
+        assert_equals(returnedValue, undefined, "Incorrect returned value");
         t.done();
     });
 
     addDraftSuccess = t.step_func(function () {
         filter = new tizen.AttributeFilter("id", "EXACTLY", message.id);
-        service.messageStorage.findMessages(filter, findSuccess);
+        returnedValue = service.messageStorage.findMessages(filter, findSuccess);
     });
 
     addDraftError = t.step_func(function (error) {
index 3206a363425504b59f741b917cf6ca2d133724f9..feb21a2d6b2775053f98706b9e51358b9b0e832a 100755 (executable)
@@ -41,7 +41,7 @@ setup({timeout: 180000});
 
 var t = async_test("MessageStorage_removeChangeListener", {timeout: 180000}),
     serviceSuccess, serviceError, service, message1, message2, listenerId,
-    addDraftSuccess, addDraftError, messageChangeCallback;
+    addDraftSuccess, addDraftError, messageChangeCallback, returnedValue = null;
 
 t.step(function () {
     //cleanup after TC
index e7e7a4b92d01ca98e6dc2ff63314348b7a1cdd22..4e6edf785e3cffe292aff1564b13133e268bd0a4 100755 (executable)
@@ -39,7 +39,7 @@ Authors:
 //==== TEST_CRITERIA MMINA MR
 setup({timeout: 180000});
 
-var t = async_test("MessageStorage_removeConversations", {timeout: 180000}), returnedValue;
+var t = async_test("MessageStorage_removeConversations", {timeout: 180000}), returnedValue = null;
 
 message_conversation_test(t, function (service, message, conversation) {
     returnedValue = service.messageStorage.removeConversations([conversation]);
index fc7b911e56a2d52a6cd4bce2b57f178dfc974fe7..5ec3f50daa58c3f3cdcd865c24405496ea1c543b 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_removeConversations_with_errorCallback", {timeout: 180000}),
-    removeSuccess, removeError, returnedValue, findFilter, findSuccessAfterRemove, findErrorAfterRemove;
+    removeSuccess, removeError, returnedValue = null, findFilter, findSuccessAfterRemove, findErrorAfterRemove;
 
 message_conversation_test(t, function (service, message, conversation) {
 
index 5c5c1afc0125ba8ec1d35afa5dc1b7ca10809ca1..1f477ecee4a0195752f664c8949a9f6b87e587df 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("MessageStorage_removeConversations_with_successCallback", {timeout: 180000}),
-    removeSuccess, returnedValue;
+    removeSuccess, returnedValue = null;
 
 message_conversation_test(t, function (service, message, conversation) {
 
index 3c5534c7a45cbc49456ba22c4393003bb99a9789..6d45eb44ef4bc88a9063327ba088733cd52ce822 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test("Messaging_email_getMessageServices", {timeout:180000}),
-    serviceSuccess, i, returnedValue;
+    serviceSuccess, i, returnedValue = null;
 
 t.step(function () {
     serviceSuccess = t.step_func(function (services) {
index 504c331f14d1e88a6d7489c5d5844caa56ef9017..3974bae2588d72d3ec6e0e3275c321ed6d523a27 100755 (executable)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 var t = async_test("Messaging_email_getMessageServices_with_errorCallback", {timeout:180000});
 t.step(function () {
-    var serviceSuccess, serviceError, i, returnedValue;
+    var serviceSuccess, serviceError, i, returnedValue = null;
 
     serviceSuccess = t.step_func(function (services) {
         assert_equals(returnedValue, undefined, "Incorrect returned value.");