[dconf/wip/reorg: 2/5] tests/: test profile parsing

[Ryan Lortie <ryanl src gnome org>]

commit 9fca034e9fef29e17bdba9aa08a50d9fee28d55e
Author: Ryan Lortie <desrt desrt ca>
Date:   Sun Jul 8 10:40:50 2012 -0400

    tests/: test profile parsing

 tests/Makefile.am |    4 +-
 tests/engine.c    |  169 ++++++++++++++++++++++++++++++++++++++++++++++++++++-
 2 files changed, 170 insertions(+), 3 deletions(-)
---
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 7201319..3c51359 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -1,6 +1,6 @@
 include $(top_srcdir)/Makefile.gtester
 
-AM_CFLAGS = -std=c89 -Wall -Wmissing-prototypes -Wwrite-strings -DSRCDIR=\"$(abs_srcdir)\"
+AM_CFLAGS = -Wall -Wmissing-prototypes -Wwrite-strings -DSRCDIR=\"$(abs_srcdir)\"
 INCLUDES = -I$(top_srcdir)/common -I$(top_srcdir)/engine -I$(top_srcdir)/client $(gio_CFLAGS) -I$(top_srcdir)/dbus-1 $(dbus_CFLAGS) -I$(top_srcdir)/gvdb
 
 noinst_LIBRARIES = libdconf-dbus-stub.a
@@ -65,7 +65,7 @@ gdbus_filter_SOURCES = \
 	dbus.c
 
 TEST_PROGS += engine
-engine_LDADD = $(engine_with_stub_LIBS) $(glib_LIBS)
+engine_LDADD = $(engine_with_stub_LIBS) $(glib_LIBS) -ldl
 engine_SOURCES = engine.c
 
 TEST_PROGS += client
diff --git a/tests/engine.c b/tests/engine.c
index be57265..cb01cda 100644
--- a/tests/engine.c
+++ b/tests/engine.c
@@ -1,6 +1,33 @@
-#define _BSD_SOURCE
+#define _GNU_SOURCE
 
 #include <dconf-engine.h>
+#include <dconf-engine-profile.h>
+#include <stdlib.h>
+#include <stdio.h>
+#include <dlfcn.h>
+
+/* Interpose to catch fopen("/etc/dconf/profile/user") */
+static const gchar *filename_to_replace;
+static const gchar *filename_to_replace_it_with;
+
+FILE *
+fopen (const char *filename,
+       const char *mode)
+{
+  static FILE * (*real_fopen) (const char *, const char *);
+
+  if (!real_fopen)
+    real_fopen = dlsym (RTLD_NEXT, "fopen");
+
+  if (filename_to_replace && g_str_equal (filename, filename_to_replace))
+    {
+      /* Crash if this file was unexpectedly opened */
+      g_assert (filename_to_replace_it_with != NULL);
+      filename = filename_to_replace_it_with;
+    }
+
+  return (* real_fopen) (filename, mode);
+}
 
 static GThread *main_thread;
 
@@ -14,6 +41,143 @@ dconf_engine_change_notify (DConfEngine         *engine,
   /* ignore */
 }
 
+static void
+verify_and_free (DConfEngineSource  **sources,
+                 gint                 n_sources,
+                 const gchar * const *expected_names,
+                 gint                 n_expected)
+{
+  gint i;
+
+  g_assert_cmpint (n_sources, ==, n_expected);
+
+  g_assert ((sources == NULL) == (n_sources == 0));
+
+  for (i = 0; i < n_sources; i++)
+    {
+      g_assert_cmpstr (sources[i]->name, ==, expected_names[i]);
+      dconf_engine_source_free (sources[i]);
+    }
+
+  g_free (sources);
+}
+
+static void
+test_five_times (const gchar *filename,
+                 gint         n_expected,
+                 ...)
+{
+  const gchar **expected_names;
+  DConfEngineSource **sources;
+  gint n_sources;
+  va_list ap;
+  gint i;
+
+  expected_names = g_new (const gchar *, n_expected);
+  va_start (ap, n_expected);
+  for (i = 0; i < n_expected; i++)
+    expected_names[i] = va_arg (ap, const gchar *);
+  va_end (ap);
+
+  /* first try by supplying the profile filename via the API */
+  g_assert (g_getenv ("DCONF_PROFILE") == NULL);
+  g_assert (filename_to_replace == NULL);
+  sources = dconf_engine_profile_open (filename, &n_sources);
+  verify_and_free (sources, n_sources, expected_names, n_expected);
+
+  /* next try supplying it via the environment */
+  g_setenv ("DCONF_PROFILE", filename, TRUE);
+  g_assert (filename_to_replace == NULL);
+  sources = dconf_engine_profile_open (NULL, &n_sources);
+  verify_and_free (sources, n_sources, expected_names, n_expected);
+  g_unsetenv ("DCONF_PROFILE");
+
+  /* next try supplying a profile name via API and intercepting fopen */
+  filename_to_replace = "/etc/dconf/profile/myprofile";
+  filename_to_replace_it_with = filename;
+  g_assert (g_getenv ("DCONF_PROFILE") == NULL);
+  sources = dconf_engine_profile_open ("myprofile", &n_sources);
+  verify_and_free (sources, n_sources, expected_names, n_expected);
+  filename_to_replace = NULL;
+
+  /* next try the same, via the environment */
+  g_setenv ("DCONF_PROFILE", "myprofile", TRUE);
+  filename_to_replace = "/etc/dconf/profile/myprofile";
+  filename_to_replace_it_with = filename;
+  sources = dconf_engine_profile_open (NULL, &n_sources);
+  verify_and_free (sources, n_sources, expected_names, n_expected);
+  g_unsetenv ("DCONF_PROFILE");
+  filename_to_replace = NULL;
+
+  /* next try to have dconf pick it up as the default user profile */
+  filename_to_replace = "/etc/dconf/profile/user";
+  filename_to_replace_it_with = filename;
+  g_assert (g_getenv ("DCONF_PROFILE") == NULL);
+  sources = dconf_engine_profile_open (NULL, &n_sources);
+  verify_and_free (sources, n_sources, expected_names, n_expected);
+  filename_to_replace = NULL;
+
+  filename_to_replace_it_with = NULL;
+  g_free (expected_names);
+}
+
+static void
+test_profile_parser (void)
+{
+  DConfEngineSource **sources;
+  gint n_sources;
+
+  if (g_test_trap_fork (0, G_TEST_TRAP_SILENCE_STDERR))
+    {
+      g_log_set_always_fatal (G_LOG_LEVEL_ERROR);
+
+      sources = dconf_engine_profile_open (SRCDIR "/profile/this-file-does-not-exist", &n_sources);
+      g_assert_cmpint (n_sources, ==, 0);
+      g_assert (sources == NULL);
+      exit (0);
+    }
+  g_test_trap_assert_passed ();
+  g_test_trap_assert_stderr ("*WARNING*: unable to open named profile*");
+
+  if (g_test_trap_fork (0, G_TEST_TRAP_SILENCE_STDERR))
+    {
+      g_log_set_always_fatal (G_LOG_LEVEL_ERROR);
+
+      sources = dconf_engine_profile_open (SRCDIR "/profile/broken-profile", &n_sources);
+      g_assert_cmpint (n_sources, ==, 0);
+      g_assert (sources == NULL);
+      exit (0);
+    }
+  g_test_trap_assert_passed ();
+  g_test_trap_assert_stderr ("*WARNING*: unknown dconf database*unknown dconf database*");
+
+  test_five_times (SRCDIR "/profile/empty-profile", 0);
+  test_five_times (SRCDIR "/profile/test-profile", 1, "test");
+  test_five_times (SRCDIR "/profile/colourful", 4,
+                   "user",
+                   "other",
+                   "verylongnameverylongnameverylongnameverylongnameverylongnameverylongnameverylongnameverylongnameverylongnameverylongnameverylongnameverylongname",
+                   "nonewline");
+  test_five_times (SRCDIR "/profile/dos", 2, "user", "site");
+  test_five_times (SRCDIR "/profile/no-newline-longline", 0);
+  test_five_times (SRCDIR "/profile/many-sources", 10,
+                   "user", "local", "room", "floor", "building",
+                   "site", "region", "division", "country", "global");
+
+  /* finally, test that we get the default profile if the user profile
+   * file cannot be located and we do not specify another profile.
+   */
+  filename_to_replace = "/etc/dconf/profile/user";
+  filename_to_replace_it_with = SRCDIR "/profile/this-file-does-not-exist";
+  g_assert (g_getenv ("DCONF_PROFILE") == NULL);
+  sources = dconf_engine_profile_open (NULL, &n_sources);
+  filename_to_replace = NULL;
+  g_assert_cmpint (n_sources, ==, 1);
+  g_assert_cmpstr (sources[0]->name, ==, "user");
+  dconf_engine_source_free (sources[0]);
+  g_free (sources);
+}
+
 static gpointer
 test_signal_threadsafety_worker (gpointer user_data)
 {
@@ -58,10 +222,13 @@ test_signal_threadsafety (void)
 int
 main (int argc, char **argv)
 {
+  g_unsetenv ("DCONF_PROFILE");
+
   main_thread = g_thread_self ();
 
   g_test_init (&argc, &argv, NULL);
 
+  g_test_add_func ("/engine/profile-parser", test_profile_parser);
   g_test_add_func ("/engine/signal-threadsafety", test_signal_threadsafety);
 
   return g_test_run ();