[folks] e-d-s: free resources properly between tests cases on link-persona test unit
- From: Raul Gutierrez Segales <raulgs src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [folks] e-d-s: free resources properly between tests cases on link-persona test unit
- Date: Wed, 7 Sep 2011 12:37:01 +0000 (UTC)
commit 3959555f24f7696bcda9af7e265682997c1076d0
Author: Raul Gutierrez Segales <rgs collabora co uk>
Date: Wed Sep 7 12:41:09 2011 +0100
e-d-s: free resources properly between tests cases on link-persona test unit
tests/eds/link-personas.vala | 21 ++++++++++++++-------
1 files changed, 14 insertions(+), 7 deletions(-)
---
diff --git a/tests/eds/link-personas.vala b/tests/eds/link-personas.vala
index 6dd1bb8..166d03c 100644
--- a/tests/eds/link-personas.vala
+++ b/tests/eds/link-personas.vala
@@ -33,8 +33,8 @@ enum LinkingMethod
public class LinkPersonasTests : Folks.TestCase
{
private GLib.MainLoop _main_loop;
- private EdsTest.Backend _eds_backend;
- private EdsTest.Backend _eds_backend_other;
+ private EdsTest.Backend? _eds_backend;
+ private EdsTest.Backend? _eds_backend_other;
private IndividualAggregator _aggregator;
private string _persona_fullname_1;
private string _persona_fullname_2;
@@ -48,7 +48,7 @@ public class LinkPersonasTests : Folks.TestCase
private HashSet<Persona> _personas;
private int _removed_individuals;
private string _folks_config_key = "/system/folks/backends/primary_store";
- private unowned GConf.Client _gconf_client;
+ private unowned GConf.Client? _gconf_client;
private Gee.HashMap<string, string> _linking_props;
private LinkingMethod _linking_method;
@@ -56,10 +56,6 @@ public class LinkPersonasTests : Folks.TestCase
{
base ("LinkPersonasTests");
- this._eds_backend = new EdsTest.Backend ();
- this._eds_backend_other = new EdsTest.Backend ();
- this._eds_backend_other.address_book_uri = "local://other";
-
this.add_test ("test linking personas via IM addresses",
this.test_linking_personas_via_im_addresses);
this.add_test ("test linking personas via local IDs",
@@ -72,6 +68,10 @@ public class LinkPersonasTests : Folks.TestCase
public override void set_up ()
{
+ this._eds_backend = new EdsTest.Backend ();
+ this._eds_backend_other = new EdsTest.Backend ();
+ this._eds_backend_other.address_book_uri = "local://other";
+
/* We configure eds as the primary (writeable) store */
this._gconf_client = GConf.Client.get_default ();
try
@@ -94,6 +94,9 @@ public class LinkPersonasTests : Folks.TestCase
this._eds_backend.tear_down ();
this._eds_backend_other.tear_down ();
+ this._eds_backend = null;
+ this._eds_backend_other = null;
+
try
{
this._gconf_client.unset (this._folks_config_key);
@@ -102,6 +105,10 @@ public class LinkPersonasTests : Folks.TestCase
{
warning ("Couldn't unset primary store: %s\n", e.message);
}
+ finally
+ {
+ this._gconf_client = null;
+ }
}
public void test_linking_personas_via_im_addresses ()
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]