[Spice-devel] [PATCH 3/3] tests: add spice-session test

[Victor Toso]

Checking if URIs are being parsed and generated correctly.
---
 tests/Makefile.am |  4 +++
 tests/session.c   | 74 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 78 insertions(+)
 create mode 100644 tests/session.c

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 0985a76..7889c5b 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -3,6 +3,7 @@ NULL =
 noinst_PROGRAMS =				\
 	coroutine				\
 	util					\
+	session					\
 	$(NULL)
 
 TESTS = $(noinst_PROGRAMS)
@@ -31,4 +32,7 @@ coroutine_LDADD = \
 	$(top_builddir)/gtk/.libs/coroutine_ucontext.o \
 	$(top_builddir)/gtk/.libs/continuation.o
 
+session_SOURCES = \
+	session.c
+
 -include $(top_srcdir)/git.mk
diff --git a/tests/session.c b/tests/session.c
new file mode 100644
index 0000000..6821d12
--- /dev/null
+++ b/tests/session.c
@@ -0,0 +1,74 @@
+#include <glib.h>
+
+#include "spice-session.h"
+
+static void test_session_uri(void)
+{
+    SpiceSession *s;
+    gint i;
+
+    struct {
+        gchar *port;
+        gchar *tls_port;
+        gchar *uri_input;
+        gchar *uri_output;
+    } tests[] = {
+        /* Arguments with empty value */
+        { "5900", NULL,
+          "spice://localhost?port=5900&tls-port=",
+          "spice://localhost?port=5900&" },
+        { "5910", NULL,
+          "spice://localhost?tls-port=&port=5910",
+          "spice://localhost?port=5910&" },
+        { NULL, "5920",
+          "spice://localhost?tls-port=5920&port=",
+          "spice://localhost?tls-port=5920" },
+        { NULL, "5930",
+          "spice://localhost?port=&tls-port=5930",
+          "spice://localhost?tls-port=5930" },
+    };
+
+    /* Set URI and check URI, port and tls_port */
+    for (i = 0; i < G_N_ELEMENTS(tests); i++) {
+        gchar *uri, *port, *tls_port;
+
+        s = spice_session_new();
+        g_object_set(s, "uri", tests[i].uri_input, NULL);
+        g_object_get(s,
+                     "uri", &uri,
+                     "port", &port,
+                     "tls-port", &tls_port,
+                      NULL);
+        g_assert_cmpstr (tests[i].uri_output, ==, uri);
+        g_assert_cmpstr (tests[i].port, ==, port);
+        g_assert_cmpstr (tests[i].tls_port, ==, tls_port);
+        g_clear_pointer (&uri, g_free);
+        g_clear_pointer (&port, g_free);
+        g_clear_pointer (&tls_port, g_free);
+        g_object_unref (s);
+    }
+
+    /* Set port and tls_port, check URI */
+    for (i = 0; i < G_N_ELEMENTS(tests); i++) {
+        gchar *uri;
+
+        s = spice_session_new();
+        g_object_set(s,
+                     "port", tests[i].port,
+                     "tls-port", tests[i].tls_port,
+                      NULL);
+        g_object_get(s, "uri", &uri, NULL);
+        g_assert_cmpstr (tests[i].uri_output, ==, uri);
+        g_clear_pointer (&uri, g_free);
+        g_object_unref (s);
+    }
+}
+
+int main(int argc, char* argv[])
+{
+    g_test_init(&argc, &argv, NULL);
+
+    g_test_add_func("/session/uri", test_session_uri);
+
+    return g_test_run ();
+}
-- 
2.1.0