From 6493b6879a0b6dbb402d6df595208f44e7eb5c22 Mon Sep 17 00:00:00 2001
From: Simon McVittie <smcv@collabora.com>
Date: Fri, 2 Jun 2017 19:35:56 +0100
Subject: [PATCH 16/49] test/containers: Exercise the new parameter checking

Signed-off-by: Simon McVittie <smcv@collabora.com>
---
 test/containers.c | 82 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 82 insertions(+)

diff --git a/test/containers.c b/test/containers.c
index 4499092f..5657b677 100644
--- a/test/containers.c
+++ b/test/containers.c
@@ -111,6 +111,84 @@ test_get_supported_arguments (Fixture *f,
 }
 
 static void
+test_unsupported_parameter (Fixture *f,
+                            gconstpointer context)
+{
+#ifdef HAVE_CONTAINERS_TEST
+  GVariant *tuple;
+  GVariant *parameters;
+  GVariantDict named_argument_builder;
+
+  if (f->skip)
+    return;
+
+  f->proxy = g_dbus_proxy_new_sync (f->unconfined_conn,
+                                    G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES,
+                                    NULL, DBUS_SERVICE_DBUS,
+                                    DBUS_PATH_DBUS, DBUS_INTERFACE_CONTAINERS1,
+                                    NULL, &f->error);
+  g_assert_no_error (f->error);
+
+  g_variant_dict_init (&named_argument_builder, NULL);
+  g_variant_dict_insert (&named_argument_builder,
+                         "ThisArgumentIsntImplemented",
+                         "b", FALSE);
+
+  /* Floating reference, call_..._sync takes ownership */
+  parameters = g_variant_new ("(ssa{sv}@a{sv})",
+                              "com.example.NotFlatpak",
+                              "sample-app",
+                              NULL, /* no metadata */
+                              g_variant_dict_end (&named_argument_builder));
+
+  tuple = g_dbus_proxy_call_sync (f->proxy, "AddContainerServer", parameters,
+                                  G_DBUS_CALL_FLAGS_NONE, -1, NULL, &f->error);
+
+  g_assert_error (f->error, G_DBUS_ERROR, G_DBUS_ERROR_INVALID_ARGS);
+  g_assert_null (tuple);
+  g_clear_error (&f->error);
+#else /* !HAVE_CONTAINERS_TEST */
+  g_test_skip ("Containers or gio-unix-2.0 not supported");
+#endif /* !HAVE_CONTAINERS_TEST */
+}
+
+static void
+test_invalid_type_name (Fixture *f,
+                        gconstpointer context)
+{
+#ifdef HAVE_CONTAINERS_TEST
+  GVariant *tuple;
+  GVariant *parameters;
+
+  if (f->skip)
+    return;
+
+  f->proxy = g_dbus_proxy_new_sync (f->unconfined_conn,
+                                    G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES,
+                                    NULL, DBUS_SERVICE_DBUS,
+                                    DBUS_PATH_DBUS, DBUS_INTERFACE_CONTAINERS1,
+                                    NULL, &f->error);
+  g_assert_no_error (f->error);
+
+  /* Floating reference, call_..._sync takes ownership */
+  parameters = g_variant_new ("(ssa{sv}a{sv})",
+                              "this is not a valid container type name",
+                              "sample-app",
+                              NULL, /* no metadata */
+                              NULL); /* no named arguments */
+
+  tuple = g_dbus_proxy_call_sync (f->proxy, "AddContainerServer", parameters,
+                                  G_DBUS_CALL_FLAGS_NONE, -1, NULL, &f->error);
+
+  g_assert_error (f->error, G_DBUS_ERROR, G_DBUS_ERROR_INVALID_ARGS);
+  g_assert_null (tuple);
+  g_clear_error (&f->error);
+#else /* !HAVE_CONTAINERS_TEST */
+  g_test_skip ("Containers or gio-unix-2.0 not supported");
+#endif /* !HAVE_CONTAINERS_TEST */
+}
+
+static void
 teardown (Fixture *f,
     gconstpointer context G_GNUC_UNUSED)
 {
@@ -149,6 +227,10 @@ main (int argc,
 
   g_test_add ("/containers/get-supported-arguments", Fixture, NULL,
               setup, test_get_supported_arguments, teardown);
+  g_test_add ("/containers/unsupported-parameter", Fixture, NULL,
+              setup, test_unsupported_parameter, teardown);
+  g_test_add ("/containers/invalid-type-name", Fixture, NULL,
+              setup, test_invalid_type_name, teardown);
 
   return g_test_run ();
 }
-- 
2.11.0