[gtk/wip/otte/listview: 152/152] xxx: expressioon tests

[Benjamin Otte <otte src gnome org>]

commit 357bf1d262a5aa1d38335239ebb05cab92c7d259
Author: Benjamin Otte <otte redhat com>
Date:   Sun Dec 1 22:25:16 2019 +0100

    xxx: expressioon tests

 testsuite/gtk/expression.c | 119 +++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 110 insertions(+), 9 deletions(-)
---
diff --git a/testsuite/gtk/expression.c b/testsuite/gtk/expression.c
index 94803badf9..697ee9b947 100644
--- a/testsuite/gtk/expression.c
+++ b/testsuite/gtk/expression.c
@@ -48,13 +48,17 @@ test_property (void)
 
   gtk_string_filter_set_search (filter, "Hello World");
   g_assert_cmpint (counter, ==, 1);
+  counter = 0;
+
   g_assert (gtk_expression_evaluate (expr, filter , &value));
   g_assert_cmpstr (g_value_get_string (&value), ==, "Hello World");
   g_value_unset (&value);
 
+  gtk_expression_watch_unwatch (watch);
+  g_assert_cmpint (counter, ==, 0);
+
   gtk_expression_unref (expr);
   g_object_unref (filter);
-  gtk_expression_watch_unwatch (watch);
 }
 
 static char *
@@ -116,9 +120,11 @@ test_closure (void)
   g_assert_cmpstr (g_value_get_string (&value), ==, "OK");
   g_value_unset (&value);
 
+  gtk_expression_watch_unwatch (watch);
+  g_assert_cmpint (counter, ==, 5);
+
   gtk_expression_unref (expr);
   g_object_unref (filter);
-  gtk_expression_watch_unwatch (watch);
 }
 
 static void
@@ -211,9 +217,9 @@ test_nested (void)
   g_assert_cmpstr (g_value_get_string (&value), ==, "word");
   g_value_unset (&value);
 
-  watch = gtk_expression_watch (expr, filter, inc_counter, &counter, NULL);
+  watch = gtk_expression_watch (expr, NULL, inc_counter, &counter, NULL);
   gtk_string_filter_set_search (GTK_STRING_FILTER (filter), "salad");
-  g_assert_cmpint (counter, >, 0);
+  g_assert_cmpint (counter, ==, 1);
   counter = 0;
 
   res = gtk_expression_evaluate (expr, NULL, &value);
@@ -228,7 +234,7 @@ test_nested (void)
   filter = gtk_string_filter_new ();
   gtk_string_filter_set_search (GTK_STRING_FILTER (filter), "salad");
   gtk_filter_list_model_set_filter (filtered, filter);
-  g_assert_cmpint (counter, >, 0);
+  g_assert_cmpint (counter, ==, 1);
   counter = 0;
 
   res = gtk_expression_evaluate (expr, NULL, &value);
@@ -237,7 +243,7 @@ test_nested (void)
   g_value_unset (&value);
 
   gtk_string_filter_set_search (GTK_STRING_FILTER (filter), "bar");
-  g_assert_cmpint (counter, >, 0);
+  g_assert_cmpint (counter, ==, 1);
   counter = 0;
 
   res = gtk_expression_evaluate (expr, NULL, &value);
@@ -246,17 +252,90 @@ test_nested (void)
   g_value_unset (&value);
 
   gtk_filter_list_model_set_filter (filtered, NULL);
-  g_assert_cmpint (counter, >, 0);
+  g_assert_cmpint (counter, ==, 1);
+  counter = 0;
 
   res = gtk_expression_evaluate (expr, NULL, &value);
   g_assert_false (res);
 
   gtk_expression_watch_unwatch (watch);
+  g_assert_cmpint (counter, ==, 0);
+
+  g_object_unref (filtered);
+  g_object_unref (list);
+  g_object_unref (filter);
+  gtk_expression_unref (expr);
+}
+
+/* This test uses the same setup as the last test, but
+ * passes the filter as the "this" object when creating
+ * the watch.
+ *
+ * So when we set a new filter and the old one gets desroyed,
+ * the watch should invalidate itself because its this object
+ * is gone.
+ */
+static void
+test_nested_this_destroyed (void)
+{
+  GtkExpression *list_expr;
+  GtkExpression *filter_expr;
+  GtkExpression *expr;
+  GtkFilter *filter;
+  GListModel *list;
+  GtkFilterListModel *filtered;
+  GValue value = G_VALUE_INIT;
+  gboolean res;
+  GtkExpressionWatch *watch;
+  guint counter = 0;
+
+  filter = gtk_string_filter_new ();
+  gtk_string_filter_set_search (GTK_STRING_FILTER (filter), "word");
+  list = G_LIST_MODEL (g_list_store_new (G_TYPE_OBJECT));
+  filtered = gtk_filter_list_model_new (list, filter);
+
+  list_expr = gtk_object_expression_new (G_OBJECT (filtered));
+  filter_expr = gtk_property_expression_new (GTK_TYPE_FILTER_LIST_MODEL, list_expr, "filter");
+  expr = gtk_property_expression_new (GTK_TYPE_STRING_FILTER, filter_expr, "search");
+
+  watch = gtk_expression_watch (expr, filter, inc_counter, &counter, NULL);
+  gtk_expression_watch_ref (watch);
+  res = gtk_expression_watch_evaluate (watch, &value);
+  g_assert_true (res);
+  g_assert_cmpstr (g_value_get_string (&value), ==, "word");
+  g_value_unset (&value);
+
+  g_clear_object (&filter);
+  g_assert_cmpint (counter, ==, 0);
+
+  filter = gtk_string_filter_new ();
+  gtk_string_filter_set_search (GTK_STRING_FILTER (filter), "salad");
+  gtk_filter_list_model_set_filter (filtered, filter);
+  g_assert_cmpint (counter, ==, 1);
+  counter = 0;
+
+  res = gtk_expression_watch_evaluate (watch, &value);
+  g_assert_false (res);
+
+  gtk_string_filter_set_search (GTK_STRING_FILTER (filter), "bar");
+  g_assert_cmpint (counter, ==, 0);
+
+  gtk_filter_list_model_set_filter (filtered, NULL);
+  g_assert_cmpint (counter, ==, 0);
+
+  res = gtk_expression_watch_evaluate (watch, &value);
+  g_assert_false (res);
+  g_assert_false (G_IS_VALUE (&value));
+
+  /* We unwatch on purpose here to make sure it doesn't do bad things. */
+  gtk_expression_watch_unwatch (watch);
+  gtk_expression_watch_unref (watch);
+  g_assert_cmpint (counter, ==, 0);
 
   g_object_unref (filtered);
   g_object_unref (list);
   g_object_unref (filter);
-  gtk_expression_unre (expr);
+  gtk_expression_unref (expr);
 }
 
 /* Test that property expressions fail to evaluate if the
@@ -310,11 +389,31 @@ test_this (void)
   g_assert_cmpstr (g_value_get_string (&value), ==, "sausage");
   g_value_unset (&value);
 
-  g_object_unref (expr);
+  gtk_expression_unref (expr);
   g_object_unref (filter2);
   g_object_unref (filter);
 }
 
+/* Check that even for static expressions, watches can be created
+ * and destroying the "this" argument does invalidate the
+ * expression.
+ */
+static void
+test_constant_watch_this_destroyed (void)
+{
+  GtkExpression *expr;
+  GObject *this;
+  guint counter = 0;
+
+  this = g_object_new (G_TYPE_OBJECT, NULL);
+  expr = gtk_constant_expression_new (G_TYPE_INT, 42);
+  gtk_expression_watch (expr, this, inc_counter, &counter, NULL);
+  g_assert_cmpint (counter, ==, 0);
+
+  g_clear_object (&this);
+  g_assert_cmpint (counter, ==, 1);
+}
+
 int
 main (int argc, char *argv[])
 {
@@ -324,8 +423,10 @@ main (int argc, char *argv[])
   g_test_add_func ("/expression/property", test_property);
   g_test_add_func ("/expression/closure", test_closure);
   g_test_add_func ("/expression/constant", test_constant);
+  g_test_add_func ("/expression/constant-watch-this-destroyed", test_constant_watch_this_destroyed);
   g_test_add_func ("/expression/object", test_object);
   g_test_add_func ("/expression/nested", test_nested);
+  g_test_add_func ("/expression/nested-this-destroyed", test_nested_this_destroyed);
   g_test_add_func ("/expression/type-mismatch", test_type_mismatch);
   g_test_add_func ("/expression/this", test_this);