addresses-chunk: properly notify is-empty

We weren't sending out a property notification for `is-empty` when the
underlying `Folks.PostalAddress` changed, which meant that editing a
contact would only allows for a single address.

Also add a test to make sure we're not regressing on this.

1 files changed, 5 insertions, 0 deletions

diff --git a/tests/core/test-addresses-chunk.vala b/tests/core/test-addresses-chunk.vala
index 83ebb61..685c6bd 100644
--- a/tests/core/test-addresses-chunk.vala
+++ b/tests/core/test-addresses-chunk.vala
@@ -44,9 +44,14 @@ private void test_is_empty () {
   assert_true (address.is_empty);
   assert_true (chunk.is_empty);
 
+  // Make sure that the notify works correctly for the address too
+  bool notified = false;
+  address.notify["is-empty"].connect ((o, p) => { notified = true; });
+
   address.address.street = "Yellow brick road";
   assert_false (address.is_empty);
   assert_false (chunk.is_empty);
+  assert_true (notified);
 
   address.address.street = "";
   assert_true (address.is_empty);