[UTC][radio][Non-ACR][Fixed memory leak]
authorGilbok Lee <gilbok.lee@samsung.com>
Wed, 15 Jun 2016 08:07:18 +0000 (17:07 +0900)
committerGilbok Lee <gilbok.lee@samsung.com>
Thu, 16 Jun 2016 05:24:39 +0000 (22:24 -0700)
Change-Id: Idd8319cfd366e1c808e514133242ae28e46d7870
Signed-off-by: Gilbok Lee <gilbok.lee@samsung.com>
src/utc/radio/utc-media-radio.c

index 832ae96..87e2dce 100755 (executable)
 // limitations under the License.
 //
 #include <media/radio.h>
+#include <glib.h>
 #include "assert.h"
 
-
 radio_h radio;
 int sRet;
-static bool radio_supported=true;
+static bool radio_supported = true;
+static GMainLoop *g_mainloop = NULL;
+
+static gboolean timeout_func(gpointer data)
+{
+       fprintf(stderr, "waited 2 seconds\n");
+       g_main_loop_quit((GMainLoop *)data);
+       return FALSE;
+}
+
+static void wait_for_async()
+{
+       GMainContext *context = g_main_context_new();
+       GSource *source = g_timeout_source_new(2000);
+
+       /* attach source to context */
+       g_source_attach(source, context);
+
+       g_mainloop = g_main_loop_new(context, FALSE);
+
+       /* set the callback for this source */
+       g_source_set_callback(source, timeout_func, g_mainloop, NULL);
+
+       g_main_loop_run(g_mainloop);
+
+       /* after attaching with the GSource, destroy() is needed */
+       g_source_destroy(source);
+
+       /* unref the last reference we got for GSource */
+       g_source_unref(source);
+
+       /* main loop should be destroyed before unref the context */
+       g_main_loop_unref(g_mainloop);
+       g_main_context_unref(context);
+}
 
 static void radio_completed_cb(int frequency, void *data)
 {
@@ -39,10 +73,8 @@ static void radio_interrupt_cb(radio_interrupted_code_e code, void *data)
  */
 void utc_media_radio_startup(void)
 {
-    system_info_get_platform_bool("http://tizen.org/feature/fmradio", &radio_supported);
-
-    sRet = radio_create(&radio);
-
+       system_info_get_platform_bool("http://tizen.org/feature/fmradio", &radio_supported);
+       sRet = radio_create(&radio);
 }
 
 /**
@@ -53,9 +85,8 @@ void utc_media_radio_startup(void)
  */
 void utc_media_radio_cleanup(void)
 {
-       if (radio) {
-           radio_destroy(radio);
-       }
+       if (radio)
+               radio_destroy(radio);
 }
 
 /**
@@ -68,19 +99,16 @@ int utc_media_radio_get_state_p(void)
        int ret;
        radio_state_e state;
 
-       if(radio_supported){
-          assert_eq(sRet,RADIO_ERROR_NONE);
-
-          ret = radio_get_state(radio, &state);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-
-          ret= radio_get_state(radio, &state);
-          assert_eq(ret,RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_state(radio, &state);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_get_state(radio, &state);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
 
-    return 0;
+       return 0;
 }
 
 /**
@@ -93,19 +121,15 @@ int utc_media_radio_get_state_n(void)
        int ret;
        radio_state_e state;
 
-       if(radio_supported){
-            assert_eq(sRet,RADIO_ERROR_NONE);
-
-           ret = radio_get_state(NULL, &state);
-            assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-
-           ret = radio_get_state(NULL, &state);
-            assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_state(NULL, &state);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_get_state(NULL, &state);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-       return 0;
+       return 0;
 }
 
 /**
@@ -117,21 +141,17 @@ int utc_media_radio_start_p(void)
 {
        int ret;
 
-       if(radio_supported){
-            assert_eq(sRet,RADIO_ERROR_NONE);
-           ret = radio_start(radio);
-
-           assert_eq(ret, RADIO_ERROR_NONE);
-
-           radio_stop(radio);
-               }
-       else{
-
-           ret = radio_start(radio);
-            assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_start(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               ret = radio_stop(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_start(radio);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-
-       return 0;
+       return 0;
 }
 
 /**
@@ -143,17 +163,15 @@ int utc_media_radio_start_n(void)
 {
        int ret;
 
-       if(radio_supported){
-            assert_eq(sRet,RADIO_ERROR_NONE);
-           ret = radio_start(NULL);
-            assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-           ret = radio_start(NULL);
-            assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_start(NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_start(NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-       return 0;
+       return 0;
 }
 
 /**
@@ -164,19 +182,15 @@ int utc_media_radio_start_n(void)
 int utc_media_radio_set_frequency_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_frequency(radio, 91900);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-
-          ret = radio_set_frequency(radio, 91900);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_frequency(radio, 91900);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_set_frequency(radio, 91900);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
-
+       return 0;
 }
 
 /**
@@ -187,20 +201,15 @@ int utc_media_radio_set_frequency_p(void)
 int utc_media_radio_set_frequency_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-
-          ret = radio_set_frequency(radio, 100);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-
-       }
-       else{
-
-          ret = radio_set_frequency(radio, 100);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_frequency(radio, 100);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_set_frequency(radio, 100);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-       return 0;
+       return 0;
 }
 
 /**
@@ -213,16 +222,13 @@ int utc_media_radio_get_frequency_p(void)
        int ret;
        int frequency;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-                  ret = radio_get_frequency(radio, &frequency);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-
-          ret = radio_get_frequency(radio, &frequency);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_frequency(radio, &frequency);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_get_frequency(radio, &frequency);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
        return 0;
 }
@@ -235,17 +241,16 @@ int utc_media_radio_get_frequency_p(void)
 int utc_media_radio_get_frequency_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_get_frequency(NULL, NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_get_frequency(NULL, NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_frequency(NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_get_frequency(NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
 
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -257,22 +262,21 @@ int utc_media_radio_seek_up_p(void)
 {
        int ret;
 
-
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          radio_start(radio);
-          ret = radio_seek_up(radio,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-          radio_stop(radio);
-
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_start(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               ret = radio_seek_up(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               wait_for_async();
+               ret = radio_stop(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_seek_up(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-       else{
-
-          ret = radio_seek_up(radio,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
 
-       }
-        return 0;
+       return 0;
 }
 
 /**
@@ -283,22 +287,20 @@ int utc_media_radio_seek_up_p(void)
 int utc_media_radio_seek_up_n(void)
 {
        int ret;
-       if(radio_supported){
-          radio_start(radio);
-          assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_seek_up(NULL,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-          radio_stop(radio);
-       }
-       else{
-          ret = radio_seek_up(NULL,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-
+       if (radio_supported) {
+               ret = radio_start(radio);
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_seek_up(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+               ret = radio_stop(radio);
+               assert_eq(sRet, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_seek_up(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-        return 0;
+       return 0;
 }
 
-
 /**
  * @testcase           utc_media_radio_seek_down_p
  * @since_tizen                2.3
@@ -308,16 +310,18 @@ int utc_media_radio_seek_down_p(void)
 {
        int ret;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          radio_start(radio);
-          ret = radio_seek_down(radio,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-          radio_stop(radio);
-       }
-       else{
-          ret = radio_seek_down(radio,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_start(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               ret = radio_seek_down(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               wait_for_async();
+               ret = radio_stop(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_seek_down(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
        return 0;
 }
@@ -331,19 +335,20 @@ int utc_media_radio_seek_down_n(void)
 {
        int ret;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-
-           ret = radio_seek_down(radio,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_STATE);
-        }
-        else{
-
-          ret = radio_seek_down(radio,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
-        }
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_start(radio);
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_seek_down(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+               ret = radio_stop(radio);
+               assert_eq(sRet, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_seek_down(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       }
 
-    return 0;
+       return 0;
 }
 
 /**
@@ -354,19 +359,17 @@ int utc_media_radio_seek_down_n(void)
 int utc_media_radio_scan_start_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          radio_stop(radio);
-          ret = radio_scan_start(radio,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-          radio_scan_stop(radio,NULL,NULL);
-       }
-       else{
-          radio_stop(radio);
-           ret = radio_scan_start(radio,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_scan_start(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               radio_scan_stop(radio, NULL, NULL);
+       } else {
+               radio_stop(radio);
+               ret = radio_scan_start(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -378,17 +381,16 @@ int utc_media_radio_scan_start_n(void)
 {
        int ret;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_scan_start(NULL,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_scan_start(NULL,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_scan_start(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_scan_start(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
 
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -400,21 +402,17 @@ int utc_media_radio_scan_stop_p(void)
 {
        int ret;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_scan_start(radio,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-
-          ret = radio_scan_stop(radio,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-          radio_stop(radio);
-       }
-       else{
-          ret = radio_scan_stop(radio,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_scan_start(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               radio_scan_stop(radio, NULL, NULL);
+       } else {
+               ret = radio_scan_stop(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
 
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -426,17 +424,16 @@ int utc_media_radio_scan_stop_n(void)
 {
        int ret;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_scan_stop(NULL,NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_scan_stop(NULL,NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_scan_stop(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_scan_stop(NULL, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
 
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -448,17 +445,16 @@ int utc_media_radio_set_mute_p(void)
 {
        int ret;
        bool mute = true;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_mute(radio, mute);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_set_mute(radio, mute);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_mute(radio, mute);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_set_mute(radio, mute);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
 
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -469,16 +465,15 @@ int utc_media_radio_set_mute_p(void)
 int utc_media_radio_set_mute_n(void)
 {
        int ret;
-        if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_mute(NULL,true);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_set_mute(NULL,true);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_mute(NULL, true);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_set_mute(NULL, true);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -491,17 +486,15 @@ int utc_media_radio_is_muted_p(void)
        int ret;
        bool mute;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-
-          ret = radio_is_muted(radio,&mute);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_is_muted(radio,&mute);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_is_muted(radio, &mute);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_is_muted(radio, &mute);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -512,16 +505,15 @@ int utc_media_radio_is_muted_p(void)
 int utc_media_radio_is_muted_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_is_muted(NULL,NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_is_muted(NULL,NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_is_muted(NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_is_muted(NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -532,16 +524,15 @@ int utc_media_radio_is_muted_n(void)
 int utc_media_radio_set_scan_completed_cb_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_scan_completed_cb(radio, radio_completed_cb, NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_set_scan_completed_cb(radio, radio_completed_cb, NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_scan_completed_cb(radio, radio_completed_cb, NULL);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_set_scan_completed_cb(radio, radio_completed_cb, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -552,16 +543,15 @@ int utc_media_radio_set_scan_completed_cb_p(void)
 int utc_media_radio_set_scan_completed_cb_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_scan_completed_cb(radio, NULL, NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_set_scan_completed_cb(radio, NULL, NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_scan_completed_cb(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_set_scan_completed_cb(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -572,16 +562,15 @@ int utc_media_radio_set_scan_completed_cb_n(void)
 int utc_media_radio_unset_scan_completed_cb_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_unset_scan_completed_cb(radio);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_unset_scan_completed_cb(radio);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_unset_scan_completed_cb(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_unset_scan_completed_cb(radio);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -592,16 +581,15 @@ int utc_media_radio_unset_scan_completed_cb_p(void)
 int utc_media_radio_unset_scan_completed_cb_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_unset_scan_completed_cb(NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_unset_scan_completed_cb(NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_unset_scan_completed_cb(NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_unset_scan_completed_cb(NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -612,16 +600,15 @@ int utc_media_radio_unset_scan_completed_cb_n(void)
 int utc_media_radio_set_interrupted_cb_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_interrupted_cb(radio, radio_interrupt_cb, NULL);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_set_interrupted_cb(radio, radio_interrupt_cb, NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_interrupted_cb(radio, radio_interrupt_cb, NULL);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_set_interrupted_cb(radio, radio_interrupt_cb, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -632,16 +619,15 @@ int utc_media_radio_set_interrupted_cb_p(void)
 int utc_media_radio_set_interrupted_cb_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_set_interrupted_cb(radio, NULL, NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_set_interrupted_cb(radio, NULL, NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_set_interrupted_cb(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_set_interrupted_cb(radio, NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -652,16 +638,15 @@ int utc_media_radio_set_interrupted_cb_n(void)
 int utc_media_radio_unset_interrupted_cb_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_unset_interrupted_cb(radio);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_unset_interrupted_cb(radio);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_unset_interrupted_cb(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_unset_interrupted_cb(radio);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -672,16 +657,15 @@ int utc_media_radio_unset_interrupted_cb_p(void)
 int utc_media_radio_unset_interrupted_cb_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_unset_interrupted_cb(NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_unset_interrupted_cb(NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_unset_interrupted_cb(NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_unset_interrupted_cb(NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -693,16 +677,15 @@ int utc_media_radio_get_signal_strength_p(void)
 {
        int strength = 0;
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_get_signal_strength(radio, &strength);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_get_signal_strength(radio, &strength);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_signal_strength(radio, &strength);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_get_signal_strength(radio, &strength);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -713,16 +696,15 @@ int utc_media_radio_get_signal_strength_p(void)
 int utc_media_radio_get_signal_strength_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_get_signal_strength(NULL, NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_get_signal_strength(NULL, NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_signal_strength(NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_get_signal_strength(NULL, NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-    return 0;
+       return 0;
 }
 
 /**
@@ -733,20 +715,20 @@ int utc_media_radio_get_signal_strength_n(void)
 int utc_media_radio_stop_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          radio_state_e state;
-          radio_get_state(radio, &state);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               radio_state_e state;
+               radio_get_state(radio, &state);
 
-          if (state != RADIO_STATE_PLAYING) radio_start(radio);
-           ret = radio_stop(radio);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_stop(radio);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+               if (state != RADIO_STATE_PLAYING)
+                       radio_start(radio);
+               ret = radio_stop(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_stop(radio);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-        return 0;
+       return 0;
 }
 
 /**
@@ -757,16 +739,15 @@ int utc_media_radio_stop_p(void)
 int utc_media_radio_stop_n(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_stop(NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
-          ret = radio_stop(NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_stop(NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
+               ret = radio_stop(NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-       return 0;
+       return 0;
 }
 
 /**
@@ -777,18 +758,16 @@ int utc_media_radio_stop_n(void)
 int utc_media_radio_destroy_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_destroy(radio);
-          assert_eq(ret, RADIO_ERROR_NONE);
-          radio = NULL;
-       }
-       else{
-          ret = radio_destroy(radio);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_destroy(radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+               radio = NULL;
+       } else {
+               ret = radio_destroy(radio);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-
-    return 0;
+       return 0;
 }
 
 /**
@@ -800,18 +779,15 @@ int utc_media_radio_destroy_n(void)
 {
        int ret;
 
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_destroy(NULL);
-          assert_neq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_destroy(NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_destroy(NULL);
+               assert_neq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_destroy(NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-
-
-    return 0;
+       return 0;
 }
 
 /**
@@ -822,19 +798,17 @@ int utc_media_radio_destroy_n(void)
 int utc_media_radio_create_p(void)
 {
        int ret;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-
-          if (radio) radio_destroy(radio);
-          ret = radio_create(&radio);
-          assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_create(&radio);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               if (radio)
+                       radio_destroy(radio);
+               ret = radio_create(&radio);
+               assert_eq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_create(&radio);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-
-    return 0;
+       return 0;
 }
 
 /**
@@ -845,17 +819,15 @@ int utc_media_radio_create_p(void)
 int utc_media_radio_create_n(void)
 {
        int ret;
-        if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_create(NULL);
-          assert_neq(ret, RADIO_ERROR_NONE);
-       }
-       else{
-          ret = radio_create(NULL);
-           assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_create(NULL);
+               assert_neq(ret, RADIO_ERROR_NONE);
+       } else {
+               ret = radio_create(NULL);
+               assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
-
-    return 0;
+       return 0;
 }
 
 /**
@@ -868,12 +840,11 @@ int utc_media_radio_get_frequency_range_p(void)
        int ret;
        int min, max;
 
-       if(radio_supported){
-               assert_eq(sRet,RADIO_ERROR_NONE);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
                ret = radio_get_frequency_range(radio, &min, &max);
                assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
+       } else {
                ret = radio_get_frequency_range(radio, &min, &max);
                assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
@@ -889,12 +860,11 @@ int utc_media_radio_get_frequency_range_n(void)
 {
        int ret;
        int min, max;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_get_frequency_range(NULL, &min, &max);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_frequency_range(NULL, &min, &max);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
                ret = radio_get_frequency_range(NULL, &min, &max);
                assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
@@ -910,13 +880,11 @@ int utc_media_radio_get_channel_spacing_p(void)
 {
        int ret;
        int spacing;
-
-       if(radio_supported){
-               assert_eq(sRet,RADIO_ERROR_NONE);
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
                ret = radio_get_channel_spacing(radio, &spacing);
                assert_eq(ret, RADIO_ERROR_NONE);
-       }
-       else{
+       } else {
                ret = radio_get_channel_spacing(radio, &spacing);
                assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }
@@ -932,12 +900,11 @@ int utc_media_radio_get_channel_spacing_n(void)
 {
        int ret;
        int spacing;
-       if(radio_supported){
-           assert_eq(sRet,RADIO_ERROR_NONE);
-          ret = radio_get_channel_spacing(radio, NULL);
-          assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
-       }
-       else{
+       if (radio_supported) {
+               assert_eq(sRet, RADIO_ERROR_NONE);
+               ret = radio_get_channel_spacing(radio, NULL);
+               assert_eq(ret, RADIO_ERROR_INVALID_PARAMETER);
+       } else {
                ret = radio_get_channel_spacing(radio, NULL);
                assert_eq(ret, RADIO_ERROR_NOT_SUPPORTED);
        }