From: hyunuktak Date: Wed, 7 Jun 2017 09:13:31 +0000 (+0900) Subject: Excluded some lines and block coverage X-Git-Tag: submit/tizen/20170608.073827~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6b3595db981697ddd5ce7e3b2cb00cc24d76b299;p=platform%2Fcore%2Fapi%2Fsmart-traffic-control.git Excluded some lines and block coverage Change-Id: I15ed4e7ec13f0499c1a4575c181b2a4cf8dde379 Signed-off-by: hyunuktak --- diff --git a/CMakeLists.txt b/CMakeLists.txt index 8c5f60c..373503b 100755 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -44,7 +44,7 @@ FOREACH(flag ${STC_DEPS_CFLAGS}) ENDFOREACH(flag) -SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${EXTRA_CFLAGS} -fPIC -Wall") +SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${EXTRA_CFLAGS} -fPIC -Wall -fprofile-arcs -ftest-coverage") SET(CMAKE_C_FLAGS_DEBUG "-O0 -g") SET(CMAKE_C_FLAGS_RELEASE "-O2 -g") diff --git a/packaging/capi-network-stc.spec b/packaging/capi-network-stc.spec index dc68d70..1101b23 100755 --- a/packaging/capi-network-stc.spec +++ b/packaging/capi-network-stc.spec @@ -1,6 +1,6 @@ Name: capi-network-stc Summary: A Smart Traffic Control (STC) libraries in Native API -Version: 0.0.7 +Version: 0.0.8 Release: 1 Group: Network & Connectivity/API License: Apache-2.0 diff --git a/src/internal/stc-dbus.c b/src/internal/stc-dbus.c index 64c6c39..ad67a21 100755 --- a/src/internal/stc-dbus.c +++ b/src/internal/stc-dbus.c @@ -71,8 +71,8 @@ int _stc_dbus_create(void) g_stc_dbus_conn.connection = g_bus_get_sync(G_BUS_TYPE_SYSTEM, NULL, &err); if (g_stc_dbus_conn.connection == NULL) { if (err != NULL) { - STC_LOGE("Failed to connect to the D-BUS daemon [%s]", err->message); - g_error_free(err); + STC_LOGE("Failed to connect to the D-BUS daemon [%s]", err->message); //LCOV_EXCL_LINE + g_error_free(err); //LCOV_EXCL_LINE } return STC_ERROR_OPERATION_FAILED; } @@ -131,9 +131,9 @@ GVariant *_stc_dbus_invoke_method(const char *dest, connection = _stc_dbus_get_connection(); if (connection == NULL) { - STC_LOGE("GDBusconnection is NULL"); - *dbus_error = STC_ERROR_NOT_INITIALIZED; - return reply; + STC_LOGE("GDBusconnection is NULL"); //LCOV_EXCL_LINE + *dbus_error = STC_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE + return reply; //LCOV_EXCL_LINE } reply = g_dbus_connection_call_sync(connection, @@ -150,13 +150,13 @@ GVariant *_stc_dbus_invoke_method(const char *dest, if (reply == NULL) { if (error != NULL) { - STC_LOGE("g_dbus_connection_call_sync() failed " + STC_LOGE("g_dbus_connection_call_sync() failed " //LCOV_EXCL_LINE "error [%d: %s]", error->code, error->message); - *dbus_error = _stc_convert_error_string_to_enum(error->message); - g_error_free(error); + *dbus_error = _stc_convert_error_string_to_enum(error->message); //LCOV_EXCL_LINE + g_error_free(error); //LCOV_EXCL_LINE } else { - STC_LOGE("g_dbus_connection_call_sync() failed"); - *dbus_error = STC_ERROR_OPERATION_FAILED; + STC_LOGE("g_dbus_connection_call_sync() failed"); //LCOV_EXCL_LINE + *dbus_error = STC_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE } return NULL; @@ -174,8 +174,8 @@ int _stc_dbus_invoke_method_nonblock(const char *dest, connection = _stc_dbus_get_connection(); if (connection == NULL) { - STC_LOGE("GDBusconnection is NULL"); - return STC_ERROR_NOT_INITIALIZED; + STC_LOGE("GDBusconnection is NULL"); //LCOV_EXCL_LINE + return STC_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } g_dbus_connection_call(connection, diff --git a/src/internal/stc-event.c b/src/internal/stc-event.c index 03d79b9..1ee3a9e 100755 --- a/src/internal/stc-event.c +++ b/src/internal/stc-event.c @@ -135,23 +135,23 @@ int _stc_register_event_cb(stc_event_cb event_cb, void *user_data) if (_stc_event_get_ref_count() < 1) { ret = _stc_dbus_create(); if (ret != STC_ERROR_NONE) { - STC_LOGE("Failed to create DBus [%s]", + STC_LOGE("Failed to create DBus [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } ret = _stc_register_signal(); if (ret != STC_ERROR_NONE && ret != STC_ERROR_ALREADY_INITIALIZED) { - STC_LOGE("Failed to register DBus signal [%s]", + STC_LOGE("Failed to register DBus signal [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } ret = __stc_init_service(); if (ret != STC_ERROR_NONE) { - STC_LOGE("Failed to init manager [%s]", + STC_LOGE("Failed to init manager [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } } @@ -208,9 +208,9 @@ guint _stc_event_add_client_idle_cb(stc_event_info_s *event_data, data = g_try_new0(stc_event_info_s, 1); if (!data) { - g_free(idle_data); - idle_data = NULL; - return 0; + g_free(idle_data); //LCOV_EXCL_LINE + idle_data = NULL; //LCOV_EXCL_LINE + return 0; //LCOV_EXCL_LINE } memcpy(data, event_data, sizeof(stc_event_info_s)); @@ -226,10 +226,10 @@ guint _stc_event_add_client_idle_cb(stc_event_info_s *event_data, g_source_unref(src); if (!id) { - g_slist_free(idle_data->event_data->info_data); - g_free(idle_data); - idle_data = NULL; - return id; + g_slist_free(idle_data->event_data->info_data); //LCOV_EXCL_LINE + g_free(idle_data); //LCOV_EXCL_LINE + idle_data = NULL; //LCOV_EXCL_LINE + return id; //LCOV_EXCL_LINE } idle_data->id = id; @@ -245,6 +245,7 @@ void _stc_event_clear_client_cb(void) GSource *src; stc_managed_idle_data_s *data; + //LCOV_EXCL_START while (cur) { GSList *next = cur->next; data = (stc_managed_idle_data_s *)cur->data; @@ -256,6 +257,7 @@ void _stc_event_clear_client_cb(void) } else cur = next; } + //LCOV_EXCL_STOP g_slist_free(g_stc_managed_idler_list); g_stc_managed_idler_list = NULL; diff --git a/src/internal/stc-private.c b/src/internal/stc-private.c index 19ad3d7..1b8796e 100755 --- a/src/internal/stc-private.c +++ b/src/internal/stc-private.c @@ -68,6 +68,7 @@ static __thread GSList *g_stc_handle_list = NULL; * Local Functions Definition *****************************************************************************/ +//LCOV_EXCL_START const char *_stc_convert_error_type_to_string(stc_error_e err) { switch (err) { @@ -254,6 +255,7 @@ const char *_stc_convert_time_period_to_string( return "Unknown"; } } +//LCOV_EXCL_STOP static bool __stc_handle_find(stc_h stc) { @@ -285,10 +287,10 @@ static void __stc_callback_foreach(stc_event_e e, break; } } else { - handle->stats_cb(err, NULL, NULL); + handle->stats_cb(err, NULL, NULL); //LCOV_EXCL_LINE } } else { - handle->stats_cb(err, NULL, NULL); + handle->stats_cb(err, NULL, NULL); //LCOV_EXCL_LINE } handle->stats_cb = NULL; @@ -308,10 +310,10 @@ static void __stc_callback_foreach(stc_event_e e, break; } } else { - handle->stats_all_cb(err, NULL, NULL); + handle->stats_all_cb(err, NULL, NULL); //LCOV_EXCL_LINE } } else { - handle->stats_all_cb(err, NULL, NULL); + handle->stats_all_cb(err, NULL, NULL); //LCOV_EXCL_LINE } handle->stats_all_cb = NULL; @@ -331,16 +333,17 @@ static void __stc_callback_foreach(stc_event_e e, break; } } else { - handle->stats_total_cb(err, NULL, NULL); + handle->stats_total_cb(err, NULL, NULL); //LCOV_EXCL_LINE } } else { - handle->stats_total_cb(err, NULL, NULL); + handle->stats_total_cb(err, NULL, NULL); //LCOV_EXCL_LINE } handle->stats_total_cb = NULL; handle->stats_total_user_data = NULL; } break; + //LCOV_EXCL_START case STC_EVENT_RESTRICTION_GET_RSP: if (handle->restriction_cb) { if (err == STC_ERROR_NONE) { @@ -387,12 +390,14 @@ static void __stc_callback_foreach(stc_event_e e, handle->restriction_all_user_data = NULL; } break; + //LCOV_EXCL_STOP default: break; } } } +//LCOV_EXCL_START static void __stc_signal_callback(stc_event_e e, void *info_data) { GSList *list; @@ -415,6 +420,7 @@ static void __stc_signal_callback(stc_event_e e, void *info_data) } } } +//LCOV_EXCL_STOP static void __stc_event_cb(stc_event_info_s *event_info, void *user_data) { @@ -434,6 +440,7 @@ static void __stc_event_cb(stc_event_info_s *event_info, void *user_data) g_slist_free(info_data); } break; + //LCOV_EXCL_START case STC_EVENT_WARN_THRESHOLD_CROSSED_RSP: case STC_EVENT_RESTRICTION_THRESHOLD_CROSSED_RSP: { @@ -442,6 +449,7 @@ static void __stc_event_cb(stc_event_info_s *event_info, void *user_data) g_free(event_info->info_data); } break; + //LCOV_EXCL_STOP default: break; } @@ -453,9 +461,9 @@ int _stc_initialize(void) ret = _stc_register_event_cb((stc_event_cb)__stc_event_cb, NULL); if (ret != STC_ERROR_NONE && ret != STC_ERROR_ALREADY_INITIALIZED) { - STC_LOGE("Failed to register event callback [%s]", + STC_LOGE("Failed to register event callback [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } return STC_ERROR_NONE; @@ -467,9 +475,9 @@ int _stc_deinitialize(void) ret = _stc_deregister_event_cb(); if (ret != STC_ERROR_NONE && ret != STC_ERROR_NOT_INITIALIZED) { - STC_LOGE("Failed to deregister event callback [%s]", + STC_LOGE("Failed to deregister event callback [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } return STC_ERROR_NONE;; @@ -479,8 +487,8 @@ int _stc_handle_create(stc_h *stc) { *stc = g_try_malloc0(sizeof(stc_handle_s)); if (*stc == NULL) { - STC_LOGE("Failed to create stc handle"); - return STC_ERROR_OUT_OF_MEMORY; + STC_LOGE("Failed to create stc handle"); //LCOV_EXCL_LINE + return STC_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } STC_LOGI("Create new handle [%p]", *stc); @@ -543,6 +551,7 @@ void _stc_callback_set_stats_total_info(stc_h stc, handle->stats_total_user_data = user_data; } +//LCOV_EXCL_START void _stc_callback_set_restriction_info(stc_h stc, stc_restriction_info_cb user_cb, void *user_data) { @@ -578,3 +587,4 @@ void _stc_callback_set_warn_threshold_crossed(stc_h stc, handle->warn_crossed_cb = user_cb; handle->warn_crossed_user_data = user_data; } +//LCOV_EXCL_STOP diff --git a/src/internal/stc-signal.c b/src/internal/stc-signal.c index 8f70b59..35fd543 100755 --- a/src/internal/stc-signal.c +++ b/src/internal/stc-signal.c @@ -63,6 +63,7 @@ static __thread guint gdbus_subscribe_id_stc_manager_restriction = 0; * Local Functions Definition *****************************************************************************/ +//LCOV_EXCL_START static void __stc_handle_stats(GVariant *param) { } @@ -130,6 +131,7 @@ static void __stc_restriction_signal_filter(GDBusConnection *conn, else ;//Do Nothing } +//LCOV_EXCL_STOP int _stc_register_signal(void) { @@ -166,10 +168,10 @@ int _stc_register_signal(void) if (gdbus_subscribe_id_stc_manager_stats == 0 || gdbus_subscribe_id_stc_manager_restriction == 0) { - STC_LOGE("Failed register signals stc_monitor[%d], stc_manager[%d]", + STC_LOGE("Failed register signals stc_monitor[%d], stc_manager[%d]", //LCOV_EXCL_LINE gdbus_subscribe_id_stc_manager_stats, gdbus_subscribe_id_stc_manager_restriction); - err = STC_ERROR_NOT_INITIALIZED; + err = STC_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } return err; @@ -182,8 +184,8 @@ int _stc_deregister_signal(void) connection = _stc_dbus_get_connection(); if (connection == NULL) { - STC_LOGE("Already de-registered"); - return STC_ERROR_NOT_INITIALIZED; + STC_LOGE("Already de-registered"); //LCOV_EXCL_LINE + return STC_ERROR_NOT_INITIALIZED; //LCOV_EXCL_LINE } g_dbus_connection_signal_unsubscribe(connection, diff --git a/src/stc-info.c b/src/stc-info.c index bdb081a..9a70063 100755 --- a/src/stc-info.c +++ b/src/stc-info.c @@ -104,6 +104,7 @@ EXPORT_API int stc_stats_info_get_iface_name(stc_stats_info_h info, char **iface return STC_ERROR_NONE; } +//LCOV_EXCL_START EXPORT_API int stc_stats_info_get_imsi(stc_stats_info_h info, char **imsi) { CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); @@ -124,6 +125,7 @@ EXPORT_API int stc_stats_info_get_imsi(stc_stats_info_h info, char **imsi) return STC_ERROR_NONE; } +//LCOV_EXCL_STOP EXPORT_API int stc_stats_info_get_time_interval(stc_stats_info_h info, time_t *from, time_t *to) @@ -252,6 +254,7 @@ EXPORT_API int stc_stats_info_get_process_state(stc_stats_info_h info, return STC_ERROR_NONE; } +//LCOV_EXCL_START EXPORT_API int stc_restriction_info_get_app_id(stc_restriction_info_h info, char **app_id) { CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); @@ -422,4 +425,4 @@ EXPORT_API int stc_restriction_info_get_roaming_type(stc_restriction_info_h info return STC_ERROR_NONE; } - +//LCOV_EXCL_STOP diff --git a/src/stc-manager.c b/src/stc-manager.c index ecbb759..65b39db 100755 --- a/src/stc-manager.c +++ b/src/stc-manager.c @@ -64,6 +64,7 @@ * Local Functions Definition *****************************************************************************/ +//LCOV_EXCL_START EXPORT_API int stc_reset_stats(stc_h stc, stc_reset_rule_h rule) { int ret = STC_ERROR_NONE; @@ -93,6 +94,7 @@ EXPORT_API int stc_reset_stats(stc_h stc, stc_reset_rule_h rule) return STC_ERROR_NONE; } +//LCOV_EXCL_STOP EXPORT_API int stc_get_stats(stc_h stc, stc_stats_rule_h rule, stc_stats_info_cb info_cb, void *user_data) @@ -102,20 +104,20 @@ EXPORT_API int stc_get_stats(stc_h stc, stc_stats_rule_h rule, CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); if (!(_stc_handle_check_validity(stc))) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER;//LCOV_EXCL_LINE } if (info_cb == NULL) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } ret = _stc_stats_check_get_rule(rule); if (ret != STC_ERROR_NONE) { - STC_LOGE("Invalid stats rule [%s]", + STC_LOGE("Invalid stats rule [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } _stc_callback_set_stats_info(stc, @@ -123,9 +125,9 @@ EXPORT_API int stc_get_stats(stc_h stc, stc_stats_rule_h rule, ret = _stc_stats_get(rule); if (ret != STC_ERROR_NONE) { - STC_LOGE("Failed to request stats info [%s]", + STC_LOGE("Failed to request stats info [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } return STC_ERROR_NONE; @@ -139,20 +141,20 @@ EXPORT_API int stc_foreach_stats(stc_h stc, stc_stats_rule_h rule, CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); if (!(_stc_handle_check_validity(stc))) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (info_cb == NULL) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } ret = _stc_stats_check_get_all_rule(rule); if (ret != STC_ERROR_NONE) { - STC_LOGE("Invalid stats rule [%s]", + STC_LOGE("Invalid stats rule [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } _stc_callback_set_stats_all_info(stc, @@ -160,9 +162,9 @@ EXPORT_API int stc_foreach_stats(stc_h stc, stc_stats_rule_h rule, ret = _stc_stats_get_all(rule); if (ret != STC_ERROR_NONE) { - STC_LOGE("Failed to request stats all info [%s]", + STC_LOGE("Failed to request stats all info [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } return STC_ERROR_NONE; @@ -176,20 +178,20 @@ EXPORT_API int stc_get_total_stats(stc_h stc, stc_stats_rule_h rule, CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); if (!(_stc_handle_check_validity(stc))) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (info_cb == NULL) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } ret = _stc_stats_check_get_total_rule(rule); if (ret != STC_ERROR_NONE) { - STC_LOGE("Invalid stats rule [%s]", + STC_LOGE("Invalid stats rule [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } _stc_callback_set_stats_total_info(stc, @@ -197,14 +199,15 @@ EXPORT_API int stc_get_total_stats(stc_h stc, stc_stats_rule_h rule, ret = _stc_stats_get_total(rule); if (ret != STC_ERROR_NONE) { - STC_LOGE("Failed to request stats total info [%s]", + STC_LOGE("Failed to request stats total info [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } return STC_ERROR_NONE; } +//LCOV_EXCL_START EXPORT_API int stc_set_restriction(stc_h stc, stc_restriction_rule_h rule) { @@ -462,4 +465,4 @@ EXPORT_API int stc_get_restriction_status(stc_h stc, stc_restriction_rule_h rule return STC_ERROR_NONE; } - +//LCOV_EXCL_STOP diff --git a/src/stc-reset.c b/src/stc-reset.c index 7dd18f2..7bb72a0 100755 --- a/src/stc-reset.c +++ b/src/stc-reset.c @@ -61,6 +61,7 @@ * Local Functions Definition *****************************************************************************/ +//LCOV_EXCL_START static void __stc_reset_init_rule(stc_reset_rule_s *rule) { rule->iface_type = STC_IFACE_UNKNOWN; @@ -435,4 +436,4 @@ EXPORT_API int stc_reset_rule_get_connection_period(stc_reset_rule_h rule, return STC_ERROR_NONE; } - +//LCOV_EXCL_STOP diff --git a/src/stc-restriction.c b/src/stc-restriction.c index 8475f93..516b304 100755 --- a/src/stc-restriction.c +++ b/src/stc-restriction.c @@ -62,6 +62,7 @@ * Local Functions Definition *****************************************************************************/ +//LCOV_EXCL_START static void __stc_restriction_init_rule(stc_restriction_rule_s *rule) { rule->app_id[0] = '\0'; @@ -1191,4 +1192,4 @@ EXPORT_API int stc_restriction_rule_get_imsi(stc_restriction_rule_h rule, char * return STC_ERROR_NONE; } - +//LCOV_EXCL_STOP diff --git a/src/stc-statistics.c b/src/stc-statistics.c index 8eab56c..74ab63c 100755 --- a/src/stc-statistics.c +++ b/src/stc-statistics.c @@ -200,10 +200,10 @@ static void __stc_stats_extract_info_cb(const char *key, g_strlcpy(stats_info->imsi, imsi, STC_IMSI_LEN); } else if (!g_strcmp0(key, STC_STATS_INFO_INTERVAL_FROM)) { - stats_info->interval.from = g_variant_get_uint64(value); + stats_info->interval.from = g_variant_get_uint64(value); //LCOV_EXCL_LINE } else if (!g_strcmp0(key, STC_STATS_INFO_INTERVAL_TO)) { - stats_info->interval.to = g_variant_get_uint64(value); + stats_info->interval.to = g_variant_get_uint64(value); //LCOV_EXCL_LINE } else if (!g_strcmp0(key, STC_STATS_INFO_OUTGOING_BYTES)) { stats_info->cnt.outgoing_bytes = g_variant_get_int64(value); @@ -231,6 +231,7 @@ static void __stc_stats_extract_info_cb(const char *key, } +//LCOV_EXCL_START static void __stc_stats_print_info(stc_stats_info_s *info) { STC_LOGD(STC_HR_SINGLE); @@ -264,6 +265,7 @@ static void __stc_stats_print_info(stc_stats_info_s *info) STC_LOGD(STC_HR_SINGLE); } +//LCOV_EXCL_STOP static void __stc_stats_get_per_app_id_reply( GObject *source_object, GAsyncResult *res, gpointer user_data) @@ -281,8 +283,8 @@ static void __stc_stats_get_per_app_id_reply( conn = G_DBUS_CONNECTION(source_object); dbus_data = g_dbus_connection_call_finish(conn, res, &dbus_error); if (dbus_error != NULL) { - error = _stc_convert_error_string_to_enum(dbus_error->message); - g_error_free(dbus_error); + error = _stc_convert_error_string_to_enum(dbus_error->message); //LCOV_EXCL_LINE + g_error_free(dbus_error); //LCOV_EXCL_LINE } if (_stc_event_get_request_table_state(STC_REQUEST_TABLE_STATS_GET)) @@ -311,6 +313,7 @@ static void __stc_stats_get_per_app_id_reply( event_data.datalength = g_slist_length(stats_list); event_data.error = dbus_result; } else { + //LCOV_EXCL_START STC_LOGE("Get stats per app id response error [%s]", _stc_convert_error_type_to_string(error)); @@ -318,6 +321,7 @@ static void __stc_stats_get_per_app_id_reply( event_data.info_data = NULL; event_data.datalength = 0; event_data.error = error; + //LCOV_EXCL_STOP } _stc_event_add_client_idle_cb(&event_data, user_data); @@ -371,6 +375,7 @@ static void __stc_stats_get_all_reply( event_data.datalength = g_slist_length(stats_list); event_data.error = dbus_result; } else { + //LCOV_EXCL_START STC_LOGE("Get stats all response error [%s]", _stc_convert_error_type_to_string(error)); @@ -378,6 +383,7 @@ static void __stc_stats_get_all_reply( event_data.info_data = NULL; event_data.datalength = 0; event_data.error = error; + //LCOV_EXCL_STOP } _stc_event_add_client_idle_cb(&event_data, user_data); @@ -399,8 +405,8 @@ static void __stc_stats_get_total_reply( conn = G_DBUS_CONNECTION(source_object); dbus_data = g_dbus_connection_call_finish(conn, res, &dbus_error); if (dbus_error != NULL) { - error = _stc_convert_error_string_to_enum(dbus_error->message); - g_error_free(dbus_error); + error = _stc_convert_error_string_to_enum(dbus_error->message); //LCOV_EXCL_LINE + g_error_free(dbus_error); //LCOV_EXCL_LINE } if (_stc_event_get_request_table_state(STC_REQUEST_TABLE_STATS_GET_TOTAL)) @@ -429,13 +435,15 @@ static void __stc_stats_get_total_reply( event_data.datalength = g_slist_length(stats_list); event_data.error = dbus_result; } else { - STC_LOGE("Get stats total response error [%s]", + //LCOV_EXCL_START + STC_LOGE("Get stats total response error [%s]", _stc_convert_error_type_to_string(error)); event_data.event = STC_EVENT_STATS_GET_TOTAL_RSP; event_data.info_data = NULL; event_data.datalength = 0; event_data.error = error; + //LCOV_EXCL_STOP } _stc_event_add_client_idle_cb(&event_data, user_data); @@ -469,11 +477,11 @@ stc_error_e __stc_stats_get_per_app_id( NULL); if (error != STC_ERROR_NONE) { - STC_LOGE("Failed to invoke dbus method nonblock"); + STC_LOGE("Failed to invoke dbus method nonblock"); //LCOV_EXCL_LINE - _stc_event_set_request_table_state( + _stc_event_set_request_table_state( //LCOV_EXCL_LINE STC_REQUEST_TABLE_STATS_GET, false); - return error; + return error; //LCOV_EXCL_LINE } return STC_ERROR_NONE; @@ -507,11 +515,11 @@ stc_error_e __stc_stats_get_all( NULL); if (error != STC_ERROR_NONE) { - STC_LOGE("Failed to invoke dbus method nonblock"); + STC_LOGE("Failed to invoke dbus method nonblock"); //LCOV_EXCL_LINE - _stc_event_set_request_table_state( + _stc_event_set_request_table_state( //LCOV_EXCL_LINE STC_REQUEST_TABLE_STATS_GET_ALL, false); - return error; + return error; //LCOV_EXCL_LINE } return STC_ERROR_NONE; @@ -545,11 +553,11 @@ stc_error_e __stc_stats_get_total( NULL); if (error != STC_ERROR_NONE) { - STC_LOGE("Failed to invoke dbus method nonblock"); + STC_LOGE("Failed to invoke dbus method nonblock"); //LCOV_EXCL_LINE - _stc_event_set_request_table_state( + _stc_event_set_request_table_state( //LCOV_EXCL_LINE STC_REQUEST_TABLE_STATS_GET_TOTAL, false); - return error; + return error; //LCOV_EXCL_LINE } return STC_ERROR_NONE; @@ -651,19 +659,19 @@ EXPORT_API int stc_stats_rule_create(stc_h stc, stc_stats_rule_h *rule) CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); if (!(_stc_handle_check_validity(stc))) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (rule == NULL) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } stc_stats_rule_s *stats_rule = g_try_malloc0(sizeof(stc_stats_rule_s)); if (stats_rule == NULL) { - STC_LOGE("Memory allocation failed"); - return STC_ERROR_OUT_OF_MEMORY; + STC_LOGE("Memory allocation failed"); //LCOV_EXCL_LINE + return STC_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE } __stc_stats_init_rule(stats_rule); @@ -701,7 +709,7 @@ EXPORT_API int stc_stats_rule_set_app_id(stc_stats_rule_h rule, stc_stats_rule_s *stats_rule = (stc_stats_rule_s *)rule; if (app_id == NULL) - stats_rule->app_id[0] = '\0'; + stats_rule->app_id[0] = '\0'; //LCOV_EXCL_LINE else g_strlcpy(stats_rule->app_id, app_id, strlen(app_id) + 1); @@ -721,8 +729,8 @@ EXPORT_API int stc_stats_rule_set_time_interval(stc_stats_rule_h rule, } if (from < 0 || to < 0) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } stc_stats_rule_s *stats_rule = (stc_stats_rule_s *)rule; @@ -757,8 +765,8 @@ EXPORT_API int stc_stats_rule_set_iface_type( stats_rule->iface_type = iface_type; break; default: - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } STC_LOGI("Iface type [%s]", @@ -788,8 +796,8 @@ EXPORT_API int stc_stats_rule_set_time_period(stc_stats_rule_h rule, stats_rule->time_period = time_period; break; default: - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } STC_LOGI("Time period [%s]", diff --git a/src/stc.c b/src/stc.c index e11b58c..bd6136b 100755 --- a/src/stc.c +++ b/src/stc.c @@ -63,27 +63,27 @@ EXPORT_API int stc_initialize(stc_h *stc) CHECK_FEATURE_SUPPORTED(TIZEN_FEATURE_STC); if (stc == NULL) { - STC_LOGE("Invalid parameter"); - return STC_ERROR_INVALID_PARAMETER; + STC_LOGE("Invalid parameter"); //LCOV_EXCL_LINE + return STC_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE } if (_stc_handle_check_validity(*stc)) { - STC_LOGE("Already initialized"); - return STC_ERROR_ALREADY_INITIALIZED; + STC_LOGE("Already initialized"); //LCOV_EXCL_LINE + return STC_ERROR_ALREADY_INITIALIZED; //LCOV_EXCL_LINE } ret = _stc_initialize(); if (ret != STC_ERROR_NONE) { - STC_LOGE("Init failed [%s]", + STC_LOGE("Init failed [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } ret = _stc_handle_create(stc); if (ret != STC_ERROR_NONE) { - STC_LOGE("Create handle failed [%s]", + STC_LOGE("Create handle failed [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } _stc_handle_add(stc); @@ -107,9 +107,9 @@ EXPORT_API int stc_deinitialize(stc_h stc) if (_stc_handle_get_count() == 0) { int ret = _stc_deinitialize(); if (ret != STC_ERROR_NONE) { - STC_LOGE("Deinit failed [%s]", + STC_LOGE("Deinit failed [%s]", //LCOV_EXCL_LINE _stc_convert_error_type_to_string(ret)); - return ret; + return ret; //LCOV_EXCL_LINE } }