[evolution/gnome-3-24] Don't hardcode base64 image data in tests
- From: Tomas Popela <tpopela src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution/gnome-3-24] Don't hardcode base64 image data in tests
- Date: Fri, 31 Mar 2017 08:27:20 +0000 (UTC)
commit 0a450d211689e50e68f5f531baeb95d636d43ac4
Author: Tomas Popela <tpopela redhat com>
Date: Thu Mar 30 17:10:40 2017 +0200
Don't hardcode base64 image data in tests
This will avoid test breakage when the images are updated as now when in
GNOME 3.24 the emoticons were updated.
src/e-util/test-html-editor-units-utils.c | 20 ++++++
src/e-util/test-html-editor-units-utils.h | 2 +
src/e-util/test-html-editor-units.c | 92 +++++++++-------------------
3 files changed, 52 insertions(+), 62 deletions(-)
---
diff --git a/src/e-util/test-html-editor-units-utils.c b/src/e-util/test-html-editor-units-utils.c
index 8b130f8..df4cd6a 100644
--- a/src/e-util/test-html-editor-units-utils.c
+++ b/src/e-util/test-html-editor-units-utils.c
@@ -1072,3 +1072,23 @@ test_utils_get_content_editor (TestFixture *fixture)
cnt_editor = e_html_editor_get_content_editor (fixture->editor);
return cnt_editor;
}
+
+gchar *
+test_utils_get_base64_data_for_image (const gchar *path)
+{
+ gchar *image_data;
+ gchar *image_data_base64;
+ gsize image_data_length;
+ gboolean success;
+ GError *error = NULL;
+
+ success = g_file_get_contents (path, &image_data, &image_data_length, &error);
+ g_assert_no_error (error);
+ g_assert (success);
+
+ image_data_base64 = g_base64_encode ((const guchar *) image_data, image_data_length);
+
+ g_free (image_data);
+
+ return image_data_base64;
+}
diff --git a/src/e-util/test-html-editor-units-utils.h b/src/e-util/test-html-editor-units-utils.h
index 02343c5..7867346 100644
--- a/src/e-util/test-html-editor-units-utils.h
+++ b/src/e-util/test-html-editor-units-utils.h
@@ -100,6 +100,8 @@ void test_utils_set_clipboard_text (const gchar *text,
gchar * test_utils_get_clipboard_text (gboolean request_html);
EContentEditor *
test_utils_get_content_editor (TestFixture *fixture);
+gchar * test_utils_get_base64_data_for_image
+ (const gchar *path);
G_END_DECLS
diff --git a/src/e-util/test-html-editor-units.c b/src/e-util/test-html-editor-units.c
index fc24a94..f607850 100644
--- a/src/e-util/test-html-editor-units.c
+++ b/src/e-util/test-html-editor-units.c
@@ -949,11 +949,8 @@ test_image_insert (TestFixture *fixture)
EContentEditor *cnt_editor;
gchar *expected_html;
gchar *filename;
- gchar *uri;
- gchar *image_data;
gchar *image_data_base64;
- gsize image_data_length;
- gboolean success;
+ gchar *uri;
GError *error = NULL;
if (!test_utils_process_commands (fixture,
@@ -967,28 +964,23 @@ test_image_insert (TestFixture *fixture)
uri = g_filename_to_uri (filename, NULL, &error);
g_assert_no_error (error);
- success = g_file_get_contents (filename, &image_data, &image_data_length, &error);
- g_assert_no_error (error);
- g_assert (success);
-
- g_free (filename);
-
/* Mimic what the action:insert-image does, without invoking the image chooser dialog */
cnt_editor = e_html_editor_get_content_editor (fixture->editor);
e_content_editor_insert_image (cnt_editor, uri);
/* Wait some time until the operation is finished */
test_utils_wait_milliseconds (500);
+ image_data_base64 = test_utils_get_base64_data_for_image (filename);
+
g_free (uri);
+ g_free (filename);
- image_data_base64 = g_base64_encode ((const guchar *) image_data, image_data_length);
g_return_if_fail (image_data_base64 != NULL);
expected_html = g_strconcat (HTML_PREFIX "<div>before*<img src=\"data:image/png;base64,",
image_data_base64, "\">+after</div>" HTML_SUFFIX, NULL);
g_free (image_data_base64);
- g_free (image_data);
if (!test_utils_run_simple_test (fixture,
"undo:save\n" /* 1 */
@@ -1006,77 +998,53 @@ test_image_insert (TestFixture *fixture)
static void
test_emoticon_insert_typed (TestFixture *fixture)
{
+ gchar *image_data_base64;
+ gchar *expected_html;
+
test_utils_fixture_change_setting_boolean (fixture, "org.gnome.evolution.mail",
"composer-magic-smileys", TRUE);
test_utils_fixture_change_setting_boolean (fixture, "org.gnome.evolution.mail",
"composer-unicode-smileys", FALSE);
+ image_data_base64 = test_utils_get_base64_data_for_image
("/usr/share/icons/Adwaita/16x16/emotes/face-smile.png");
+
+ expected_html = g_strconcat (HTML_PREFIX "<div>before <img src=\"data:image/png;base64,",
+ image_data_base64, "\" alt=\":-)\">after</div>" HTML_SUFFIX, NULL);
+
+ g_free (image_data_base64);
+
if (!test_utils_run_simple_test (fixture,
"mode:html\n"
"type:before :)after\n",
- HTML_PREFIX "<div>before <img src=\"data:image/png;base64,"
- "iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABHNCSVQICAgIfA"
- "hkiAAAAAlwSFlzAAAN1wAADdcBQiibeAAAABl0RVh0U29mdHdhcmUAd3d3Lmlu"
- "a3NjYXBlLm9yZ5vuPBoAAAAXdEVYdEF1dGhvcgBMYXBvIENhbGFtYW5kcmVp35"
- "EaKgAAACl0RVh0RGVzY3JpcHRpb24AQmFzZWQgb2YgSmFrdWIgU3RlaW5lciBk"
- "ZXNpZ26ghAVzAAADRklEQVQ4jWWTXWiVdQCHn//7ec4+zmrbcZ+usT4wy23ilj"
- "K3cmIRIl10GYGgVCMovOjjMsSrwGEFSXohMchCKBBUWEFqrQ10yTbHNh2bp03n"
- "vrdztvN+/t/330U3C3/XD8/FDx6hlGLrLnzeXq0L/R1bVwfQxB4AYvW3H4kbkY"
- "ouHvuyb24rL7YKej7rOJYsKv7m5Vf22eW1VUZheRqUIr+8yNLsYzk00O/nNzc/"
- "fu9M/4UnBD982nat9oXdnS0HX08I/w7CWgRLAgp8E8JKIrOJvt6rXmZ8+HrXt7"
- "cPA2gA33/S3lXz3K7O1vb9iWj6PFd+vcaVgQxaugyRLoMSxenvfuTu72d47eCh"
- "RE3Diwe632/pAtDOn+io061k9562NxLe3XPE0QK3J/LcHllH2UmwCxAFRfw1km"
- "Po3gze6FlePXQkqZt298mjLXVaKMPjzc177XjmOnHuAQJI2BoJWwcVI5QCFZOw"
- "DCxLIHMZePgH+/d12FEkjxuRDDrLKrbrwUQvQsTg+xxpT6OltyG83H9PbWZ596"
- "16GlPrCG+N4NEtKp49qkcy6DQCpTcWPVXB+uI0q2jUFHrsrHLRyx3ihVkEArWx"
- "wZsvScIFH5mTLMxPUbwjRSC1RiOUwvCyC+ihz9OFNlf716mtlWyvd6moKsQ0BM"
- "r1eTiT5d5Ejrl5l+ZSRaWlEUTKMFzXGc3cH9pba5RgyU0O7ypnaDnBz79lWd2Y"
- "oyChIaRCKEFjXQmdO1OkYsnU2C183xs1vIib45P3W9OFBZrlriCyKzSlK9nd2A"
- "ZFNfjuBpZdjB75+JkB5KNxfLue4fHx2JPcNGSkekamFj+qbtCTOyIF+KhonnC1"
- "F19LEGOioggt2MCWEt2PeSBN+kcyfiC1Hn1gbHHpcs/X1j8rbmupuW6mlESPFF"
- "ocEwcBkesifAc7DFFOQMaBS2Oak3Pj0z/dmL6kAVTkq09lA3Py8ly1M7hmMJ8L"
- "8bMu5qZDgeti5F3WciF3VjUuTpY6yw6TS/rMqf+18EFLi5lPrZxUSp14piiXqE"
- "n6ojLpA/DYsZh1bDWVLfIU4qtyb9sX5wYHwydqBHi7o6FJI/xQINqjWDwPoGtq"
- "UqH6Ysyzv/w5PbyV/xd0ZaEGG/mx/wAAAABJRU5ErkJggg==\" alt=\":-)\">"
- "after</div>" HTML_SUFFIX,
+ expected_html,
"before :-)after"))
g_test_fail ();
+
+ g_free (expected_html);
}
static void
test_emoticon_insert_typed_dash (TestFixture *fixture)
{
+ gchar *image_data_base64;
+ gchar *expected_html;
+
test_utils_fixture_change_setting_boolean (fixture, "org.gnome.evolution.mail",
"composer-magic-smileys", TRUE);
test_utils_fixture_change_setting_boolean (fixture, "org.gnome.evolution.mail",
"composer-unicode-smileys", FALSE);
+ image_data_base64 = test_utils_get_base64_data_for_image
("/usr/share/icons/Adwaita/16x16/emotes/face-smile.png");
+
+ expected_html = g_strconcat (HTML_PREFIX "<div>before <img src=\"data:image/png;base64,",
+ image_data_base64, "\" alt=\":-)\">after</div>" HTML_SUFFIX, NULL);
+
+ g_free (image_data_base64);
+
if (!test_utils_run_simple_test (fixture,
"mode:html\n"
"type:before :-)after\n",
- HTML_PREFIX "<div>before <img src=\"data:image/png;base64,"
- "iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABHNCSVQICAgIfA"
- "hkiAAAAAlwSFlzAAAN1wAADdcBQiibeAAAABl0RVh0U29mdHdhcmUAd3d3Lmlu"
- "a3NjYXBlLm9yZ5vuPBoAAAAXdEVYdEF1dGhvcgBMYXBvIENhbGFtYW5kcmVp35"
- "EaKgAAACl0RVh0RGVzY3JpcHRpb24AQmFzZWQgb2YgSmFrdWIgU3RlaW5lciBk"
- "ZXNpZ26ghAVzAAADRklEQVQ4jWWTXWiVdQCHn//7ec4+zmrbcZ+usT4wy23ilj"
- "K3cmIRIl10GYGgVCMovOjjMsSrwGEFSXohMchCKBBUWEFqrQ10yTbHNh2bp03n"
- "vrdztvN+/t/330U3C3/XD8/FDx6hlGLrLnzeXq0L/R1bVwfQxB4AYvW3H4kbkY"
- "ouHvuyb24rL7YKej7rOJYsKv7m5Vf22eW1VUZheRqUIr+8yNLsYzk00O/nNzc/"
- "fu9M/4UnBD982nat9oXdnS0HX08I/w7CWgRLAgp8E8JKIrOJvt6rXmZ8+HrXt7"
- "cPA2gA33/S3lXz3K7O1vb9iWj6PFd+vcaVgQxaugyRLoMSxenvfuTu72d47eCh"
- "RE3Diwe632/pAtDOn+io061k9562NxLe3XPE0QK3J/LcHllH2UmwCxAFRfw1km"
- "Po3gze6FlePXQkqZt298mjLXVaKMPjzc177XjmOnHuAQJI2BoJWwcVI5QCFZOw"
- "DCxLIHMZePgH+/d12FEkjxuRDDrLKrbrwUQvQsTg+xxpT6OltyG83H9PbWZ596"
- "16GlPrCG+N4NEtKp49qkcy6DQCpTcWPVXB+uI0q2jUFHrsrHLRyx3ihVkEArWx"
- "wZsvScIFH5mTLMxPUbwjRSC1RiOUwvCyC+ihz9OFNlf716mtlWyvd6moKsQ0BM"
- "r1eTiT5d5Ejrl5l+ZSRaWlEUTKMFzXGc3cH9pba5RgyU0O7ypnaDnBz79lWd2Y"
- "oyChIaRCKEFjXQmdO1OkYsnU2C183xs1vIib45P3W9OFBZrlriCyKzSlK9nd2A"
- "ZFNfjuBpZdjB75+JkB5KNxfLue4fHx2JPcNGSkekamFj+qbtCTOyIF+KhonnC1"
- "F19LEGOioggt2MCWEt2PeSBN+kcyfiC1Hn1gbHHpcs/X1j8rbmupuW6mlESPFF"
- "ocEwcBkesifAc7DFFOQMaBS2Oak3Pj0z/dmL6kAVTkq09lA3Py8ly1M7hmMJ8L"
- "8bMu5qZDgeti5F3WciF3VjUuTpY6yw6TS/rMqf+18EFLi5lPrZxUSp14piiXqE"
- "n6ojLpA/DYsZh1bDWVLfIU4qtyb9sX5wYHwydqBHi7o6FJI/xQINqjWDwPoGtq"
- "UqH6Ysyzv/w5PbyV/xd0ZaEGG/mx/wAAAABJRU5ErkJggg==\" alt=\":-)\">"
- "after</div>" HTML_SUFFIX,
+ expected_html,
"before :-)after"))
g_test_fail ();
+
+ g_free (expected_html);
}
static void
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]