From 2d0162c97b02b26336ef39a708cf02877ba16573 Mon Sep 17 00:00:00 2001
From: Benjamin Otte <otte@redhat.com>
Date: Fri, 8 Apr 2011 01:32:13 +0200
Subject: [PATCH] tests: Check errors in css test

---
 tests/css/parser/Makefile.am                 |  2 +
 tests/css/parser/close-at-end-of-file.errors |  1 +
 tests/css/parser/does-not-exist.errors       |  1 +
 tests/css/parser/test-css-parser.c           | 79 +++++++++++++++++++-
 4 files changed, 80 insertions(+), 3 deletions(-)
 create mode 100644 tests/css/parser/close-at-end-of-file.errors
 create mode 100644 tests/css/parser/does-not-exist.errors

diff --git a/tests/css/parser/Makefile.am b/tests/css/parser/Makefile.am
index 36354f9ec..b606dcd20 100644
--- a/tests/css/parser/Makefile.am
+++ b/tests/css/parser/Makefile.am
@@ -26,8 +26,10 @@ clean-local:
 
 EXTRA_DIST += \
 	close-at-end-of-file.css \
+	close-at-end-of-file.errors \
 	close-at-end-of-file.ref.css \
 	does-not-exist.css \
+	does-not-exist.errors \
 	does-not-exist.ref.css \
 	no-semicolon.css \
 	no-semicolon.ref.css \
diff --git a/tests/css/parser/close-at-end-of-file.errors b/tests/css/parser/close-at-end-of-file.errors
new file mode 100644
index 000000000..341b007cf
--- /dev/null
+++ b/tests/css/parser/close-at-end-of-file.errors
@@ -0,0 +1 @@
+close-at-end-of-file.css:3: error: gtk-css-provider-error-quark 1
diff --git a/tests/css/parser/does-not-exist.errors b/tests/css/parser/does-not-exist.errors
new file mode 100644
index 000000000..eb5799827
--- /dev/null
+++ b/tests/css/parser/does-not-exist.errors
@@ -0,0 +1 @@
+does-not-exist.css:2: error: gtk-css-provider-error-quark 2
diff --git a/tests/css/parser/test-css-parser.c b/tests/css/parser/test-css-parser.c
index 8c67e3b9d..b947bdf9c 100644
--- a/tests/css/parser/test-css-parser.c
+++ b/tests/css/parser/test-css-parser.c
@@ -47,6 +47,27 @@ test_get_reference_file (const char *css_file)
   return g_string_free (file, FALSE);
 }
 
+static char *
+test_get_errors_file (const char *css_file)
+{
+  GString *file = g_string_new (NULL);
+
+  if (g_str_has_suffix (css_file, ".css"))
+    g_string_append_len (file, css_file, strlen (css_file) - 4);
+  else
+    g_string_append (file, css_file);
+  
+  g_string_append (file, ".errors");
+
+  if (!g_file_test (file->str, G_FILE_TEST_EXISTS))
+    {
+      g_string_free (file, TRUE);
+      return NULL;
+    }
+
+  return g_string_free (file, FALSE);
+}
+
 static char *
 diff_with_file (const char  *file1,
                 char        *text,
@@ -95,17 +116,47 @@ done:
   return diff;
 }
 
+static void
+parsing_error_cb (GtkCssProvider *provider,
+                  const gchar     *path,
+                  guint            line,
+                  guint            position,
+                  const GError *   error,
+                  GString *        errors)
+{
+  char *basename;
+
+  g_assert (path);
+  g_assert (line > 0);
+
+  basename = g_path_get_basename (path);
+
+  g_string_append_printf (errors,
+                          "%s:%u: error: %s %u\n",
+                          basename, line,
+                          g_quark_to_string (error->domain),
+                          error->code);
+
+  g_free (basename);
+}
+
 static void
 test_css_file (GFile *file)
 {
   GtkCssProvider *provider;
   char *css, *diff;
-  char *css_file, *reference_file;
+  char *css_file, *reference_file, *errors_file;
+  GString *errors;
   GError *error = NULL;
 
   css_file = g_file_get_path (file);
+  errors = g_string_new ("");
 
   provider = gtk_css_provider_new ();
+  g_signal_connect (provider, 
+                    "parsing-error",
+                    G_CALLBACK (parsing_error_cb),
+                    errors);
   gtk_css_provider_load_from_path (provider,
                                    css_file,
                                    &error);
@@ -120,15 +171,37 @@ test_css_file (GFile *file)
   diff = diff_with_file (reference_file, css, -1, &error);
   g_assert_no_error (error);
 
+  if (diff && diff[0])
+    {
+      g_test_message ("%s", diff);
+      g_assert_not_reached ();
+    }
+
   g_free (css);
   g_free (reference_file);
 
-  if (diff && diff[0])
+  errors_file = test_get_errors_file (css_file);
+
+  if (errors_file)
     {
-      g_test_message ("%s", diff);
+      diff = diff_with_file (errors_file, errors->str, errors->len, &error);
+      g_assert_no_error (error);
+
+      if (diff && diff[0])
+        {
+          g_test_message ("%s", diff);
+          g_assert_not_reached ();
+        }
+    }
+  else if (errors->str[0])
+    {
+      g_test_message ("Unexpected errors:\n%s", errors->str);
       g_assert_not_reached ();
     }
 
+  g_free (errors_file);
+  g_string_free (errors, TRUE);
+
   g_free (diff);
   g_free (css_file);
 }
-- 
2.43.2