tests: make sure all vars are unreffed when not needed

Change-Id: I9320fa28495280d801cc6be6dc7ca42b13877787
Signed-off-by: Maciej Słodczyk <m.slodczyk2@partner.samsung.com>

diff --git a/tests/cmocka-proc-dbus-typecheck.c b/tests/cmocka-proc-dbus-typecheck.c
index 6eb15ca..ddb0707 100644 (file)
--- a/tests/cmocka-proc-dbus-typecheck.c
+++ b/tests/cmocka-proc-dbus-typecheck.c
@@ -182,6 +182,7 @@ static void test_dbus_get_app_memory(void **state)
 
        expect_value(__wrap_g_dbus_method_invocation_return_value, invocation, (GDBusMethodInvocation *)0x424242);
        dbus_get_app_memory((GDBusMethodInvocation *)0x424242, params);
+       g_variant_unref(params);
 }
 
 int __wrap_proc_get_cpu_time(pid_t pid, unsigned long *utime,
@@ -208,6 +209,7 @@ static void test_dbus_get_app_cpu(void **state)
        expect_value(__wrap_g_dbus_method_invocation_return_value, invocation, (GDBusMethodInvocation *)0x31337);
 
        dbus_get_app_cpu((GDBusMethodInvocation *)0x31337, params);
+       g_variant_unref(params);
 }
 
 int __wrap_resourced_proc_status_change(int status, pid_t pid, char *app_name, char *pkg_name, int apptype)
@@ -234,6 +236,7 @@ static void test_proc_dbus_aul_launch(void **state)
        expect_string(__wrap_resourced_proc_status_change, pkg_name, "pkgid");
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_SERVICE);
        proc_dbus_aul_launch(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(isss)", 445, "appid", "pkgid", "ui");
        expect_value(__wrap_resourced_proc_status_change, status, PROC_CGROUP_SET_LAUNCH_REQUEST);
@@ -242,6 +245,7 @@ static void test_proc_dbus_aul_launch(void **state)
        expect_string(__wrap_resourced_proc_status_change, pkg_name, "pkgid");
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_GUI);
        proc_dbus_aul_launch(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(isss)", 789, "appid", "pkgid", "widget");
        expect_value(__wrap_resourced_proc_status_change, status, PROC_CGROUP_SET_LAUNCH_REQUEST);
@@ -250,6 +254,7 @@ static void test_proc_dbus_aul_launch(void **state)
        expect_string(__wrap_resourced_proc_status_change, pkg_name, "pkgid");
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_WIDGET);
        proc_dbus_aul_launch(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(isss)", 101, "appid", "pkgid", "watch");
        expect_value(__wrap_resourced_proc_status_change, status, PROC_CGROUP_SET_LAUNCH_REQUEST);
@@ -258,6 +263,7 @@ static void test_proc_dbus_aul_launch(void **state)
        expect_string(__wrap_resourced_proc_status_change, pkg_name, "pkgid");
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_WATCH);
        proc_dbus_aul_launch(params);
+       g_variant_unref(params);
 }
 
 void __wrap_resourced_notify(enum notifier_type status, void *data)
@@ -311,6 +317,7 @@ static void test_proc_dbus_exclude_appid_signal_handler(void **state)
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_APP_WAKEUP);
        expect_check(__wrap_resourced_notify, data, check_data_in_resourced_notify_wakeup, &ps);
        proc_dbus_exclude_appid_signal_handler(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(ss)", "exclude", "appid2");
        expect_string(__wrap_find_app_info_by_appid, appid, "appid2");
@@ -328,6 +335,7 @@ static void test_proc_dbus_exclude_appid_signal_handler(void **state)
        expect_value(__wrap_proc_set_runtime_exclude_list, type, pe.type);
        will_return(__wrap_proc_set_runtime_exclude_list, RESOURCED_ERROR_NONE);
        proc_dbus_exclude_appid_signal_handler(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(ss)", "include", "appid3");
        expect_string(__wrap_find_app_info_by_appid, appid, "appid3");
@@ -344,6 +352,7 @@ static void test_proc_dbus_exclude_appid_signal_handler(void **state)
        expect_value(__wrap_proc_set_runtime_exclude_list, type, pe.type);
        will_return(__wrap_proc_set_runtime_exclude_list, RESOURCED_ERROR_NONE);
        proc_dbus_exclude_appid_signal_handler(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(ss)", "include", NULL);
        expect_any(__wrap_find_app_info_by_appid, appid);
@@ -361,6 +370,7 @@ static void test_proc_dbus_exclude_appid_signal_handler(void **state)
        will_return(__wrap_proc_set_runtime_exclude_list, RESOURCED_ERROR_NONE);
 
        proc_dbus_exclude_appid_signal_handler(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(ss)", "exclude", NULL);
        expect_any(__wrap_find_app_info_by_appid, appid);
@@ -378,6 +388,7 @@ static void test_proc_dbus_exclude_appid_signal_handler(void **state)
        will_return(__wrap_proc_set_runtime_exclude_list, RESOURCED_ERROR_NONE);
 
        proc_dbus_exclude_appid_signal_handler(params);
+       g_variant_unref(params);
 }
 
 struct proc_app_info *__wrap_find_app_info(const pid_t pid)
@@ -402,6 +413,7 @@ static void test_proc_dbus_exclude_signal_handler(void **state)
        expect_check(__wrap_resourced_notify, data, check_data_in_resourced_notify_wakeup, &((struct proc_status){.pid = 1286}));
 
        __real_proc_dbus_exclude_signal_handler(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(si)", "exclude", 1416);
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_CONTROL_EXCLUDE);
@@ -412,6 +424,7 @@ static void test_proc_dbus_exclude_signal_handler(void **state)
        will_return(__wrap_proc_set_runtime_exclude_list, RESOURCED_ERROR_NONE);
 
        __real_proc_dbus_exclude_signal_handler(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(si)", "include", 3213);
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_CONTROL_EXCLUDE);
@@ -422,6 +435,7 @@ static void test_proc_dbus_exclude_signal_handler(void **state)
        will_return(__wrap_proc_set_runtime_exclude_list, RESOURCED_ERROR_NONE);
 
        __real_proc_dbus_exclude_signal_handler(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_active_signal_handler(void **state)
@@ -441,6 +455,7 @@ static void test_proc_dbus_active_signal_handler(void **state)
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_NONE);
 
        proc_dbus_active_signal_handler(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_aul_changestate(void **state)
@@ -460,6 +475,7 @@ static void test_proc_dbus_aul_changestate(void **state)
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_SERVICE);
 
        proc_dbus_aul_changestate(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_aul_terminated(void **state)
@@ -479,6 +495,7 @@ static void test_proc_dbus_aul_terminated(void **state)
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_NONE);
 
        proc_dbus_aul_terminated(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_aul_resume(void **state)
@@ -497,6 +514,7 @@ static void test_proc_dbus_aul_resume(void **state)
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_SERVICE);
 
        proc_dbus_aul_resume(params);
+       g_variant_unref(params);
 }
 
 struct proc_app_info *__wrap_proc_create_app_info(const char *appid,
@@ -563,6 +581,7 @@ static void test_proc_dbus_prelaunch_signal_handler(void **state)
        expect_any(__wrap_resourced_notify, data);
 
        proc_dbus_prelaunch_signal_handler(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_aul_terminate(void **state)
@@ -582,6 +601,7 @@ static void test_proc_dbus_aul_terminate(void **state)
        expect_value(__wrap_resourced_proc_status_change, apptype, PROC_TYPE_NONE);
 
        proc_dbus_aul_terminate(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_lcd_on(void **state)
@@ -591,6 +611,7 @@ static void test_proc_dbus_lcd_on(void **state)
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_LCD_ON);
        expect_any(__wrap_resourced_notify, data);
        proc_dbus_lcd_on(params);
+       g_variant_unref(params);
 }
 
 static void test_proc_dbus_lcd_off(void **state)
@@ -600,6 +621,7 @@ static void test_proc_dbus_lcd_off(void **state)
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_LCD_OFF);
        expect_any(__wrap_resourced_notify, data);
        proc_dbus_lcd_off(params);
+       g_variant_unref(params);
 }
 
 static void test_booting_done_signal_handler(void **state)
@@ -609,6 +631,7 @@ static void test_booting_done_signal_handler(void **state)
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_BOOTING_DONE);
        expect_any(__wrap_resourced_notify, data);
        booting_done_signal_handler(params);
+       g_variant_unref(params);
 }
 
 static void test_low_battery_signal_handler(void **state)
@@ -618,6 +641,7 @@ static void test_low_battery_signal_handler(void **state)
        expect_value(__wrap_resourced_notify, status, RESOURCED_NOTIFIER_LOW_BATTERY);
        expect_any(__wrap_resourced_notify, data);
        low_battery_signal_handler(params);
+       g_variant_unref(params);
 }
 
 enum proc_state __wrap_proc_check_suspend_state(struct proc_app_info *pai)
@@ -643,6 +667,7 @@ static void test_proc_dbus_suspend_hint(void **state)
        will_return(__wrap_proc_check_suspend_state, PROC_STATE_DEFAULT);
 
        proc_dbus_suspend_hint(params);
+       g_variant_unref(params);
 
        params = g_variant_new("(i)", 2589);
        pai.main_pid = 2589;
@@ -656,6 +681,7 @@ static void test_proc_dbus_suspend_hint(void **state)
        expect_any(__wrap_resourced_notify, data);
 
        proc_dbus_suspend_hint(params);
+       g_variant_unref(params);
 }
 
 void __wrap_proc_dbus_exclude_signal_handler(GVariant *params)