X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=tests%2Fcheck_account-mgr.c;h=6c006672277a857142aa09dd6b70b1bb0c59ca78;hp=c745accd8cb8e938b4656bed52e79350e56070b0;hb=250e23c3f8c9b93a86641c4c8acecc1adb817e02;hpb=47b6bed5e41055dcffdbfdb9c989b49bf6500293 diff --git a/tests/check_account-mgr.c b/tests/check_account-mgr.c index c745acc..6c00667 100644 --- a/tests/check_account-mgr.c +++ b/tests/check_account-mgr.c @@ -31,6 +31,7 @@ #include #include #include +#include /* ----------------------- Defines ---------------------- */ @@ -45,17 +46,31 @@ static ModestAccountMgr *account_mgr = NULL; static void fx_setup_default_account_mgr () { - ModestConf *conf = NULL; - - g_type_init (); - - conf = modest_conf_new (); - fail_unless (MODEST_IS_CONF (conf), - "modest_conf_new failed"); - - account_mgr = modest_account_mgr_new (conf); - fail_unless (MODEST_IS_ACCOUNT_MGR (account_mgr), - "modest_account_mgr_new failed"); + ModestConf *conf = NULL; + + g_type_init (); + + conf = modest_conf_new (); + fail_unless (MODEST_IS_CONF (conf), + "modest_conf_new failed"); + + account_mgr = modest_account_mgr_new (conf); + fail_unless (MODEST_IS_ACCOUNT_MGR (account_mgr), + "modest_account_mgr_new failed"); + + /* cleanup old garbage (from previous runs)*/ + if (modest_account_mgr_account_exists(account_mgr, + TEST_MODEST_ACCOUNT_NAME, + FALSE)) + modest_account_mgr_remove_account (account_mgr, + TEST_MODEST_ACCOUNT_NAME, + FALSE); + if (modest_account_mgr_account_exists(account_mgr, + TEST_MODEST_ACCOUNT_NAME, + TRUE)) + modest_account_mgr_remove_account (account_mgr, + TEST_MODEST_ACCOUNT_NAME, + TRUE); } static void @@ -89,25 +104,21 @@ START_TEST (test_add_exists_remove_account_regular) gchar *hostname = NULL; gchar *username = NULL; gchar *password = NULL; - gchar *proto = NULL; - GError *error = NULL; + ModestTransportStoreProtocol proto; gboolean result; - + name = g_strdup (TEST_MODEST_ACCOUNT_NAME); - /* Test 1 */ store_account = g_strdup ("imap://me@myserver"); transport_account = g_strdup ("local-smtp"); result = modest_account_mgr_add_account (account_mgr, name, store_account, - transport_account, - &error); - fail_unless (result && !error, + transport_account, TRUE); + fail_unless (result, "modest_account_mgr_add_account failed:\n" \ - "name: %s\nstore: %s\ntransport: %s\nerror: %s", - name, store_account, transport_account, - error ? error->message : ""); + "name: %s\nstore: %s\ntransport: %s\n", + name, store_account, transport_account, TRUE); g_free (store_account); g_free (transport_account); @@ -115,34 +126,34 @@ START_TEST (test_add_exists_remove_account_regular) /* Test 2 */ result = modest_account_mgr_account_exists (account_mgr, name, - FALSE, - &error); - fail_unless (result && !error, + FALSE); + fail_unless (result, "modest_account_mgr_account_exists failed: " \ - "Account with name \"%s\" should exist. Error: %s", - name, error ? error->message : ""); + "Account with name \"%s\" should exist.\n", name); /* Test 3 */ result = modest_account_mgr_remove_account (account_mgr, name, - FALSE, - &error); - fail_unless (result && !error, + FALSE); + fail_unless (result, "modest_account_mgr_remove_account failed:\nname: %s\nerror: %s", - name, error ? error->message : ""); + name); + /* Test 4 */ hostname = g_strdup ("myhostname.mydomain.com"); username = g_strdup ("myusername"); password = g_strdup ("mypassword"); - proto = g_strdup ("smtp"); + proto = MODEST_PROTOCOL_TRANSPORT_SMTP; result = modest_account_mgr_add_server_account (account_mgr, name, hostname, username, password, - proto); + proto, + MODEST_PROTOCOL_SECURITY_NONE, + MODEST_PROTOCOL_AUTH_NONE); fail_unless (result, "modest_account_mgr_add_server_account failed:\n" \ "name: %s\nhostname: %s\nusername: %s\npassword: %s\nproto: %s", @@ -151,47 +162,38 @@ START_TEST (test_add_exists_remove_account_regular) g_free (hostname); g_free (username); g_free (password); - g_free (proto); - + /* Test 5 */ - result = modest_account_mgr_account_exists (account_mgr, - name, - TRUE, - &error); - fail_unless (result && !error, + result = modest_account_mgr_account_exists (account_mgr,name,TRUE); + fail_unless (result, "modest_account_mgr_account_exists failed: " \ - "Server account with name \"%s\" should exist. Error: %s", - name, error ? error->message : ""); - + "Server account with name \"%s\" should exist. Error: %s", name); /* Test 6 */ result = modest_account_mgr_remove_account (account_mgr, name, - TRUE, - &error); - fail_unless (result && !error, + TRUE); + fail_unless (result, "modest_account_mgr_remove_account failed:\nname: %s\nerror: %s", - name, error ? error->message : ""); + name); /* Test 7 */ result = modest_account_mgr_account_exists (account_mgr, "a_name_that_does_not_exist", - FALSE, - NULL); - fail_unless (!result, + FALSE); + fail_unless (result, "modest_account_mgr_exists_account does not return " \ "FALSE when passing an account that does not exist"); /* Test 8 */ result = modest_account_mgr_account_exists (account_mgr, "a_name_that_does_not_exist", - TRUE, - NULL); - fail_unless (!result, + TRUE); + fail_unless (result, "modest_account_mgr_exists_account does not return " \ "FALSE when passing a server account that does not exist"); - + g_free (name); } END_TEST @@ -225,8 +227,7 @@ START_TEST (test_add_exists_remove_account_invalid) result = modest_account_mgr_add_account (NULL, TEST_MODEST_ACCOUNT_NAME, "store_account", - "transport_account", - NULL); + "transport_account", TRUE); fail_unless (!result, "modest_account_mgr_add_account does not return FALSE when" \ "passing a NULL ModestAccountMgr"); @@ -235,8 +236,7 @@ START_TEST (test_add_exists_remove_account_invalid) result = modest_account_mgr_add_account (account_mgr, NULL, "store_account", - "transport_account", - NULL); + "transport_account", TRUE); fail_unless (!result, "modest_account_mgr_add_account does not return FALSE when" \ "passing a NULL account name"); @@ -245,8 +245,7 @@ START_TEST (test_add_exists_remove_account_invalid) result = modest_account_mgr_add_account (account_mgr, "ïnválid//accountñ//nÄméç", "store_account", - "transport_account", - NULL); + "transport_account", TRUE); fail_unless (!result, "modest_account_mgr_add_account does not return FALSE when" \ "passing an invalid account name"); @@ -257,7 +256,9 @@ START_TEST (test_add_exists_remove_account_invalid) "hostname", "username", "password", - "proto"); + MODEST_PROTOCOL_STORE_IMAP, + MODEST_PROTOCOL_SECURITY_NONE, + MODEST_PROTOCOL_AUTH_NONE); fail_unless (!result, "modest_account_mgr_add_server_account does not return " \ "FALSE when passing a NULL ModestAccountMgr"); @@ -268,7 +269,9 @@ START_TEST (test_add_exists_remove_account_invalid) "hostname", "username", "password", - "proto"); + MODEST_PROTOCOL_STORE_IMAP, + MODEST_PROTOCOL_SECURITY_NONE, + MODEST_PROTOCOL_AUTH_NONE); fail_unless (!result, "modest_account_mgr_add_server_account does not return " \ "FALSE when passing a NULL account name"); @@ -279,7 +282,9 @@ START_TEST (test_add_exists_remove_account_invalid) "hostname", "username", "password", - "proto"); + MODEST_PROTOCOL_STORE_IMAP, + MODEST_PROTOCOL_SECURITY_NONE, + MODEST_PROTOCOL_AUTH_NONE); fail_unless (!result, "modest_account_mgr_add_server_account does not return " \ "FALSE when passing an invalid account name"); @@ -287,8 +292,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 7 */ result = modest_account_mgr_remove_account (account_mgr, "a_name_that_does_not_exist", - FALSE, - NULL); + FALSE); fail_unless (!result, "modest_account_mgr_remove_acccount does not return FALSE " \ "when trying to remove an account that does not exist"); @@ -296,8 +300,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 8 */ result = modest_account_mgr_remove_account (account_mgr, "a_name_that_does_not_exist", - TRUE, - NULL); + TRUE); fail_unless (!result, "modest_account_mgr_remove_acccount does not return FALSE " \ "when trying to remove a server account that does not exist"); @@ -305,8 +308,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 9 */ result = modest_account_mgr_remove_account (NULL, TEST_MODEST_ACCOUNT_NAME, - FALSE, - NULL); + FALSE); fail_unless (!result, "modest_account_mgr_remove_acccount does not return " \ "FALSE when passing a NULL ModestAccountMgr"); @@ -314,8 +316,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 10 */ result = modest_account_mgr_remove_account (account_mgr, NULL, - FALSE, - NULL); + FALSE); fail_unless (!result, "modest_account_mgr_remove_acccount does not return " \ "FALSE when passing a NULL account name"); @@ -323,8 +324,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 11 */ result = modest_account_mgr_account_exists (NULL, TEST_MODEST_ACCOUNT_NAME, - TRUE, - NULL); + TRUE); fail_unless (!result, "modest_account_mgr_exists_account does not return " \ "FALSE when passing a NULL ModestAccountMgr"); @@ -332,8 +332,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 12 */ result = modest_account_mgr_account_exists (NULL, TEST_MODEST_ACCOUNT_NAME, - FALSE, - NULL); + FALSE); fail_unless (!result, "modest_account_mgr_exists_account does not return " \ "FALSE when passing a NULL ModestAccountMgr"); @@ -341,8 +340,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 13 */ result = modest_account_mgr_account_exists (account_mgr, NULL, - FALSE, - NULL); + FALSE); fail_unless (!result, "modest_account_mgr_exists_acccount does not return " \ "FALSE when passing a NULL account name"); @@ -350,8 +348,7 @@ START_TEST (test_add_exists_remove_account_invalid) /* Test 14 */ result = modest_account_mgr_account_exists (account_mgr, NULL, - TRUE, - NULL); + TRUE); fail_unless (!result, "modest_account_mgr_exists_account does not return " \ "FALSE when passing a NULL server account name");