diff options
| -rw-r--r-- | src/mcd-account-manager-default.c | 9 | ||||
| -rw-r--r-- | tests/twisted/dbus-account-plugin.c | 10 |
2 files changed, 4 insertions, 15 deletions
diff --git a/src/mcd-account-manager-default.c b/src/mcd-account-manager-default.c index 49f04079..fed283b1 100644 --- a/src/mcd-account-manager-default.c +++ b/src/mcd-account-manager-default.c @@ -361,13 +361,8 @@ delete_async (McpAccountStorage *self, task = g_task_new (amd, cancellable, callback, user_data); - if (sa == NULL || sa->absent) - { - /* Apparently we never had this account anyway. The plugin API - * considers this to be "success". */ - g_task_return_boolean (task, TRUE); - goto finally; - } + g_return_if_fail (sa != NULL); + g_return_if_fail (!sa->absent); filename = account_file_in (g_get_user_data_dir (), account); diff --git a/tests/twisted/dbus-account-plugin.c b/tests/twisted/dbus-account-plugin.c index 455095fa..c7c5dbde 100644 --- a/tests/twisted/dbus-account-plugin.c +++ b/tests/twisted/dbus-account-plugin.c @@ -916,14 +916,8 @@ test_dbus_account_plugin_delete_async (McpAccountStorage *storage, DEBUG ("called"); - if (account == NULL || !self->active) - { - /* We were asked to delete an account we don't have. It's - * a bit like success. */ - g_task_return_boolean (task, TRUE); - g_object_unref (task); - return; - } + g_return_if_fail (self->active); + g_return_if_fail (account != NULL); /* deletion used to be delayed, so the regression tests will expect this * to happen - leave them unmodified for now */ |