[folks] Add test case for properties race condition.

[Travis Reitter <treitter src gnome org>]

commit 8e359ce63d151fbb52678e34c47541868b771b77
Author: Travis Reitter <travis reitter collabora co uk>
Date:   Wed Aug 24 10:50:54 2011 -0700

    Add test case for properties race condition.

 tests/eds/Makefile.am              |    6 ++
 tests/eds/set-properties-race.vala |  174 ++++++++++++++++++++++++++++++++++++
 2 files changed, 180 insertions(+), 0 deletions(-)
---
diff --git a/tests/eds/Makefile.am b/tests/eds/Makefile.am
index d5db601..07bc809 100644
--- a/tests/eds/Makefile.am
+++ b/tests/eds/Makefile.am
@@ -67,6 +67,7 @@ noinst_PROGRAMS = \
 	add-contacts-stress-test \
 	set-gender \
 	set-urls \
+	set-properties-race \
 	$(NULL)
 
 RUN_WITH_PRIVATE_BUS = $(top_srcdir)/tests/tools/with-session-bus-eds.sh
@@ -161,6 +162,10 @@ set_postal_addresses_SOURCES = \
 	set-postal-addresses.vala \
 	$(NULL)
 
+set_properties_race_SOURCES = \
+	set-properties-race.vala \
+	$(NULL)
+
 link_personas_SOURCES = \
 	link-personas.vala \
 	$(NULL)
@@ -208,6 +213,7 @@ MAINTAINERCLEANFILES = \
         set_structured_name_vala.stamp \
         set_phones_vala.stamp \
         set_postal_addresses_vala.stamp \
+        set_properties_race_vala.stamp \
         link_personas_vala.stamp \
         set_notes_vala.stamp \
 	add_contacts_stress_test_vala.stamp \
diff --git a/tests/eds/set-properties-race.vala b/tests/eds/set-properties-race.vala
new file mode 100644
index 0000000..0ea3698
--- /dev/null
+++ b/tests/eds/set-properties-race.vala
@@ -0,0 +1,174 @@
+/*
+ * Copyright (C) 2011 Collabora Ltd.
+ *
+ * This library is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 2.1 of the License, or
+ * (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this library.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authors: Raul Gutierrez Segales <raul gutierrez segales collabora co uk>
+ *
+ */
+
+using EdsTest;
+using Folks;
+using Gee;
+
+public class SetPropertiesRaceTests : Folks.TestCase
+{
+  private EdsTest.Backend _eds_backend;
+  private IndividualAggregator _aggregator;
+  private GLib.MainLoop _main_loop;
+  private bool _found_before_update;
+  private bool _found_after_update;
+  private PostalAddressFieldDetails _pa_fd;
+
+  public SetPropertiesRaceTests ()
+    {
+      base ("SetPropertiesRace");
+
+      this._eds_backend = new EdsTest.Backend ();
+
+      this.add_test ("setting postal addresss on e-d-s persona",
+          this.test_set_postal_addresses);
+    }
+
+  public override void set_up ()
+    {
+      this._eds_backend.set_up ();
+    }
+
+  public override void tear_down ()
+    {
+      this._eds_backend.tear_down ();
+    }
+
+  void test_set_postal_addresses ()
+    {
+      Gee.HashMap<string, Value?> c1 = new Gee.HashMap<string, Value?> ();
+      this._main_loop = new GLib.MainLoop (null, false);
+      Value? v;
+      var pa = new PostalAddress ("123", "extension", "street",
+          "locality", "region", "postal code", "country", "",
+          "123");
+      this._pa_fd = new PostalAddressFieldDetails (pa);
+      this._pa_fd.add_parameter ("type", "address_other");
+
+      this._found_before_update = false;
+      this._found_after_update = false;
+
+      this._eds_backend.reset ();
+
+      v = Value (typeof (string));
+      v.set_string ("bernie h. innocenti");
+      c1.set ("full_name", (owned) v);
+      this._eds_backend.add_contact (c1);
+
+      this._test_set_postal_addresses_async ();
+
+      Timeout.add_seconds (5, () => {
+            this._main_loop.quit ();
+            assert_not_reached ();
+          });
+
+      this._main_loop.run ();
+
+      assert (this._found_before_update);
+      assert (this._found_after_update);
+    }
+
+  private async void _test_set_postal_addresses_async ()
+    {
+      yield this._eds_backend.commit_contacts_to_addressbook ();
+
+      var store = BackendStore.dup ();
+      yield store.prepare ();
+      this._aggregator = new IndividualAggregator ();
+      this._aggregator.individuals_changed.connect
+          (this._individuals_changed_cb);
+      try
+        {
+          yield this._aggregator.prepare ();
+        }
+      catch (GLib.Error e)
+        {
+          GLib.warning ("Error when calling prepare: %s\n", e.message);
+        }
+    }
+
+  private void _individuals_changed_cb
+      (Set<Individual> added,
+       Set<Individual> removed,
+       string? message,
+       Persona? actor,
+       GroupDetails.ChangeReason reason)
+    {
+      foreach (Individual i in added)
+        {
+          var name = (Folks.NameDetails) i;
+
+          if (name.full_name == "bernie h. innocenti")
+            {
+              this._found_before_update = true;
+
+              foreach (var p in i.personas)
+                {
+                  p.notify["postal-addresses"].connect (
+                      this._notify_postal_addresses_cb);
+                  var pa_fds = new HashSet<PostalAddressFieldDetails> ();
+                  var pa_1 = new PostalAddress ("123", "extension", "street",
+                      "locality", "region", "postal code", "country", "format",
+                      "123");
+                  var pa_fd_1 = new PostalAddressFieldDetails (pa_1);
+                  pa_fd_1.add_parameter ("type", "address_other");
+                  pa_fds.add (pa_fd_1);
+                  ((PostalAddressDetails) p).postal_addresses = pa_fds;
+                }
+            }
+        }
+
+      assert (removed.size == 0);
+    }
+
+  private void _notify_postal_addresses_cb (Object persona_obj, ParamSpec ps)
+    {
+      var p = (Edsf.Persona) persona_obj;
+
+      /* ensure we aren't getting notified before the property set completes */
+      assert (p.postal_addresses.size == 1);
+
+      foreach (var pa_fd in p.postal_addresses)
+        {
+          /* make sure the content is the type expected */
+          assert (pa_fd is PostalAddressFieldDetails);
+          assert (pa_fd.value is PostalAddress);
+
+          pa_fd.value.uid = this._pa_fd.value.uid;
+          if (pa_fd.equal (this._pa_fd))
+            {
+              this._found_after_update = true;
+              this._main_loop.quit ();
+            }
+        }
+    }
+}
+
+public int main (string[] args)
+{
+  Test.init (ref args);
+
+  TestSuite root = TestSuite.get_root ();
+  root.add_suite (new SetPropertiesRaceTests ().get_suite ());
+
+  Test.run ();
+
+  return 0;
+}