Skip to content

Commit

Permalink
[unit] Added unit test for ril_ecclist. JB#47953
Browse files Browse the repository at this point in the history
  • Loading branch information
@monich
monich committed Nov 22, 2019
1 parent 36f971d commit 4d513b6
Show file tree
Hide file tree
Showing 4 changed files with 335 additions and 0 deletions.
1 change: 1 addition & 0 deletions ofono/.gitignore
View file
Expand Up @@ -48,6 +48,7 @@ unit/test-dbus-access
unit/test-dbus-queue
unit/test-gprs-filter
unit/test-ril_config
unit/test-ril_ecclist
unit/test-ril_util
unit/test-ril_vendor
unit/test-ril-transport
Expand Down
7 changes: 7 additions & 0 deletions ofono/Makefile.am
View file
Expand Up @@ -1062,6 +1062,13 @@ unit_test_ril_config_LDADD = @GLIB_LIBS@ -ldl
unit_objects += $(unit_test_ril_config_OBJECTS)
unit_tests += unit/test-ril_config

unit_test_ril_ecclist_SOURCES = unit/test-ril_ecclist.c \
drivers/ril/ril_ecclist.c src/log.c
unit_test_ril_ecclist_CFLAGS = $(COVERAGE_OPT) $(AM_CFLAGS)
unit_test_ril_ecclist_LDADD = @GLIB_LIBS@ -ldl
unit_objects += $(unit_test_ril_ecclist_OBJECTS)
unit_tests += unit/test-ril_ecclist

unit_test_ril_util_SOURCES = unit/test-ril_util.c drivers/ril/ril_util.c \
src/log.c
unit_test_ril_util_CFLAGS = $(COVERAGE_OPT) $(AM_CFLAGS)
Expand Down
1 change: 1 addition & 0 deletions ofono/unit/coverage
View file
Expand Up @@ -23,6 +23,7 @@ TESTS="\
test-watch \
test-ril_util \
test-ril_config \
test-ril_ecclist \
test-ril-transport \
test-ril_vendor \
test-sms-filter \
Expand Down
326 changes: 326 additions & 0 deletions ofono/unit/test-ril_ecclist.c
View file
@@ -0,0 +1,326 @@
/*
* oFono - Open Source Telephony
*
* Copyright (C) 2019 Jolla Ltd.
* Copyright (C) 2019 Open Mobile Platform LLC.
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 2 as
* published by the Free Software Foundation.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*/

#include "drivers/ril/ril_ecclist.h"
#include "drivers/ril/ril_log.h"

#include "ofono.h"

#include <gutil_strv.h>
#include <sys/stat.h>

#define TMP_DIR_TEMPLATE "test-ril_ecclist-XXXXXX"
#define TEST_TIMEOUT_SEC (20)

GLOG_MODULE_DEFINE("rilmodem");

static gboolean test_debug = FALSE;

struct ril_ecclist_parse_test {
const char* name;
const char* in;
const char* const* out;
};

static gboolean test_timeout_cb(gpointer user_data)
{
g_assert_not_reached();
return G_SOURCE_REMOVE;
}

static gboolean test_idle_quit_cb(gpointer loop)
{
g_main_loop_quit(loop);
return G_SOURCE_REMOVE;
}

static void test_quit_cb(struct ril_ecclist *ecc, gpointer loop)
{
g_idle_add(test_idle_quit_cb, loop);
}

static void test_inc_cb(struct ril_ecclist *ecc, gpointer ptr)
{
(*(int*)ptr)++;
}

/* ==== parse ==== */

static void test_parse(gconstpointer data)
{
const struct ril_ecclist_parse_test *test = data;
char *dir = g_dir_make_tmp(TMP_DIR_TEMPLATE, NULL);
char *file = g_build_filename(dir, "ecclist", NULL);
struct ril_ecclist *ecc;

GDEBUG("Created file %s", file);
g_assert(g_file_set_contents(file, test->in, -1, NULL));
ecc = ril_ecclist_new(file);
g_assert(gutil_strv_equal(ecc->list, (char**)test->out));

ril_ecclist_unref(ecc);
remove(file);
remove(dir);
g_free(file);
g_free(dir);
}

static const char* null_str = NULL;
static const char single_str_in[] = "911";
static const char* single_str_out[] = { "911", NULL };
static const char double_str_in[] = "911,112";
static const char double2_str_in[] = "911, 112,";
static const char double3_str_in[] = "911, 911, 112 ";
static const char* double_str_out[] = { "112", "911", NULL };
static const char mtk_str_in[] = "112,31;911,31;112,-1;911,-1";
static const char mtk2_str_in[] = "112,31; 911,31; 112; 911 ";

static const struct ril_ecclist_parse_test tests[] = {
{ "empty", "", &null_str },
{ "single", single_str_in, single_str_out },
{ "double", double_str_in, double_str_out },
{ "double2", double2_str_in, double_str_out },
{ "double3", double3_str_in, double_str_out },
{ "mtk", mtk_str_in, double_str_out },
{ "mtk2", mtk2_str_in, double_str_out }
};

/* ==== file_perm ==== */

static void test_file_perm()
{
char *dir = g_dir_make_tmp(TMP_DIR_TEMPLATE, NULL);
char *file = g_build_filename(dir, "ecclist", NULL);
int count = 0;
struct ril_ecclist *ecc;
gulong id[2];
GMainLoop *loop = g_main_loop_new(NULL, FALSE);
guint test_timeout_id = test_debug ? 0 :
g_timeout_add_seconds(TEST_TIMEOUT_SEC, test_timeout_cb, NULL);

GDEBUG("Created file %s", file);
g_assert(g_file_set_contents(file, single_str_in, -1, NULL));
ecc = ril_ecclist_new(file);
id[0] = ril_ecclist_add_list_changed_handler(ecc, test_inc_cb, &count);
id[1] = ril_ecclist_add_list_changed_handler(ecc, test_quit_cb, loop);

g_assert(id[0]);
g_assert(id[1]);
g_assert(gutil_strv_equal(ecc->list, (char**)single_str_out));

/* Modify the file */
g_assert(g_file_set_contents(file, double_str_in, -1, NULL));

/* ril_ecclist needs event loop to process filesystem change events */
g_main_loop_run(loop);

g_assert(count == 1);
g_assert(gutil_strv_equal(ecc->list, (char**)double_str_out));

/* Making file unreadable resets the ecc list */
GDEBUG("Making file %s unreadable", file);
g_assert(g_file_set_contents(file, single_str_in, -1, NULL));
g_assert(chmod(file, 0) == 0);
count = 0;
g_main_loop_run(loop);
g_assert(count == 1);
g_assert(!ecc->list);

if (test_timeout_id) {
g_source_remove(test_timeout_id);
}
g_main_loop_unref(loop);
ril_ecclist_remove_handler(ecc, id[0]);
ril_ecclist_remove_handler(ecc, id[1]);
ril_ecclist_unref(ecc);
remove(dir);
g_free(file);
g_free(dir);
}

/* ==== file_change ==== */

static void test_file_change()
{
char *dir = g_dir_make_tmp(TMP_DIR_TEMPLATE, NULL);
char *file = g_build_filename(dir, "ecclist", NULL);
int count = 0;
struct ril_ecclist *ecc;
gulong id[2];
GMainLoop *loop = g_main_loop_new(NULL, FALSE);
guint test_timeout_id = test_debug ? 0 :
g_timeout_add_seconds(TEST_TIMEOUT_SEC, test_timeout_cb, NULL);

GDEBUG("Created file %s", file);
g_assert(g_file_set_contents(file, single_str_in, -1, NULL));
ecc = ril_ecclist_new(file);
id[0] = ril_ecclist_add_list_changed_handler(ecc, test_inc_cb, &count);
id[1] = ril_ecclist_add_list_changed_handler(ecc, test_quit_cb, loop);

g_assert(id[0]);
g_assert(id[1]);
g_assert(gutil_strv_equal(ecc->list, (char**)single_str_out));

/* Modify the file */
g_assert(g_file_set_contents(file, double_str_in, -1, NULL));

/* ril_ecclist needs event loop to process filesystem change events */
g_main_loop_run(loop);

g_assert(count == 1);
g_assert(gutil_strv_equal(ecc->list, (char**)double_str_out));

/* Removing the file resets the ecc list */
GDEBUG("Removing file %s", file);
g_assert(remove(file) == 0);
count = 0;
g_main_loop_run(loop);
g_assert(count == 1);
g_assert(!ecc->list);

if (test_timeout_id) {
g_source_remove(test_timeout_id);
}
g_main_loop_unref(loop);
ril_ecclist_remove_handler(ecc, id[0]);
ril_ecclist_remove_handler(ecc, id[1]);
ril_ecclist_unref(ecc);
remove(dir);
g_free(file);
g_free(dir);
}

/* ==== dir_change ==== */

static void test_dir_change()
{
char *dir = g_dir_make_tmp(TMP_DIR_TEMPLATE, NULL);
char *file = g_build_filename(dir, "ecclist", NULL);
int count = 0;
struct ril_ecclist *ecc;
gulong id[3];
GMainLoop *loop = g_main_loop_new(NULL, FALSE);
guint test_timeout_id = test_debug ? 0 :
g_timeout_add_seconds(TEST_TIMEOUT_SEC, test_timeout_cb, NULL);

GDEBUG("Created directory %s", dir);
ecc = ril_ecclist_new(file);
id[0] = ril_ecclist_add_list_changed_handler(ecc, test_inc_cb, &count);
id[1] = ril_ecclist_add_list_changed_handler(ecc, test_quit_cb, loop);

g_assert(id[0]);
g_assert(id[1]);
g_assert(!ecc->list);
GDEBUG("Created file %s", file);
g_assert(g_file_set_contents(file, single_str_in, -1, NULL));

/* ril_ecclist needs event loop to process filesystem change events */
g_main_loop_run(loop);

g_assert(count == 1);
g_assert(gutil_strv_equal(ecc->list, (char**)single_str_out));

/* Removing the directory resets the ecc list */
GDEBUG("Removing directory %s", dir);
g_assert(remove(file) == 0);
g_assert(remove(dir) == 0);
count = 0;
g_main_loop_run(loop);
g_assert(count == 1);
g_assert(!ecc->list);

if (test_timeout_id) {
g_source_remove(test_timeout_id);
}
g_main_loop_unref(loop);
ril_ecclist_remove_handler(ecc, id[0]);
ril_ecclist_remove_handler(ecc, id[1]);
ril_ecclist_unref(ecc);
g_free(file);
g_free(dir);
}

/* ==== null ==== */

static void test_null(void)
{
char *dir = g_dir_make_tmp(TMP_DIR_TEMPLATE, NULL);
char *file = g_build_filename(dir, "ecclist", NULL);
struct ril_ecclist *ecc;

/* Make sure neither directory nor file exist */
remove(dir);
ecc = ril_ecclist_new(file);
g_assert(ecc);
g_assert(!ecc->list);
g_assert(!ril_ecclist_new(NULL));
g_assert(!ril_ecclist_ref(NULL));
g_assert(!ril_ecclist_add_list_changed_handler(NULL, NULL, NULL));
g_assert(!ril_ecclist_add_list_changed_handler(ecc, NULL, NULL));
ril_ecclist_unref(NULL);
ril_ecclist_remove_handler(NULL, 0);
ril_ecclist_remove_handler(ecc, 0);
ril_ecclist_unref(ril_ecclist_ref(ecc));
ril_ecclist_unref(ecc);

g_free(file);
g_free(dir);
}

#define TEST_(name) "/ril_ecclist/" name

int main(int argc, char *argv[])
{
int i;

g_test_init(&argc, &argv, NULL);

gutil_log_timestamp = FALSE;
gutil_log_default.name = "test-ril_ecclist";
gutil_log_default.level = g_test_verbose() ?
GLOG_LEVEL_VERBOSE : GLOG_LEVEL_NONE;
__ofono_log_init(gutil_log_default.name,
g_test_verbose() ? "*" : NULL,
FALSE, FALSE);

if (argc > 1 && !strcmp(argv[1] , "-d")) {
test_debug = TRUE;
GDEBUG("Debugging on (no timeout)");
}

for (i = 0; i < G_N_ELEMENTS(tests); i++) {
const struct ril_ecclist_parse_test* test = tests + i;
char* path = g_strconcat(TEST_("parse/"), test->name, NULL);

g_test_add_data_func(path, test, test_parse);
g_free(path);
}

g_test_add_func(TEST_("null"), test_null);
g_test_add_func(TEST_("file_perm"), test_file_perm);
g_test_add_func(TEST_("file_change"), test_file_change);
g_test_add_func(TEST_("dir_change"), test_dir_change);

return g_test_run();
}

/*
* Local Variables:
* mode: C
* c-basic-offset: 8
* indent-tabs-mode: t
* End:
*/

0 comments on commit 4d513b6

Please sign in to comment.