diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2013-11-15 16:46:23 +0000 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2014-01-29 19:28:33 +0000 |
| commit | 28f6731e92a596eabea5c4975249d5927428f459 (patch) | |
| tree | 132a465a22fc08d705c50e04cf656d5c1c67c07d /tests/twisted | |
| parent | 183af896db254c19eedc62ad58d5e2b08297791f (diff) | |
delete_async: being asked to delete a nonexistent account is an error
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=27727
Diffstat (limited to 'tests/twisted')
| -rw-r--r-- | tests/twisted/dbus-account-plugin.c | 10 |
1 files changed, 2 insertions, 8 deletions
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 */ |