[Messaging-email] cleanup
authorAndrzej Krolikowski <a.krolikowsk@samsung.com>
Thu, 10 Oct 2013 12:32:14 +0000 (14:32 +0200)
committerAndrzej Krolikowski <a.krolikowsk@samsung.com>
Thu, 10 Oct 2013 13:22:12 +0000 (15:22 +0200)
Change-Id: I119f7909b755cfe17cb110f564dfd8f8f8d8f6a1

49 files changed:
tct-messaging-email-tizen-tests/messaging/MessageAttachment_constructor_minargs.html
tct-messaging-email-tizen-tests/messaging/MessageBody_email_plainBody_attribute.html
tct-messaging-email-tizen-tests/messaging/MessageConversation_from_attribute.html
tct-messaging-email-tizen-tests/messaging/MessageConversationsChangeCallback_conversationsadded.html
tct-messaging-email-tizen-tests/messaging/MessageConversationsChangeCallback_email_conversationsupdated.html
tct-messaging-email-tizen-tests/messaging/MessageService_email_loadMessageAttachment_exist.html
tct-messaging-email-tizen-tests/messaging/MessageService_sendMessage_emailWithBcc.html
tct-messaging-email-tizen-tests/messaging/MessageService_sync.html
tct-messaging-email-tizen-tests/messaging/MessageService_syncFolder_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageService_syncFolder_with_successCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_addConversationsChangeListener.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_addConversationsChangeListener_with_filter.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_addConversationsChangeListener_withoutcb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addDraftMessage_errorCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addDraftMessage_missarg.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addDraftMessage_successCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addFoldersChangeListener_foldersChangeCallback_TypeMismatch.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_addMessagesChangeListener_without_filter.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findConversations_filter_from.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findConversations_filter_to.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findConversations_without_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_bcc_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_cc_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_folderId_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_from_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_hasAttachment_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_id_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_isHighPriority_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_isRead_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_plainBody_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_serviceId_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_timestamp_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_to_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_findMessages_type_with_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages_errorCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages_successCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages_without_errorCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_email_updateMessages_without_successCallback.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_findFolders_errorCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_findFolders_successCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeChangeListener.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeConversations_conversations_TypeMismatch.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeMessages_email_errorCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeMessages_email_missarg.html
tct-messaging-email-tizen-tests/messaging/MessageStorage_removeMessages_email_successCallback_invalid_cb.html
tct-messaging-email-tizen-tests/messaging/MessagesChangeCallback_email_messagesadded_without_filter.html
tct-messaging-email-tizen-tests/messaging/MessagesChangeCallback_email_messagesremoved_with_filter.html
tct-messaging-email-tizen-tests/messaging/MessagesChangeCallback_email_messagesremoved_without_filter.html

index f707409f295b5cb4822a427431a4e9587e5f7b94..30571b0300940a1ddc49d9d08f9be29a55d8ecb3 100644 (file)
@@ -44,7 +44,8 @@ test(function () {
     assert_equals(attachment.id, null, "'attachment.id' should be null by default");
     assert_equals(attachment.messageId, null, "'attachment.messageId' should be null by default");
     assert_equals(attachment.mimeType, null, "'attachment.mimeType' null by default");
-    assert_equals(attachment.filePath, MESSAGE_ATTACHMENT_IMAGE_PATH, "'attachment.filePath' should be \""+MESSAGE_ATTACHMENT_IMAGE_PATH+"\" by default");
+    assert_equals(attachment.filePath, MESSAGE_ATTACHMENT_IMAGE_PATH,
+        "'attachment.filePath' should be \""+MESSAGE_ATTACHMENT_IMAGE_PATH+"\" by default");
 
 }, document.title);
 
index b3300709b7fd74b08c5344d1933a7dc460e41a15..075c16e85869a343cef733fca239bdb761862ef0 100644 (file)
@@ -45,7 +45,7 @@ test(function () {
     check_attribute(messageBody, "plainBody", "", "string", "sampleValue");
 
     messageBody.plainBody = null;
-    assert_equals(messageBody.plainBody, "null", "null is incorrectly converted to 'null'. plainBody is not nullable string so after set to plainBody null. plainBody should be equals 'null'");
+    assert_equals(messageBody.plainBody, "null", "Incorrect plainBody after null assignment");
 
 }, document.title);
 
index 9728f20b44980b74f0fdf6f174142e8dfedb399a..0cbbcf1c63cdbae5045ae3a1d8165a9b25fa91cc 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, sendSuccess, sendError, syncSuccess, syncError,
-    filter, findSuccess, findError;
+    filter, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (conversations) {
         assert_type(conversations, "array", "Not an array");
         assert_greater_than(conversations.length, 0, "Received empty messages array");
 
-        for (var i in conversations) {
+        for (i = 0; i < conversations.length; i++) {
             assert_own_property(conversations[i], "from", "MessageConversation does not own from property.");
             assert_equals(conversations[i].from, TEST_EMAIL_RECIPIENT_1, "returned from is not equal");
             check_readonly(conversations[i], "from", conversations[i].from, "string", "dummyValue");
index b3a731ac2b0ca52d04ce2e2eb7e77b6da41b4af0..c1d551fe31e72ee0e3c87e0ecdb35c914bd8a018 100644 (file)
@@ -50,7 +50,7 @@ Authors:
 //==== TIMEOUT 180
 //==== TEST_CRITERIA CBOA CBT
 setup({timeout: 180000});
-var t = async_test(document.title, {timeout: 180000}), service,
+var t = async_test(document.title, {timeout: 180000}), service, i,
     serviceSuccess, serviceError, messageService, addDraftError, message, conversationChangeCB, watchId;
 
 t.step(function () {
@@ -67,7 +67,7 @@ t.step(function () {
             assert_type(conversations, "array", "type check");
             assert_greater_than(conversations.length, 0, "Received empty conversations array");
 
-            for (var i in conversations) {
+            for (i = 0; i < conversations.length; i++) {
                 assert_own_property(conversations[i], "id", "id not present.");
                 assert_own_property(conversations[i], "type", "type not present.");
                 assert_own_property(conversations[i], "timestamp", "timestamp not present.");
index dcb18ca200352f666b0c184ff5cbdbab793f9232..2e1343d5cc8e46ae8ac9b36f7f23107bf65c79be 100644 (file)
@@ -39,7 +39,7 @@ Authors:
 //==== TEST_CRITERIA CBT CBOA
 setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, messageService, messageStorage, message,
+    serviceSuccess, serviceError, messageService, messageStorage, message, i,
     addDraftSuccess, addDraftError, updateError, oldTimestamp, changeCallback, listenerId;
 
 t.step(function () {
@@ -58,7 +58,7 @@ t.step(function () {
             assert_type(updatedConversations, "array", "Not an array");
             assert_greater_than(updatedConversations.length, 0, "Received empty conversations array");
 
-            for (var i in updatedConversations) {
+            for (i = 0; i < updatedConversations.length; i++) {
                 if (updatedConversations[i].id === message.conversationId) {
                     check_readonly(updatedConversations[i], "id", updatedConversations[i].id, "string", "dummyValue");
                     check_readonly(updatedConversations[i], "type", "messaging.email", "string", "dummyValue");
index 91ac88bcdf60fce127fa86dfe6f35a093f1a64f2..9ad56d089522ef0f8e1b7d7c6bac5fb9461b4dfa 100644 (file)
@@ -39,16 +39,16 @@ Authors:
 //==== TEST_CRITERIA ME
 setup({timeout: 180000});
 
-var t = async_test(document.title, {timeout: 180000}), message_service, serviceSuccess, serviceError;
+var t = async_test(document.title, {timeout: 180000}), service, serviceSuccess, serviceError;
 
 t.step(function () {
 
     serviceSuccess = t.step_func(function (services) {
-        assert_true(services.length > 0, "Received empty services array");
-        message_service = services[0];
+        assert_greater_than(services.length, 0, "Received empty services array");
+        service = services[0];
 
-        assert_true("loadMessageAttachment" in message_service, "No loadMessageAttachment method in message_service");
-        check_method_exists(message_service, "loadMessageAttachment");
+        assert_true("loadMessageAttachment" in service, "No loadMessageAttachment method in service");
+        check_method_exists(service, "loadMessageAttachment");
 
         t.done();
     });
index 441432a95831a16a7e4ad44eea6b6994aa103e0b..a9795a2c842b58c810844a5f8cfcc464e42472f7 100644 (file)
@@ -56,7 +56,7 @@ var t = async_test(document.title, {timeout: 180000}),
 t.step(function () {
 
     messageSent = t.step_func(function (recipients) {
-        assert_true(recipients.length > 0, "No message sent successfully");
+        assert_greater_than(recipients.length, 0, "Received empty recipients array");
         t.done();
     });
 
index 05751b9e4e5f1620929e045ae2569bfeeb1edb1e..4c3b667b11ab66c70f4218a90b2df70072e4ad30 100644 (file)
@@ -46,7 +46,7 @@ t.step(function () {
 
     serviceSuccess = t.step_func(function (service) {
         returnedValue = service.sync();
-        assert_type(returnedValue, "number", "type check");
+        assert_type(returnedValue, "long", "type check");
         service.stopSync(returnedValue);
         t.done();
     });
index 9292b3181db97b09b32e10a4ea2637e1b2ecfb14..be1a34448547fcee2fca21377fa5c95a4c4efd85 100644 (file)
@@ -42,11 +42,11 @@ setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, findFoldersError, findFoldersSuccess,
     inboxFolder, message, sendError, sendSuccess, syncId = null,
-    syncFolderSuccess, syncFolderError, findMessageError, findMesageSuccess;
+    syncFolderSuccess, syncFolderError, findMessageError, findMessageSuccess;
 
 t.step(function () {
 
-    findMesageSuccess = t.step_func(function (messages) {
+    findMessageSuccess = t.step_func(function (messages) {
         assert_inbox_message_equals(t, messages, message, inboxFolder);
         t.done();
     });
@@ -57,7 +57,7 @@ t.step(function () {
 
     syncFolderSuccess = t.step_func(function () {
         assert_type(syncId, "long", "Incorrect returned value");
-        findMessages(t, service, inboxFolder, message.subject, findMesageSuccess, findMessageError);
+        findMessages(t, service, inboxFolder, message.subject, findMessageSuccess, findMessageError);
     });
 
 
index 7a6f0f26644f06e612dbace3882ff6b7cebe6a63..4cf985fafea72e85c675e497fe85e7ac93eebe48 100644 (file)
@@ -42,11 +42,11 @@ setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, findFoldersError, findFoldersSuccess,
     inboxFolder, message, sendError, sendSuccess, syncId = null, syncFolderSuccess,
-    findMessageError, findMesageSuccess;
+    findMessageError, findMessageSuccess;
 
 t.step(function () {
 
-    findMesageSuccess = t.step_func(function (messages) {
+    findMessageSuccess = t.step_func(function (messages) {
         assert_inbox_message_equals(t, messages, message, inboxFolder);
         t.done();
     });
@@ -57,7 +57,7 @@ t.step(function () {
 
     syncFolderSuccess = t.step_func(function () {
         assert_type(syncId, "long", "Incorrect returned value");
-        findMessages(t, service, inboxFolder, message.subject, findMesageSuccess, findMessageError);
+        findMessages(t, service, inboxFolder, message.subject, findMessageSuccess, findMessageError);
     });
 
     findFoldersSuccess = t.step_func(function (folders) {
index 129d56d906724a5be47b2411787f08f95e771444..9fd0693787fda7f1adfcfb9cbdb1b157eb0a96d4 100644 (file)
@@ -51,7 +51,7 @@ Authors:
 //==== TEST_CRITERIA MR MMINA MAST
 setup({timeout: 180000});
 
-var t = async_test(document.title, {timeout: 180000}), service,
+var t = async_test(document.title, {timeout: 180000}), service, i,
     serviceSuccess, serviceError, messageService, message, addDraftError, conversationChangeCB, returnedValue = null;
 
 t.step(function () {
@@ -67,7 +67,7 @@ t.step(function () {
         conversationsadded: t.step_func(function (conversations) {
             assert_greater_than(conversations.length, 0, "Received empty conversations array");
 
-            for (var i in conversations) {
+            for (i = 0; i < conversations.length; i++) {
                 if (conversations[i].id === message.conversationId) {
                     t.done();
                     break;
@@ -86,7 +86,7 @@ t.step(function () {
         message = createSimpleMessageTo(TEST_EMAIL_RECIPIENT_2);
 
         returnedValue = messageService.messageStorage.addConversationsChangeListener(conversationChangeCB);
-        assert_type(returnedValue, "number", "Incorrect returned value type - should be 'number'");
+        assert_type(returnedValue, "long", "Incorrect returned value type");
 
         messageService.messageStorage.addDraftMessage(message, null, addDraftError);
     });
index f1624d3fe535014e701f3bd7a94aabca8bc68b2a..5314af729cd48f439f058488c1b0995879b101a4 100644 (file)
@@ -38,7 +38,7 @@ Authors:
 //==== TEST_CRITERIA MOA MAST
 setup({timeout: 180000});
 
-var t = async_test(document.title, {timeout: 180000}), addDraftError,
+var t = async_test(document.title, {timeout: 180000}), addDraftError, i,
     service, serviceSuccess, serviceError, message, conversationChangeCB, filter, watchId;
 
 t.step(function () {
@@ -54,7 +54,7 @@ t.step(function () {
         conversationsadded: t.step_func(function (conversations) {
             assert_greater_than(conversations.length, 0, "Received empty conversations array");
 
-            for (var i in conversations) {
+            for (i = 0; i < conversations.length; i++) {
                 if (conversations[i].id === message.conversationId) {
                     t.done();
                     break;
index 5368289ddd0404a10e0d0cd13f53ba7e74f7256b..bd2e651c5ce59d78696ce37e33dd45909134f20f 100644 (file)
@@ -58,7 +58,7 @@ t.step(function () {
     serviceSuccess = t.step_func(function (service) {
         assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
             service.messageStorage.addConversationsChangeListener();
-        }, "TypeMismatchError exception expected but not thrown.");
+        }, "Missing non-optional argument");
 
         t.done();
     });
index 9e44866d9828c38c110a51897812173faf1799f8..21b4fce2c054ef48663a50a976b266504db0917b 100644 (file)
@@ -50,7 +50,7 @@ t.step(function () {
 
     incorrectCallback = {
         onerror: t.step_func(function (error) {
-           assert_unreached("addDraftMessage() incorrect error callback invoked: name:" + error.name + ", msg:" + error.message);
+            assert_unreached("addDraftMessage() incorrect error callback invoked: name:" + error.name + ", msg:" + error.message);
         })
     };
 
index 4d6407580231211c0757f888bd3290a837dd5f7d..ade0c97efee3b1dea3b181bf58cac7666f5e1814 100644 (file)
@@ -40,14 +40,13 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, exceptionName = "TypeMismatchError";
+    serviceSuccess, serviceError;
 
 t.step(function () {
     serviceSuccess = t.step_func(function (service) {
-        assert_throws({name: exceptionName},
-            function () {
-                 service.messageStorage.addDraftMessage();
-            }, exceptionName + " should be thrown - missing argument");
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.addDraftMessage();
+        }, "Missing argument");
         t.done();
     });
 
index a2512797858b91cd50e5e1ba562f83156a725cab..f4717648f8e6bde419ae74427ae100140f8253c0 100644 (file)
@@ -40,12 +40,12 @@ Authors:
 //==== TEST_CRITERIA MTCB
 setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, message, incorrectCallback, addDraftError, exceptionName = "TypeMismatchError";
+    serviceSuccess, serviceError, message, incorrectCallback, addDraftError;
 
 t.step(function () {
     incorrectCallback = {
         onsuccess: t.step_func(function () {
-           assert_unreached("addDraftMessage() success callback invoked.");
+            assert_unreached("addDraftMessage() success callback invoked.");
         })
     };
 
@@ -56,10 +56,9 @@ t.step(function () {
     serviceSuccess = t.step_func(function (service) {
         message = createSimpleMessageTo(TEST_EMAIL_RECIPIENT_2);
 
-        assert_throws({name: exceptionName},
-            function () {
-                service.messageStorage.addDraftMessage(message, incorrectCallback, addDraftError);
-            }, exceptionName + " should be thrown - given incorrect success callback - " + incorrectCallback);
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.addDraftMessage(message, incorrectCallback, addDraftError);
+        }, "Given incorrect success callback - " + incorrectCallback);
 
         t.done();
     });
index a760dc51b0fc6b9fde7aadca93d57280e5a4d7a6..ad39038d628fab1f4f5bf2750e4aca44471d9da7 100644 (file)
@@ -54,10 +54,9 @@ t.step(function () {
             incorrectCallback = conversionTable[i][0];
             exceptionName = conversionTable[i][1];
 
-            assert_throws({name: exceptionName},
-            function () {
-                 service.messageStorage.addFoldersChangeListener(incorrectCallback);
-            }, exceptionName + " should be thrown - given incorrect error callback - " + incorrectCallback);
+            assert_throws({name: exceptionName}, function () {
+                service.messageStorage.addFoldersChangeListener(incorrectCallback);
+            }, "Given incorrect error callback - " + incorrectCallback);
         }
 
         t.done();
index 81bcaafc6b9aa62251d6c35ecb93a37311365cd4..0ff8e4e23e29fdbb2aa7c43919017bd0c4c854e5 100644 (file)
@@ -38,7 +38,7 @@ Authors:
 //==== TIMEOUT 180
 //==== TEST_CRITERIA MR MMINA MAST
 setup({timeout: 180000});
-var t = async_test(document.title, {timeout: 180000}),
+var t = async_test(document.title, {timeout: 180000}), i,
     serviceSuccess, serviceError, service, message, listenerId, addDraftError, changeCallback;
 
 t.step(function () {
@@ -56,7 +56,7 @@ t.step(function () {
             assert_type(addedMessages, "array", "Not an array.");
             assert_greater_than(addedMessages.length, 0, "Received empty messages array");
 
-            for (var i in addedMessages) {
+            for (i = 0; i < addedMessages.length; i++) {
                 assert_true(addedMessages[i] instanceof tizen.Message, "Not a message.");
 
                 if (addedMessages[0].id === message.id) {
index 0de3695002c71f25a03ce0dbf19fcda072e31545..6e0b315d9c75ff05b509932e20a326cc85419f9d 100644 (file)
@@ -51,14 +51,14 @@ Authors:
 //==== TIMEOUT 180
 setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, service, message, filter,
+    serviceSuccess, serviceError, service, message, filter, i,
     sendSuccess, sendError, syncSuccess, syncError, findSuccess, findError;
 
 t.step(function () {
     findSuccess = t.step_func(function (conversations) {
         assert_greater_than(conversations.length, 0, "Received empty conversations array");
 
-        for (var i in conversations) {
+        for (i = 0; i < conversations.length; i++) {
             assert_equals(conversations[i].from, TEST_EMAIL_RECIPIENT_1, "from should match");
         }
 
index 6eeecea0d9f8f07a5c3662db22ca7935c79ccad4..075fbe549edd05688b86d312e7d223790cd4182b 100644 (file)
@@ -51,14 +51,14 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, service, message,
+    serviceSuccess, serviceError, service, message, i,
     addDraftSuccess, addDraftError, filter, findSuccess, findError;
 
 t.step(function () {
     findSuccess = t.step_func(function (conversations) {
         assert_greater_than(conversations.length, 0, "Received empty conversations array");
 
-        for (var i in conversations) {
+        for (i = 0; i < conversations.length; i++) {
             assert_array_equals(conversations[i].to, [TEST_EMAIL_RECIPIENT_2], "to should match");
         }
 
index 678ee2eb859b756250b2f1df56dc2afe043f4942..f45cc2ee75dba167bf902da4e67b4d168970b430 100644 (file)
@@ -50,7 +50,7 @@ Authors:
 //==== TEST_CRITERIA MMINA
 setup({timeout: 180000});
 
-var t = async_test(document.title, {timeout: 180000}),
+var t = async_test(document.title, {timeout: 180000}), i,
     serviceSuccess, serviceError, service, message, filter, addDraftError, findConversationsSuccess;
 
 t.step(function () {
@@ -58,7 +58,7 @@ t.step(function () {
         assert_type(conversations, "array", "Not an array");
         assert_greater_than(conversations.length, 0, "Received empty messages array");
 
-        for (var i in conversations) {
+        for (i = 0; i < conversations.length; i++) {
             assert_equals(conversations[i].type, service.type, "Incorrect service type");
         }
 
index f1e754cf8040ab22658c484448901286e848dceb..852ea7e2ae3aad9ae0851ad75ea3f6ec732fe24a 100644 (file)
@@ -44,14 +44,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_in_array(TEST_EMAIL_RECIPIENT_1, messages[i].bcc, "bcc not found");
 
index 5232eec07032bc6647c2f3e5c3fcfa0f190634b0..f03efe1f0766c9045843f39a943ace64c9fb748a 100644 (file)
@@ -42,14 +42,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_in_array(TEST_EMAIL_RECIPIENT_1, messages[i].cc, "cc not found");
 
index d708af0de9ba45c78b741678ae31f0ed37e98977..325ab6639acfcb4d1384a8e033ee7195d4fde1f3 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].folderId, message.folderId, "returned folderId is not equal");
 
index 52e4826df16f0db13e93f93a01bfa7b12a91d594..b16ccff453cb61e5e1ca093f093cf1606c7fb96b 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, sendSuccess, sendError, syncSuccess, syncError,
-    filter, findSuccess, findError;
+    filter, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].from, TEST_EMAIL_RECIPIENT_1, "returned from is not equal");
         }
index 7a40a328106aadf7cba158af8b44a52c31db3524..b0ec19cdb1b9892b5131f5c4192f56dc851ee621 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
-   findSuccess = t.step_func(function (messages) {
+    findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].hasAttachment, true, "returned hasAttachment is not equal to true");
 
index 930532f38cf624272ac8e654879c6c0bc5f66425..dea973b8c4b06f0b483401b280c14f197d05abbd 100644 (file)
@@ -40,11 +40,11 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, service, message, filter, messageFound = false,
+    serviceSuccess, serviceError, service, message, filter,
     addDraftSuccess, addDraftError, findSuccess, findError;
 
 t.step(function () {
-   findSuccess = t.step_func(function (messages) {
+    findSuccess = t.step_func(function (messages) {
         assert_draft_message_equals(t, messages, message);
         t.done();
     });
index 4fe4297360272e9d4a131afb500144d05e23dc4a..be1d67a0226ad094e7a8defea4fb1070e0328e60 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
-   findSuccess = t.step_func(function (messages) {
+    findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].isHighPriority, true, "returned isHighPriority is not equal to true");
 
index 1af68553bdda779d84c0876adf525a5d03bc661d..faaaff4c12d03c0d80f513084faf1250de13630d 100644 (file)
@@ -43,14 +43,14 @@ var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false, inboxMessageId,
     sendSuccess, sendError, syncSuccess, syncError, findFoldersSuccess, findFoldersError,
     findInboxMessageSuccess, findInboxMessageError, updateSuccess, updateError,
-    findUpdatedMessagesSuccess, findUpdatedMessagesError, inboxFolder;
+    findUpdatedMessagesSuccess, findUpdatedMessagesError, inboxFolder, i;
 
 t.step(function () {
-   findUpdatedMessagesSuccess = t.step_func(function (messages) {
+    findUpdatedMessagesSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].isRead, true, "returned isRead is not equal to true");
 
@@ -85,8 +85,6 @@ t.step(function () {
         inboxMessageId = messages[0].id;
 
         service.messageStorage.updateMessages(messages, updateSuccess, updateError);
-
-        t.done();
     });
 
     findInboxMessageError = t.step_func(function (error) {
index 1d03375d9b5028e18ae45701b4a781e9e277da68..01735a283474ac02d1fa59dfc9a1c2de33f073b0 100644 (file)
@@ -41,7 +41,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}), serviceSuccess, serviceError, service, message, addDraftSuccess,
-    addDraftError, filter, findSuccess, findError, loadMessageBodySuccess, loadMessageBodyError;
+    addDraftError, filter, findSuccess, findError, loadMessageBodySuccess, loadMessageBodyError, i;
 
 t.step(function () {
 
@@ -59,7 +59,7 @@ t.step(function () {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             if (messages[i].id === message.id) {
                 service.loadMessageBody(messages[i], loadMessageBodySuccess, loadMessageBodyError);
index ffbe16c2fe38d98a150903ac2c266e17e5b59963..12c9a45c51362c9314ffdc276cad0090ac6b14a6 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
-   findSuccess = t.step_func(function (messages) {
+    findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].type, service.type, "Invalid service type");
 
index d9a9dff1ca61ffef8236e45a924f083a4e3faa84..5dd4a9233296a0a9c29c4a52d4615b29f32d2238 100644 (file)
@@ -39,16 +39,16 @@ Authors:
 //==== TEST_CRITERIA MOA
 setup({timeout: 180000});
 
-var t = async_test(document.title, {timeout: 180000}),
+var t = async_test(document.title, {timeout: 180000}), i,
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
     addDraftSuccess, addDraftError, findSuccess, findError, timeRange = 180000;
 
 t.step(function () {
-   findSuccess = t.step_func(function (messages) {
+    findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
 
             if (messages[i].id === message.id) {
index 9282f902c584f3d6b384af0545ec177f3cbc6400..1c40ce68ccab7349085ff71245ba6cc9cd43847d 100644 (file)
@@ -40,14 +40,14 @@ Authors:
 setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_in_array(TEST_EMAIL_RECIPIENT_2, messages[i].to, "to not found");
 
index 633d7b56f4ff7131fa65da10f1f65a281048a076..cab2fa7d6aee7709898bfa9c4acc365b1dc8600d 100644 (file)
@@ -41,14 +41,14 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, filter, messageFound = false,
-    addDraftSuccess, addDraftError, findSuccess, findError;
+    addDraftSuccess, addDraftError, findSuccess, findError, i;
 
 t.step(function () {
     findSuccess = t.step_func(function (messages) {
         assert_type(messages, "array", "Not an array");
         assert_greater_than(messages.length, 0, "Received empty messages array");
 
-        for (var i in messages) {
+        for (i = 0; i < messages.length; i++) {
             assert_true(messages[i] instanceof tizen.Message, "Not a Message");
             assert_equals(messages[i].type, message.type, "returned type is not equal");
 
index bf297290747aa904e696875da70589791df19844..abce193a2a51a55590d067b92d14c878413f976d 100644 (file)
@@ -42,7 +42,7 @@ setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service,
     mutableMessage, constMessage,
-    plainBodyToSetForMutableMesage = MESSAGE_BODY_PLAIN + "_mutable",
+    plainBodyToSetForMutableMessage = MESSAGE_BODY_PLAIN + "_mutable",
     plainBodyToSetForConstMessage = MESSAGE_BODY_PLAIN + "_const",
     addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue = null,
     updateSuccess, updateError, mutableMessagefilter, constMessagefilter,
@@ -102,7 +102,7 @@ t.step(function () {
     });
 
     loadConstAttachmentSuccess = t.step_func(function (attachment) {
-         testAttachment(attachment, consts);
+        testAttachment(attachment, consts);
     });
 
     loadConstAttachmentError = t.step_func(function (error) {
@@ -126,12 +126,17 @@ t.step(function () {
         assert_equals(messageToTest.body.plainBody, originalMessage.body.plainBody, type + " body.plainBody attribute.");
         assert_equals(messageToTest.body.htmlBody, originalMessage.body.htmlBody, type + " body.htmlBody attribute.");
 
-        assert_equals(messageToTest.body.inlineAttachments.length, originalMessage.body.inlineAttachments.length, type + " incorrect length of inlineAttachments");
+        assert_equals(messageToTest.body.inlineAttachments.length, originalMessage.body.inlineAttachments.length,
+            type + " incorrect length of inlineAttachments");
         for (i = 0; i < originalMessage.body.inlineAttachments.length; i++) {
-            assert_equals(messageToTest.body.inlineAttachments[i].id, originalMessage.body.inlineAttachments[i].id, type + " message.body.inlineAttachments[" + i + "].id");
-            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.body.inlineAttachments[i].messageId, type + " message.body.inlineAttachments[" + i + "].messageId");
-            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.id, type + " message.body.inlineAttachments[" + i + "].messageId - (with message.id)");
-            assert_equals(messageToTest.body.inlineAttachments[i].mimeType, originalMessage.body.inlineAttachments[i].mimeType, type + " message.body.inlineAttachments[" + i + "].mimeType");
+            assert_equals(messageToTest.body.inlineAttachments[i].id, originalMessage.body.inlineAttachments[i].id,
+                type + " message.body.inlineAttachments[" + i + "].id");
+            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.body.inlineAttachments[i].messageId,
+                type + " message.body.inlineAttachments[" + i + "].messageId");
+            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.id,
+                type + " message.body.inlineAttachments[" + i + "].messageId - (with message.id)");
+            assert_equals(messageToTest.body.inlineAttachments[i].mimeType, originalMessage.body.inlineAttachments[i].mimeType,
+                type + " message.body.inlineAttachments[" + i + "].mimeType");
             assert_type(messageToTest.body.inlineAttachments[i].filePath, "string",  type + " attachment.filePath is not a string");
             assert_not_equals(messageToTest.body.inlineAttachments[i].filePath, "", type + " attachment.filePath is empty");
         }
@@ -230,7 +235,7 @@ t.step(function () {
         mutableMessage = new tizen.Message("messaging.email", {
             subject: generateSubject(),
             to: [TEST_EMAIL_RECIPIENT_2],
-            plainBody: plainBodyToSetForMutableMesage,
+            plainBody: plainBodyToSetForMutableMessage,
             htmlBody: MESSAGE_BODY_HTML,
             isHighPriority: false
         });
index bd52dec5dda635922f7be059792f02808083cb48..7e0201c1467699b602f83af72a21d2d9a238bc5b 100644 (file)
@@ -42,7 +42,7 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message,
-    addDraftSuccess, addDraftError, exceptionName = "TypeMismatchError",
+    addDraftSuccess, addDraftError,
     updateSuccess, incorrectCallback;
 
 t.step(function () {
@@ -52,15 +52,14 @@ t.step(function () {
 
     incorrectCallback = {
         onerror: t.step_func(function (error) {
-           assert_unreached("addDraftMessage() incorrect error callback invoked: name:" + error.name + ", msg:" + error.message);
+            assert_unreached("addDraftMessage() incorrect error callback invoked: name:" + error.name + ", msg:" + error.message);
         })
     };
 
     addDraftSuccess = t.step_func(function () {
-         assert_throws({name: exceptionName},
-            function () {
-                service.messageStorage.updateMessages([message], updateSuccess, incorrectCallback);
-            }, exceptionName + " should be thrown - given incorrect error callback - " + incorrectCallback);
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.updateMessages([message], updateSuccess, incorrectCallback);
+        }, "Given invalid error callback");
 
         t.done();
     });
index cf9533cdf6445f38e2516e5fe019f60492701b7a..de6aac3031b298e6c3dec26a01b17a11e33f75a6 100644 (file)
@@ -52,7 +52,7 @@ t.step(function () {
 
     incorrectCallback = {
         onsuccess: t.step_func(function () {
-           assert_unreached("addDraftMessage() incorrect success callback invoked");
+            assert_unreached("addDraftMessage() incorrect success callback invoked");
         })
     };
 
index 2ae111245d99ed64e72435323c2e40e01d3ad90b..1e71e73e381f3a32af5f02fa983b1d00f83e79a2 100644 (file)
@@ -43,7 +43,7 @@ var t = async_test(document.title, {timeout: 180000});
 t.step(function () {
     var serviceSuccess, serviceError, service,
         mutableMessage, constMessage,
-        plainBodyToSetForMutableMesage = MESSAGE_BODY_PLAIN + "_mutable",
+        plainBodyToSetForMutableMessage = MESSAGE_BODY_PLAIN + "_mutable",
         plainBodyToSetForConstMessage = MESSAGE_BODY_PLAIN + "_const",
         addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue = null,
         updateSuccess, mutableMessagefilter, constMessagefilter,
@@ -102,7 +102,7 @@ t.step(function () {
     });
 
     loadConstAttachmentSuccess = t.step_func(function (attachment) {
-         testAttachment(attachment, consts);
+        testAttachment(attachment, consts);
     });
 
     loadConstAttachmentError = t.step_func(function (error) {
@@ -126,12 +126,17 @@ t.step(function () {
         assert_equals(messageToTest.body.plainBody, originalMessage.body.plainBody, type + " body.plainBody attribute.");
         assert_equals(messageToTest.body.htmlBody, originalMessage.body.htmlBody, type + " body.htmlBody attribute.");
 
-        assert_equals(messageToTest.body.inlineAttachments.length, originalMessage.body.inlineAttachments.length, type + " incorrect length of inlineAttachments");
+        assert_equals(messageToTest.body.inlineAttachments.length, originalMessage.body.inlineAttachments.length,
+            type + " incorrect length of inlineAttachments");
         for (i = 0; i < originalMessage.body.inlineAttachments.length; i++) {
-            assert_equals(messageToTest.body.inlineAttachments[i].id, originalMessage.body.inlineAttachments[i].id, type + " message.body.inlineAttachments[" + i + "].id");
-            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.body.inlineAttachments[i].messageId, type + " message.body.inlineAttachments[" + i + "].messageId");
-            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.id, type + " message.body.inlineAttachments[" + i + "].messageId - (with message.id)");
-            assert_equals(messageToTest.body.inlineAttachments[i].mimeType, originalMessage.body.inlineAttachments[i].mimeType, type + " message.body.inlineAttachments[" + i + "].mimeType");
+            assert_equals(messageToTest.body.inlineAttachments[i].id, originalMessage.body.inlineAttachments[i].id,
+                type + " message.body.inlineAttachments[" + i + "].id");
+            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.body.inlineAttachments[i].messageId,
+                type + " message.body.inlineAttachments[" + i + "].messageId");
+            assert_equals(messageToTest.body.inlineAttachments[i].messageId, originalMessage.id,
+                type + " message.body.inlineAttachments[" + i + "].messageId - (with message.id)");
+            assert_equals(messageToTest.body.inlineAttachments[i].mimeType, originalMessage.body.inlineAttachments[i].mimeType,
+                type + " message.body.inlineAttachments[" + i + "].mimeType");
             assert_type(messageToTest.body.inlineAttachments[i].filePath, "string",  type + " attachment.filePath is not a string");
             assert_not_equals(messageToTest.body.inlineAttachments[i].filePath, "", type + " attachment.filePath is empty");
         }
@@ -236,7 +241,7 @@ t.step(function () {
         mutableMessage = new tizen.Message("messaging.email", {
             subject: generateSubject(),
             to: [TEST_EMAIL_RECIPIENT_2],
-            plainBody: plainBodyToSetForMutableMesage,
+            plainBody: plainBodyToSetForMutableMessage,
             htmlBody: MESSAGE_BODY_HTML,
             isHighPriority: false
         });
index 7bc2b405a16adda3c783de98cd9e4c856fd2aa98..da406312b9711889b58e785639629d6921acd46d 100644 (file)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}), serviceSuccess, serviceError, service, mutableMessage,
-    constMessage, plainBodyToSetForMutableMesage = MESSAGE_BODY_PLAIN + "_mutable", plainBodyToSetForConstMessage = MESSAGE_BODY_PLAIN + "_const",
+    constMessage, plainBodyToSetForMutableMessage = MESSAGE_BODY_PLAIN + "_mutable", plainBodyToSetForConstMessage = MESSAGE_BODY_PLAIN + "_const",
     addDraftSuccess, addDraftError, addDraftSuccessCounter = 0, returnedValue;
 t.step(function () {
 
@@ -79,7 +79,7 @@ t.step(function () {
         mutableMessage = new tizen.Message("messaging.email", {
             subject: generateSubject(),
             to: [TEST_EMAIL_RECIPIENT_2],
-            plainBody: plainBodyToSetForMutableMesage,
+            plainBody: plainBodyToSetForMutableMessage,
             htmlBody: MESSAGE_BODY_HTML,
             isHighPriority: false
         });
index c04ffe235805cedc5aec97e600e6846f579a8b96..503417ed39ae26ac9e1b28780aa8a7f7a243131a 100644 (file)
@@ -50,19 +50,16 @@ t.step(function () {
 
     incorrectCallback = {
         onerror: t.step_func(function () {
-           assert_unreached("findFolders() incorrect error callback invoked");
+            assert_unreached("findFolders() incorrect error callback invoked");
         })
     };
 
     serviceSuccess = t.step_func(function (service) {
         filter = new tizen.AttributeFilter("serviceId", "EXACTLY", service.id);
 
-        assert_throws(TYPE_MISMATCH_EXCEPTION,
-            function () {
-                service.messageStorage.findFolders(filter, findSuccess, incorrectCallback);
-            },
-            "TypeMismatch expected when invalid error callback given."
-        );
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.findFolders(filter, findSuccess, incorrectCallback);
+        }, "Given invalid error callback");
 
         t.done();
     });
index b73c0d71156bd917e8cdf5cd08fbd95c4671c6ba..2b5540836d203e687170ad9e7844a58f9861bb6a 100644 (file)
@@ -46,19 +46,16 @@ var t = async_test(document.title, {timeout: 180000}),
 t.step(function () {
     incorrectCallback = {
         onsuccess: t.step_func(function () {
-           assert_unreached("findFolders() incorrect error callback invoked");
+            assert_unreached("findFolders() incorrect error callback invoked");
         })
     };
 
     serviceSuccess = t.step_func(function (service) {
         filter = new tizen.AttributeFilter("serviceId", "EXACTLY", service.id);
 
-        assert_throws(TYPE_MISMATCH_EXCEPTION,
-            function () {
-                service.messageStorage.findFolders(filter, incorrectCallback);
-            },
-            "TypeMismatch expected when invalid success callback given."
-        );
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.findFolders(filter, incorrectCallback);
+        }, "Given invalid success callback");
 
         t.done();
     });
index c8c0a449d401e1be1dc76f2fad00d88ffcbcfd45..78e7d79066030a3c3a67312ece8ead1bd504054d 100644 (file)
@@ -40,7 +40,7 @@ Authors:
 setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
-    serviceSuccess, serviceError, service, message1, message2, listenerId,
+    serviceSuccess, serviceError, service, message1, message2, listenerId, i,
     addDraftSuccess, addDraftError, messageChangeCallback, returnedValue = null;
 
 t.step(function () {
@@ -68,7 +68,7 @@ t.step(function () {
             assert_type(addedMessages, "array", "Not an array");
             assert_greater_than(addedMessages.length, 0, "Received empty messages array");
 
-            for (var i in addedMessages) {
+            for (i = 0; i < addedMessages.length; i++) {
                 if (addedMessages[i].id === message1.id) {
                     returnedValue = service.messageStorage.removeChangeListener(listenerId);
                     assert_equals(returnedValue, undefined, "Incorrect value returned.");
index 79ed6716dff12c521db3936c25a616b33fb76ee3..d4d5ca0b5080000cba565c6596ecd851027f6bb5 100644 (file)
@@ -49,8 +49,8 @@ message_conversation_test(t, function (service, message, conversation) {
         exceptionName = conversionTable[i][1];
 
         assert_throws({name: exceptionName}, function () {
-             service.messageStorage.removeConversations(conversations);
-        }, exceptionName + " should be thrown - given incorrect successCallback.");
+            service.messageStorage.removeConversations(conversations);
+        }, "Given incorrect conversations");
     }
     t.done();
 });
index c7ce4024940911e74a53dea1dde73cf982a0075d..6afe2fe90834a8c154a20eadb345f761e0e1ff00 100644 (file)
@@ -42,7 +42,7 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, addDraftSuccess, addDraftError,
-    successCallback, exceptionName = "TypeMismatchError", incorrectCallback;
+    successCallback, incorrectCallback;
 
 t.step(function () {
     successCallback = t.step_func(function () {
@@ -56,10 +56,9 @@ t.step(function () {
     };
 
     addDraftSuccess = t.step_func(function () {
-        assert_throws({name: exceptionName},
-            function () {
-                service.messageStorage.removeMessages([message], successCallback, incorrectCallback);
-        }, exceptionName + " should be thrown - given incorrect success callback.");
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.removeMessages([message], successCallback, incorrectCallback);
+        }, "Incorrect error callback.");
 
         t.done();
     });
index 679ac95fcb11f3c8040bec392151225c4a612740..7132c461ec1e1dc85a52e429a38cf313d9a2f418 100644 (file)
@@ -44,10 +44,9 @@ var t = async_test(document.title, {timeout: 180000}),
 
 t.step(function () {
     addDraftSuccess = t.step_func(function () {
-        assert_throws(TYPE_MISMATCH_EXCEPTION,
-            function () {
-                service.messageStorage.removeMessages();
-        }, "TypeMismatchError" + " should be thrown - given incorrect success callback.");
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.removeMessages();
+        }, "Missing argument.");
         t.done();
     });
 
index 2e81e5c844f557f8d7b0b35572cc37a8f7171281..bd0016483afd818905c58a0c0925d5ae79744c47 100644 (file)
@@ -42,7 +42,7 @@ setup({timeout: 180000});
 
 var t = async_test(document.title, {timeout: 180000}),
     serviceSuccess, serviceError, service, message, addDraftSuccess, addDraftError,
-    exceptionName = "TypeMismatchError", incorrectCallback;
+    incorrectCallback;
 
 t.step(function () {
     incorrectCallback = {
@@ -52,10 +52,10 @@ t.step(function () {
     };
 
     addDraftSuccess = t.step_func(function () {
-        assert_throws({name: exceptionName},
-            function () {
-                service.messageStorage.removeMessages([message], incorrectCallback);
-        }, exceptionName + " should be thrown - given incorrect success callback.");
+        assert_throws(TYPE_MISMATCH_EXCEPTION, function () {
+            service.messageStorage.removeMessages([message], incorrectCallback);
+        }, "Given incorrect success callback.");
+
         t.done();
     });
 
index 43a0be86b8232cdfa7a7f7fd23615291ed548a2e..4758759b6803b0674f8f7837599aebc025badfa0 100644 (file)
@@ -40,7 +40,7 @@ Authors:
 //==== TEST_CRITERIA CBOA CBT
 setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}), serviceSuccess, serviceError, service, watcherId,
-    message, changeCallback, addDraftError;
+    message, changeCallback, addDraftError, i;
 t.step(function () {
 
     add_result_callback(function (res) {
@@ -56,7 +56,7 @@ t.step(function () {
             assert_type(addedMessages, "array", "Not an array.");
             assert_greater_than(addedMessages.length, 0, "Received incorrect number of messages");
 
-            for (var i in addedMessages) {
+            for (i = 0; i < addedMessages.length; i++) {
                 assert_true(addedMessages[i] instanceof tizen.Message, "Not a message.");
                 assert_equals(addedMessages[i].subject, message.subject, "Incorrect subject.");
 
index 286c3de451eaef3a20b10a5fce94dd5924b5125d..f859e0207f6ae25fe0e3cb70e50ddaeeba5a72e2 100644 (file)
@@ -40,7 +40,7 @@ Authors:
 //==== TEST_CRITERIA CBOA CBT
 setup({timeout: 180000});
 
-var t = async_test(document.title, {timeout: 180000}),
+var t = async_test(document.title, {timeout: 180000}), i,
     serviceSuccess, serviceError, service, watcherId, filter, matchMessage, mismatchMessage,
     changeCallback, addDraftError, addDraftSuccess, addDraftSuccessCounter = 0, removeError;
 
@@ -58,7 +58,7 @@ t.step(function () {
             assert_type(removedMessages, "array", "Not an array.");
             assert_greater_than(removedMessages.length, 0, "Received empty messages array");
 
-            for (var i in removedMessages) {
+            for (i = 0; i < removedMessages.length; i++) {
                 assert_true(removedMessages[i] instanceof tizen.Message, "Not a message.");
 
                 if (removedMessages[i].id === matchMessage.id) {
index 701edfc588d4a1a4ab6e4e467537738d0b96aeeb..0f370b4d5cb9fdee6589d522699a29311ed5b03b 100644 (file)
@@ -40,7 +40,7 @@ Authors:
 //==== TEST_CRITERIA CBOA CBT
 setup({timeout: 180000});
 var t = async_test(document.title, {timeout: 180000}), serviceSuccess, serviceError, messageService, messageStorage,
-    watcherId, message, changeCallback, addDraftSuccess, addDraftError, removeError;
+    watcherId, message, changeCallback, addDraftSuccess, addDraftError, removeError, i;
 t.step(function () {
 
     add_result_callback(function (res) {
@@ -56,7 +56,7 @@ t.step(function () {
             assert_type(removedMessages, "array", "Not an array.");
             assert_greater_than(removedMessages.length, 0, "Received incorrect number of messages");
 
-            for (var i in removedMessages) {
+            for (i = 0; i < removedMessages.length; i++) {
                 assert_true(removedMessages[i] instanceof tizen.Message, "Not a message.");
 
                 if (removedMessages[i].id === message.id) {