Fix coverity issue

[platform/core/system/sensord.git] / src / sensorctl / testcase / sensor_listener.cpp

diff --git a/src/sensorctl/testcase/sensor_listener.cpp b/src/sensorctl/testcase/sensor_listener.cpp
index 70050b0..ac3b12f 100644 (file)
--- a/src/sensorctl/testcase/sensor_listener.cpp
+++ b/src/sensorctl/testcase/sensor_listener.cpp
@@ -59,6 +59,7 @@ TESTCASE(sensor_listener, get_sensors_p_1)
 
        err = sensord_get_sensors(ACCELEROMETER_SENSOR, &sensors, &count);
        ASSERT_EQ(err, 0);
+       ASSERT_FREE((count < 0), sensors);
        ASSERT_GT(count, 0);
 
        free(sensors);
@@ -105,26 +106,33 @@ TESTCASE(sensor_listener, all_api_p_1)
        ASSERT_EQ(err, 0);
 
        ret = sensord_register_event(handle, 1, 100, 100, event_cb, NULL);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        ret = sensord_start(handle, 0);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        ret = sensord_change_event_interval(handle, 0, 100);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        ret = sensord_change_event_max_batch_latency(handle, 0, 100);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        mainloop::run();
 
        ret = sensord_stop(handle);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        ret = sensord_unregister_event(handle, 1);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        ret = sensord_disconnect(handle);
+       ASSERT_FREE(((ret != true) && list), list);
        ASSERT_TRUE(ret);
 
        free(list);
@@ -278,6 +286,36 @@ TESTCASE(sensor_listener, set_get_attribute_int_2)
        return true;
 }
 
+TESTCASE(sensor_listener, set_get_attribute_int_3)
+{
+       int err = 0;
+       bool ret = true;
+       int handle = 0;
+       sensor_t sensor = NULL;
+       int attr = 20;
+       int value = -1;
+
+       err = sensord_get_default_sensor(ACCELEROMETER_SENSOR, &sensor);
+       ASSERT_EQ(err, 0);
+
+       handle = sensord_connect(sensor);
+
+       err = sensord_set_attribute_int(handle, attr, 1);
+       ASSERT_EQ(err, 0);
+
+       for (int i = 0 ; i < 10; i ++) {
+               err = sensord_get_attribute_int(handle, attr, &value);
+               ASSERT_EQ(err, 0);
+       }
+
+       ASSERT_EQ(value, 1);
+
+       ret = sensord_disconnect(handle);
+       ASSERT_TRUE(ret);
+
+       return true;
+}
+
 TESTCASE(sensor_listener, get_attribute_int_1)
 {
        int err = 0;
@@ -386,6 +424,37 @@ TESTCASE(sensor_listener, set_get_attribute_string_2)
        return true;
 }
 
+TESTCASE(sensor_listener, set_get_attribute_string_3)
+{
+       int err = 0;
+       bool ret = true;
+       int handle = 0;
+       char *value = NULL;
+       int len = 0;
+       sensor_t sensor = NULL;
+       int attr = 1;
+
+       err = sensord_get_default_sensor(ACCELEROMETER_SENSOR, &sensor);
+       ASSERT_EQ(err, 0);
+
+       handle = sensord_connect(sensor);
+       err = sensord_set_attribute_str(handle, attr, TEST_STRING, TEST_STRING_LEN);
+       ASSERT_EQ(err, 0);
+
+       for (int i = 0; i < 10; i++) {
+               err = sensord_get_attribute_str(handle, attr, &value, &len);
+               ASSERT_EQ(err, 0);
+               ASSERT_EQ(len, TEST_STRING_LEN);
+               ASSERT_EQ(strncmp(value, TEST_STRING, len), 0);
+       }
+
+       ret = sensord_disconnect(handle);
+       ASSERT_TRUE(ret);
+
+       free(value);
+       return true;
+}
+
 TESTCASE(sensor_listener, set_get_get_attribute_string_1)
 {
        int err;
@@ -627,7 +696,7 @@ TESTCASE(skip_sensor_listener, register_attribute_str_changed)
        ret = sensord_stop(handle);
        ASSERT_TRUE(ret);
 
-       ret = sensord_unregister_attribute_int_changed_cb(handle);
+       ret = sensord_unregister_attribute_str_changed_cb(handle);
        ASSERT_TRUE(ret);
 
        ret = sensord_disconnect(handle);
@@ -687,4 +756,53 @@ TESTCASE(skip_sensor_listener, attribute_str_changer)
        ASSERT_TRUE(ret);
 
        return true;
-}
\ No newline at end of file
+}
+
+TESTCASE(skip_sensor_listener, light_sensor_set_attribute_int_1)
+{
+       int err = 0;
+       bool ret = true;
+       int handle = 0;
+       sensor_t sensor = NULL;
+       int attr = 2;
+       int value = 0;
+
+       err = sensord_get_default_sensor(LIGHT_SENSOR, &sensor);
+       ASSERT_EQ(err, 0);
+
+       handle = sensord_connect(sensor);
+       err = sensord_set_attribute_int(handle, attr, 2);
+       ASSERT_EQ(err, 0);
+
+       err = sensord_get_attribute_int(handle, attr, &value);
+       ASSERT_EQ(err, 0);
+       ASSERT_EQ(value, 2);
+
+       ret = sensord_disconnect(handle);
+       ASSERT_TRUE(ret);
+
+       return true;
+}
+
+TESTCASE(skip_sensor_listener, light_sensor_get_attribute_int_1)
+{
+       int err = 0;
+       bool ret = true;
+       int handle = 0;
+       sensor_t sensor = NULL;
+       int attr = 2;
+       int value = 0;
+
+       err = sensord_get_default_sensor(LIGHT_SENSOR, &sensor);
+       ASSERT_EQ(err, 0);
+
+       handle = sensord_connect(sensor);
+       err = sensord_get_attribute_int(handle, attr, &value);
+       ASSERT_EQ(err, 0);
+       ASSERT_EQ(value, 2);
+
+       ret = sensord_disconnect(handle);
+       ASSERT_TRUE(ret);
+
+       return true;
+}