Test case Description Correctionsubmit/tizen/20180821.085626accepted/tizen/unified/20180823.071931

Change-Id: I0bea2ea2d16d14aff8d17c604de87f9a94feb75f

6 files changed, 134 insertions, 192 deletions

diff --git a/testcase/accel_device_haltest.cpp b/testcase/accel_device_haltest.cpp
index 149ffe3..fdfd036 100755
--- a/testcase/accel_device_haltest.cpp
+++ b/testcase/accel_device_haltest.cpp
@@ -89,8 +89,8 @@ public:
  * @type    		auto
  * @description		Positive, get fd of sensor device
  * @apicovered		get_poll_fd
- * @passcase		returns valid node handle value
- * @failcase		returns invalid node handle value
+ * @passcase		Returns valid node handle value
+ * @failcase		Returns invalid node handle value
  * @precondition	None
  * @postcondition	None
  */
@@ -102,15 +102,13 @@ TEST_F(AccelSensorHalTest, get_poll_fdP)
 	EXPECT_NE(node_handle, 0);
 }
 
-
-
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Get sensor data
  * @apicovered		get_sensors
  * @passcase		get valid sensor info
  * @failcase		get invalid sensor info
@@ -127,12 +125,12 @@ TEST_F(AccelSensorHalTest, get_sensorsP)
 }
 
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Get sensor data
  * @apicovered		get_sensors
  * @passcase		when invalid parameter passed, returns invalid parameter error
  * @failcase		when invalid parameter passed, doesn't return invalid paramter error
@@ -147,15 +145,13 @@ TEST_F(AccelSensorHalTest, get_sensorsN)
 
 }
 
-
-
 /**
  * @testcase		enableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Enable Sensor
  * @apicovered		enable
  * @passcase		when valid id passed, returns 1
  * @failcase		when valid id passed, doesn't returns 1
@@ -169,8 +165,6 @@ TEST_F(AccelSensorHalTest, enableP)
 	uint32_t id = 1;
 	res = sensor_handle->enable(id);
 	EXPECT_EQ(res, true);
-
-
 }
 
 /**
@@ -179,7 +173,7 @@ TEST_F(AccelSensorHalTest, enableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Enable Sensor
  * @apicovered		enable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -200,15 +194,13 @@ TEST_F(AccelSensorHalTest, enableN)
 
 }
 
-
-
 /**
  * @testcase		disableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Disable Sensor
  * @apicovered		disable
  * @passcase		when valid id passed, returns true
  * @failcase		when valid id passed, doesn't returns true
@@ -231,8 +223,8 @@ TEST_F(AccelSensorHalTest, disableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		disabel
+ * @description		Negative, Disable Sensor
+ * @apicovered		disable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
  * @precondition	None
@@ -257,7 +249,7 @@ TEST_F(AccelSensorHalTest, disableN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		get valid id and returns other than zero
  * @failcase		get invalid id and returns other zero
@@ -277,12 +269,12 @@ TEST_F(AccelSensorHalTest, read_fdP)
 }
 
 /**
- * @testcase		get_dataN
+ * @testcase		read_fdN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -294,8 +286,6 @@ TEST_F(AccelSensorHalTest, read_fdN)
 	SENSOR_SUPPORT_CHECK;
 	ret = sensor_handle->read_fd(nullptr);
 	EXPECT_EQ(ret, SENSOR_ERROR_INVALID_PARAMETER);
-
-
 }
 
 /**
@@ -304,7 +294,7 @@ TEST_F(AccelSensorHalTest, read_fdN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Gets Sensor data and data size
  * @apicovered		get_data
  * @passcase		Gets valid data and length and returns 0
  * @failcase		Doesn't get valid or length or doesn't return 0
@@ -324,13 +314,13 @@ TEST_F(AccelSensorHalTest, get_dataP)
 }
 
 /**
- * @testcase		read_fdN
+ * @testcase		get_dataN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type....		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		read_fd
+ * @description		Negative, Gets Sensor data and data size
+ * @apicovered		get_data
  * @passcase		Returns invalid parameter error
  * @failcase		Doesn't returns invalid parameter error
  * @precondition	None
@@ -361,7 +351,7 @@ TEST_F(AccelSensorHalTest, get_dataN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		when innterval is set, it returns true
  * @failcase		Returns false
@@ -385,7 +375,7 @@ TEST_F(AccelSensorHalTest, set_intervalP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		Returns Invalid paramter error
  * @failcase		Doesn't returns Invalid paramter error
diff --git a/testcase/gyro_device_haltest.cpp b/testcase/gyro_device_haltest.cpp
index f214628..de7544b 100755
--- a/testcase/gyro_device_haltest.cpp
+++ b/testcase/gyro_device_haltest.cpp
@@ -62,7 +62,7 @@ public:
 		sensor_supported = false;
 		sensor_handle = new gyro_device;
 		if (sensor_handle == NULL) {
-			cout << "accel sensor init failed " << endl;
+			cout << "gyro sensor init failed " << endl;
 			return;
 		}
 		sensor_supported = true;
@@ -91,8 +91,8 @@ public:
  * @type    		auto
  * @description		Positive, get fd of sensor device
  * @apicovered		get_poll_fd
- * @passcase		returns valid node handle value
- * @failcase		returns invalid node handle value
+ * @passcase		Returns valid node handle value
+ * @failcase		Returns invalid node handle value
  * @precondition	None
  * @postcondition	None
  */
@@ -104,15 +104,13 @@ TEST_F(GyroSensorHalTest, get_poll_fdP)
 	EXPECT_NE(node_handle, 0);
 }
 
-
-
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Get sensor data
  * @apicovered		get_sensors
  * @passcase		get valid sensor info
  * @failcase		get invalid sensor info
@@ -129,12 +127,12 @@ TEST_F(GyroSensorHalTest, get_sensorsP)
 }
 
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Get sensor data
  * @apicovered		get_sensors
  * @passcase		when invalid parameter passed, returns invalid parameter error
  * @failcase		when invalid parameter passed, doesn't return invalid paramter error
@@ -149,15 +147,13 @@ TEST_F(GyroSensorHalTest, get_sensorsN)
 
 }
 
-
-
 /**
  * @testcase		enableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Enable Sensor
  * @apicovered		enable
  * @passcase		when valid id passed, returns 1
  * @failcase		when valid id passed, doesn't returns 1
@@ -171,8 +167,6 @@ TEST_F(GyroSensorHalTest, enableP)
 	uint32_t id = 1;
 	res = sensor_handle->enable(id);
 	EXPECT_EQ(res, true);
-
-
 }
 
 /**
@@ -181,7 +175,7 @@ TEST_F(GyroSensorHalTest, enableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Enable Sensor
  * @apicovered		enable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -202,15 +196,13 @@ TEST_F(GyroSensorHalTest, enableN)
 
 }
 
-
-
 /**
  * @testcase		disableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Disable Sensor
  * @apicovered		disable
  * @passcase		when valid id passed, returns true
  * @failcase		when valid id passed, doesn't returns true
@@ -233,8 +225,8 @@ TEST_F(GyroSensorHalTest, disableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		disabel
+ * @description		Negative, Disable Sensor
+ * @apicovered		disable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
  * @precondition	None
@@ -259,7 +251,7 @@ TEST_F(GyroSensorHalTest, disableN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		get valid id and returns other than zero
  * @failcase		get invalid id and returns other zero
@@ -279,12 +271,12 @@ TEST_F(GyroSensorHalTest, read_fdP)
 }
 
 /**
- * @testcase		get_dataN
+ * @testcase		read_fdN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -296,8 +288,6 @@ TEST_F(GyroSensorHalTest, read_fdN)
 	SENSOR_SUPPORT_CHECK;
 	ret = sensor_handle->read_fd(nullptr);
 	EXPECT_EQ(ret, SENSOR_ERROR_INVALID_PARAMETER);
-
-
 }
 
 /**
@@ -306,7 +296,7 @@ TEST_F(GyroSensorHalTest, read_fdN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Gets Sensor data and data size
  * @apicovered		get_data
  * @passcase		Gets valid data and length and returns 0
  * @failcase		Doesn't get valid or length or doesn't return 0
@@ -326,13 +316,13 @@ TEST_F(GyroSensorHalTest, get_dataP)
 }
 
 /**
- * @testcase		read_fdN
+ * @testcase		get_dataN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
- * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		read_fd
+ * @type....		auto
+ * @description		Negative, Gets Sensor data and data size
+ * @apicovered		get_data
  * @passcase		Returns invalid parameter error
  * @failcase		Doesn't returns invalid parameter error
  * @precondition	None
@@ -363,7 +353,7 @@ TEST_F(GyroSensorHalTest, get_dataN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		when innterval is set, it returns true
  * @failcase		Returns false
@@ -387,7 +377,7 @@ TEST_F(GyroSensorHalTest, set_intervalP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		Returns Invalid paramter error
  * @failcase		Doesn't returns Invalid paramter error
diff --git a/testcase/hrm_device_haltest.cpp b/testcase/hrm_device_haltest.cpp
index cb23429..5c2e364 100755
--- a/testcase/hrm_device_haltest.cpp
+++ b/testcase/hrm_device_haltest.cpp
@@ -41,7 +41,7 @@ sensor_device *sensor_handle = NULL;
 #define SENSOR_SUPPORT_CHECK \
 	do {\
 		if (!sensor_supported) {\
-			cout << "ACCEL SENSOR NOT SUPPORTED" << endl;\
+			cout << "HRM SENSOR NOT SUPPORTED" << endl;\
 			EXPECT_EQ(ret, SENSOR_ERROR_NOT_SUPPORTED);\
 			return;\
 		}\
@@ -61,7 +61,7 @@ public:
 		sensor_supported = false;
 		sensor_handle = new hrm_device;
 		if (sensor_handle == NULL) {
-			cout << "accel sensor init failed " << endl;
+			cout << "hrm sensor init failed " << endl;
 			return;
 		}
 		sensor_supported = true;
@@ -90,8 +90,8 @@ public:
  * @type    		auto
  * @description		Positive, get fd of sensor device
  * @apicovered		get_poll_fd
- * @passcase		returns valid node handle value
- * @failcase		returns invalid node handle value
+ * @passcase		Returns valid node handle value
+ * @failcase		Returns invalid node handle value
  * @precondition	None
  * @postcondition	None
  */
@@ -103,15 +103,13 @@ TEST_F(HRMSensorHalTest, get_poll_fdP)
 	EXPECT_NE(node_handle, 0);
 }
 
-
-
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Get sensor data
  * @apicovered		get_sensors
  * @passcase		get valid sensor info
  * @failcase		get invalid sensor info
@@ -128,12 +126,12 @@ TEST_F(HRMSensorHalTest, get_sensorsP)
 }
 
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Get sensor data
  * @apicovered		get_sensors
  * @passcase		when invalid parameter passed, returns invalid parameter error
  * @failcase		when invalid parameter passed, doesn't return invalid paramter error
@@ -148,15 +146,13 @@ TEST_F(HRMSensorHalTest, get_sensorsN)
 
 }
 
-
-
 /**
  * @testcase		enableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Enable Sensor
  * @apicovered		enable
  * @passcase		when valid id passed, returns 1
  * @failcase		when valid id passed, doesn't returns 1
@@ -170,8 +166,6 @@ TEST_F(HRMSensorHalTest, enableP)
 	uint32_t id = 1;
 	res = sensor_handle->enable(id);
 	EXPECT_EQ(res, true);
-
-
 }
 
 /**
@@ -180,7 +174,7 @@ TEST_F(HRMSensorHalTest, enableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Enable Sensor
  * @apicovered		enable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -201,15 +195,13 @@ TEST_F(HRMSensorHalTest, enableN)
 
 }
 
-
-
 /**
  * @testcase		disableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Disable Sensor
  * @apicovered		disable
  * @passcase		when valid id passed, returns true
  * @failcase		when valid id passed, doesn't returns true
@@ -232,8 +224,8 @@ TEST_F(HRMSensorHalTest, disableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		disabel
+ * @description		Negative, Disable Sensor
+ * @apicovered		disable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
  * @precondition	None
@@ -258,7 +250,7 @@ TEST_F(HRMSensorHalTest, disableN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		get valid id and returns other than zero
  * @failcase		get invalid id and returns other zero
@@ -278,12 +270,12 @@ TEST_F(HRMSensorHalTest, read_fdP)
 }
 
 /**
- * @testcase		get_dataN
+ * @testcase		read_fdN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -295,8 +287,6 @@ TEST_F(HRMSensorHalTest, read_fdN)
 	SENSOR_SUPPORT_CHECK;
 	ret = sensor_handle->read_fd(nullptr);
 	EXPECT_EQ(ret, SENSOR_ERROR_INVALID_PARAMETER);
-
-
 }
 
 /**
@@ -305,7 +295,7 @@ TEST_F(HRMSensorHalTest, read_fdN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Gets Sensor data and data size
  * @apicovered		get_data
  * @passcase		Gets valid data and length and returns 0
  * @failcase		Doesn't get valid or length or doesn't return 0
@@ -325,13 +315,13 @@ TEST_F(HRMSensorHalTest, get_dataP)
 }
 
 /**
- * @testcase		read_fdN
+ * @testcase		get_dataN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
- * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		read_fd
+ * @type....		auto
+ * @description		Negative, Gets Sensor data and data size
+ * @apicovered		get_data
  * @passcase		Returns invalid parameter error
  * @failcase		Doesn't returns invalid parameter error
  * @precondition	None
@@ -362,7 +352,7 @@ TEST_F(HRMSensorHalTest, get_dataN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		when innterval is set, it returns true
  * @failcase		Returns false
@@ -386,7 +376,7 @@ TEST_F(HRMSensorHalTest, set_intervalP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		Returns Invalid paramter error
  * @failcase		Doesn't returns Invalid paramter error
diff --git a/testcase/hrm_raw_device_haltest.cpp b/testcase/hrm_raw_device_haltest.cpp
index 927dbad..ea3c021 100755
--- a/testcase/hrm_raw_device_haltest.cpp
+++ b/testcase/hrm_raw_device_haltest.cpp
@@ -41,7 +41,7 @@ sensor_device *sensor_handle = NULL;
 #define SENSOR_SUPPORT_CHECK \
 	do {\
 		if (!sensor_supported) {\
-			cout << "ACCEL SENSOR NOT SUPPORTED" << endl;\
+			cout << "HRM RAW SENSOR NOT SUPPORTED" << endl;\
 			EXPECT_EQ(ret, SENSOR_ERROR_NOT_SUPPORTED);\
 			return;\
 		}\
@@ -61,7 +61,7 @@ public:
 		sensor_supported = false;
 		sensor_handle = new hrm_raw_device;
 		if (sensor_handle == NULL) {
-			cout << "accel sensor init failed " << endl;
+			cout << "hrm raw sensor init failed " << endl;
 			return;
 		}
 		sensor_supported = true;
@@ -90,8 +90,8 @@ public:
  * @type    		auto
  * @description		Positive, get fd of sensor device
  * @apicovered		get_poll_fd
- * @passcase		returns valid node handle value
- * @failcase		returns invalid node handle value
+ * @passcase		Returns valid node handle value
+ * @failcase		Returns invalid node handle value
  * @precondition	None
  * @postcondition	None
  */
@@ -103,15 +103,13 @@ TEST_F(HRMRawSensorHalTest, get_poll_fdP)
 	EXPECT_NE(node_handle, 0);
 }
 
-
-
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Get sensor data
  * @apicovered		get_sensors
  * @passcase		get valid sensor info
  * @failcase		get invalid sensor info
@@ -128,12 +126,12 @@ TEST_F(HRMRawSensorHalTest, get_sensorsP)
 }
 
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Get sensor data
  * @apicovered		get_sensors
  * @passcase		when invalid parameter passed, returns invalid parameter error
  * @failcase		when invalid parameter passed, doesn't return invalid paramter error
@@ -148,15 +146,13 @@ TEST_F(HRMRawSensorHalTest, get_sensorsN)
 
 }
 
-
-
 /**
  * @testcase		enableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Enable Sensor
  * @apicovered		enable
  * @passcase		when valid id passed, returns 1
  * @failcase		when valid id passed, doesn't returns 1
@@ -170,8 +166,6 @@ TEST_F(HRMRawSensorHalTest, enableP)
 	uint32_t id = 1;
 	res = sensor_handle->enable(id);
 	EXPECT_EQ(res, true);
-
-
 }
 
 /**
@@ -180,7 +174,7 @@ TEST_F(HRMRawSensorHalTest, enableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Enable Sensor
  * @apicovered		enable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -201,15 +195,13 @@ TEST_F(HRMRawSensorHalTest, enableN)
 
 }
 
-
-
 /**
  * @testcase		disableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Disable Sensor
  * @apicovered		disable
  * @passcase		when valid id passed, returns true
  * @failcase		when valid id passed, doesn't returns true
@@ -232,8 +224,8 @@ TEST_F(HRMRawSensorHalTest, disableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		disabel
+ * @description		Negative, Disable Sensor
+ * @apicovered		disable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
  * @precondition	None
@@ -258,7 +250,7 @@ TEST_F(HRMRawSensorHalTest, disableN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		get valid id and returns other than zero
  * @failcase		get invalid id and returns other zero
@@ -278,12 +270,12 @@ TEST_F(HRMRawSensorHalTest, read_fdP)
 }
 
 /**
- * @testcase		get_dataN
+ * @testcase		read_fdN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -295,8 +287,6 @@ TEST_F(HRMRawSensorHalTest, read_fdN)
 	SENSOR_SUPPORT_CHECK;
 	ret = sensor_handle->read_fd(nullptr);
 	EXPECT_EQ(ret, SENSOR_ERROR_INVALID_PARAMETER);
-
-
 }
 
 /**
@@ -305,7 +295,7 @@ TEST_F(HRMRawSensorHalTest, read_fdN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Gets Sensor data and data size
  * @apicovered		get_data
  * @passcase		Gets valid data and length and returns 0
  * @failcase		Doesn't get valid or length or doesn't return 0
@@ -325,13 +315,13 @@ TEST_F(HRMRawSensorHalTest, get_dataP)
 }
 
 /**
- * @testcase		read_fdN
+ * @testcase		get_dataN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
- * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		read_fd
+ * @type....		auto
+ * @description		Negative, Gets Sensor data and data size
+ * @apicovered		get_data
  * @passcase		Returns invalid parameter error
  * @failcase		Doesn't returns invalid parameter error
  * @precondition	None
@@ -362,7 +352,7 @@ TEST_F(HRMRawSensorHalTest, get_dataN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		when innterval is set, it returns true
  * @failcase		Returns false
@@ -386,7 +376,7 @@ TEST_F(HRMRawSensorHalTest, set_intervalP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		Returns Invalid paramter error
  * @failcase		Doesn't returns Invalid paramter error
diff --git a/testcase/light_device_haltest.cpp b/testcase/light_device_haltest.cpp
index e5ae5b5..cbdbcb9 100755
--- a/testcase/light_device_haltest.cpp
+++ b/testcase/light_device_haltest.cpp
@@ -41,7 +41,7 @@ sensor_device *sensor_handle = NULL;
 #define SENSOR_SUPPORT_CHECK \
 	do {\
 		if (!sensor_supported) {\
-			cout << "ACCEL SENSOR NOT SUPPORTED" << endl;\
+			cout << "LIGHT SENSOR NOT SUPPORTED" << endl;\
 			EXPECT_EQ(ret, SENSOR_ERROR_NOT_SUPPORTED);\
 			return;\
 		}\
@@ -61,7 +61,7 @@ public:
 		sensor_supported = false;
 		sensor_handle = new light_device;
 		if (sensor_handle == NULL) {
-			cout << "accel sensor init failed " << endl;
+			cout << "Light sensor init failed " << endl;
 			return;
 		}
 		sensor_supported = true;
@@ -90,8 +90,8 @@ public:
  * @type    		auto
  * @description		Positive, get fd of sensor device
  * @apicovered		get_poll_fd
- * @passcase		returns valid node handle value
- * @failcase		returns invalid node handle value
+ * @passcase		Returns valid node handle value
+ * @failcase		Returns invalid node handle value
  * @precondition	None
  * @postcondition	None
  */
@@ -103,15 +103,13 @@ TEST_F(LightSensorHalTest, get_poll_fdP)
 	EXPECT_NE(node_handle, 0);
 }
 
-
-
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Get sensor data
  * @apicovered		get_sensors
  * @passcase		get valid sensor info
  * @failcase		get invalid sensor info
@@ -128,12 +126,12 @@ TEST_F(LightSensorHalTest, get_sensorsP)
 }
 
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Get sensor data
  * @apicovered		get_sensors
  * @passcase		when invalid parameter passed, returns invalid parameter error
  * @failcase		when invalid parameter passed, doesn't return invalid paramter error
@@ -148,15 +146,13 @@ TEST_F(LightSensorHalTest, get_sensorsN)
 
 }
 
-
-
 /**
  * @testcase		enableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Enable Sensor
  * @apicovered		enable
  * @passcase		when valid id passed, returns 1
  * @failcase		when valid id passed, doesn't returns 1
@@ -170,8 +166,6 @@ TEST_F(LightSensorHalTest, enableP)
 	uint32_t id = 1;
 	res = sensor_handle->enable(id);
 	EXPECT_EQ(res, true);
-
-
 }
 
 /**
@@ -180,7 +174,7 @@ TEST_F(LightSensorHalTest, enableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Enable Sensor
  * @apicovered		enable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -201,15 +195,13 @@ TEST_F(LightSensorHalTest, enableN)
 
 }
 
-
-
 /**
  * @testcase		disableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Disable Sensor
  * @apicovered		disable
  * @passcase		when valid id passed, returns true
  * @failcase		when valid id passed, doesn't returns true
@@ -232,8 +224,8 @@ TEST_F(LightSensorHalTest, disableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		disabel
+ * @description		Negative, Disable Sensor
+ * @apicovered		disable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
  * @precondition	None
@@ -258,7 +250,7 @@ TEST_F(LightSensorHalTest, disableN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		get valid id and returns other than zero
  * @failcase		get invalid id and returns other zero
@@ -278,12 +270,12 @@ TEST_F(LightSensorHalTest, read_fdP)
 }
 
 /**
- * @testcase		get_dataN
+ * @testcase		read_fdN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -295,8 +287,6 @@ TEST_F(LightSensorHalTest, read_fdN)
 	SENSOR_SUPPORT_CHECK;
 	ret = sensor_handle->read_fd(nullptr);
 	EXPECT_EQ(ret, SENSOR_ERROR_INVALID_PARAMETER);
-
-
 }
 
 /**
@@ -305,7 +295,7 @@ TEST_F(LightSensorHalTest, read_fdN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Gets Sensor data and data size
  * @apicovered		get_data
  * @passcase		Gets valid data and length and returns 0
  * @failcase		Doesn't get valid or length or doesn't return 0
@@ -325,13 +315,13 @@ TEST_F(LightSensorHalTest, get_dataP)
 }
 
 /**
- * @testcase		read_fdN
+ * @testcase		get_dataN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
- * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		read_fd
+ * @type....		auto
+ * @description		Negative, Gets Sensor data and data size
+ * @apicovered		get_data
  * @passcase		Returns invalid parameter error
  * @failcase		Doesn't returns invalid parameter error
  * @precondition	None
@@ -362,7 +352,7 @@ TEST_F(LightSensorHalTest, get_dataN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		when innterval is set, it returns true
  * @failcase		Returns false
@@ -386,7 +376,7 @@ TEST_F(LightSensorHalTest, set_intervalP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		Returns Invalid paramter error
  * @failcase		Doesn't returns Invalid paramter error
diff --git a/testcase/pressure_device_haltest.cpp b/testcase/pressure_device_haltest.cpp
index 35f6a9e..7bab3fd 100755
--- a/testcase/pressure_device_haltest.cpp
+++ b/testcase/pressure_device_haltest.cpp
@@ -40,7 +40,7 @@ sensor_device *sensor_handle = NULL;
 #define SENSOR_SUPPORT_CHECK \
 	do {\
 		if (!sensor_supported) {\
-			cout << "ACCEL SENSOR NOT SUPPORTED" << endl;\
+			cout << "PRESSURE SENSOR NOT SUPPORTED" << endl;\
 			EXPECT_EQ(ret, SENSOR_ERROR_NOT_SUPPORTED);\
 			return;\
 		}\
@@ -60,7 +60,7 @@ public:
 		sensor_supported = false;
 		sensor_handle = new pressure_device;
 		if (sensor_handle == NULL) {
-			cout << "accel sensor init failed " << endl;
+			cout << "pressure sensor init failed " << endl;
 			return;
 		}
 		sensor_supported = true;
@@ -89,8 +89,8 @@ public:
  * @type    		auto
  * @description		Positive, get fd of sensor device
  * @apicovered		get_poll_fd
- * @passcase		returns valid node handle value
- * @failcase		returns invalid node handle value
+ * @passcase		Returns valid node handle value
+ * @failcase		Returns invalid node handle value
  * @precondition	None
  * @postcondition	None
  */
@@ -102,15 +102,13 @@ TEST_F(PressureSensorHalTest, get_poll_fdP)
 	EXPECT_NE(node_handle, 0);
 }
 
-
-
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Get sensor data
  * @apicovered		get_sensors
  * @passcase		get valid sensor info
  * @failcase		get invalid sensor info
@@ -127,12 +125,12 @@ TEST_F(PressureSensorHalTest, get_sensorsP)
 }
 
 /**
- * @testcase		get_hal_versionN
+ * @testcase		get_sensorsN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Get sensor data
  * @apicovered		get_sensors
  * @passcase		when invalid parameter passed, returns invalid parameter error
  * @failcase		when invalid parameter passed, doesn't return invalid paramter error
@@ -147,15 +145,13 @@ TEST_F(PressureSensorHalTest, get_sensorsN)
 
 }
 
-
-
 /**
  * @testcase		enableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Enable Sensor
  * @apicovered		enable
  * @passcase		when valid id passed, returns 1
  * @failcase		when valid id passed, doesn't returns 1
@@ -177,7 +173,7 @@ TEST_F(PressureSensorHalTest, enableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Enable Sensor
  * @apicovered		enable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -198,15 +194,13 @@ TEST_F(PressureSensorHalTest, enableN)
 
 }
 
-
-
 /**
  * @testcase		disableP
  * @since_tizen 	4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Disable Sensor
  * @apicovered		disable
  * @passcase		when valid id passed, returns true
  * @failcase		when valid id passed, doesn't returns true
@@ -229,8 +223,8 @@ TEST_F(PressureSensorHalTest, disableP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		disabel
+ * @description		Negative, Disable Sensor
+ * @apicovered		disable
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
  * @precondition	None
@@ -255,7 +249,7 @@ TEST_F(PressureSensorHalTest, disableN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		get valid id and returns other than zero
  * @failcase		get invalid id and returns other zero
@@ -275,12 +269,12 @@ TEST_F(PressureSensorHalTest, read_fdP)
 }
 
 /**
- * @testcase		get_dataN
+ * @testcase		read_fdN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Reads sensosr device id
  * @apicovered		read_fd
  * @passcase		when invalid id passed, returns invalid parameter error
  * @failcase		when invalid id passed, doesn't returns invalid parameter error
@@ -292,8 +286,6 @@ TEST_F(PressureSensorHalTest, read_fdN)
 	SENSOR_SUPPORT_CHECK;
 	ret = sensor_handle->read_fd(nullptr);
 	EXPECT_EQ(ret, SENSOR_ERROR_INVALID_PARAMETER);
-
-
 }
 
 /**
@@ -302,7 +294,7 @@ TEST_F(PressureSensorHalTest, read_fdN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Gets Sensor data and data size
  * @apicovered		get_data
  * @passcase		Gets valid data and length and returns 0
  * @failcase		Doesn't get valid or length or doesn't return 0
@@ -322,13 +314,13 @@ TEST_F(PressureSensorHalTest, get_dataP)
 }
 
 /**
- * @testcase		read_fdN
+ * @testcase		get_dataN
  * @since_tizen		4.0
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
- * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
- * @apicovered		read_fd
+ * @type....		auto
+ * @description		Negative, Gets Sensor data and data size
+ * @apicovered		get_data
  * @passcase		Returns invalid parameter error
  * @failcase		Doesn't returns invalid parameter error
  * @precondition	None
@@ -359,7 +351,7 @@ TEST_F(PressureSensorHalTest, get_dataN)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description 	Positive, Release video buffer to get new frame in it
+ * @description 	Positive, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		when innterval is set, it returns true
  * @failcase		Returns false
@@ -383,7 +375,7 @@ TEST_F(PressureSensorHalTest, set_intervalP)
  * @author  		SRID(srinivasa.m)
  * @reviewer		SRID(randeep.s)
  * @type    		auto
- * @description		Positive, Release video buffer to get new frame in it
+ * @description		Negative, Set Interval for sensor response
  * @apicovered		set_interval
  * @passcase		Returns Invalid paramter error
  * @failcase		Doesn't returns Invalid paramter error