From: Youngjae Cho <y0.cho@samsung.com>
Date: Mon, 6 Jan 2025 04:47:10 +0000 (+0900)
Subject: [ITC/UTC][capi-system-device][Non-ACR] Fix to correctly test headless profile
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0cb33e5cb2aab62795e9a016ec84a560f3f55dd1;p=test%2Ftct%2Fnative%2Fapi.git

[ITC/UTC][capi-system-device][Non-ACR] Fix to correctly test headless profile

Instead of skipping test on headless profile, do it like headed profile.
This is possible due to change of device API and deviced that has
become working properly on headless profile.

Change-Id: Iee0a301fc43a0656a76cf20ce267d49ed830c92d
Signed-off-by: Youngjae Cho <y0.cho@samsung.com>
---

diff --git a/src/itc/device/ITs-device_power.c b/src/itc/device/ITs-device_power.c
index 7a91b17fb..0825b3cf8 100644
--- a/src/itc/device/ITs-device_power.c
+++ b/src/itc/device/ITs-device_power.c
@@ -91,30 +91,46 @@ int ITc_device_power_request_release_lock_p(void)
 {
 	START_TEST;
 
-	power_lock_e power_locks[] = { POWER_LOCK_CPU, POWER_LOCK_DISPLAY, POWER_LOCK_DISPLAY_DIM };
-	int enum_size = sizeof(power_locks) / sizeof(power_locks[0]);
+	struct expect_return_by_profile {
+		power_lock_e lock_type;
+		int headed;
+		int headless;
+	} expect_return[] = {
+		{
+			.lock_type = POWER_LOCK_CPU,
+			.headed = DEVICE_ERROR_NONE,
+			.headless = DEVICE_ERROR_NONE,
+		}, {
+			.lock_type = POWER_LOCK_DISPLAY,
+			.headed = DEVICE_ERROR_NONE,
+			.headless = DEVICE_ERROR_NOT_SUPPORTED,
+		}, {
+			.lock_type = POWER_LOCK_DISPLAY_DIM,
+			.headed = DEVICE_ERROR_NONE,
+			.headless = DEVICE_ERROR_NOT_SUPPORTED,
+		},
+	};
+	int enum_size = sizeof(expect_return) / sizeof(expect_return[0]);
 	int enum_counter = 0;
 
-	if ((bIsDisplayFeatureSupported == false) || (bIsDisplayStateFeatureSupported == false))
-		return 0;
-
 	for ( enum_counter=0; enum_counter<enum_size; enum_counter++)
 	{
 		// Target API
-		int nRet = device_power_request_lock(power_locks[enum_counter], 10);
-		if ( nRet != DEVICE_ERROR_NONE )
+		int nRet = device_power_request_lock(expect_return[enum_counter].lock_type, 10);
+		int nExpectedRet = bIsDisplayFeatureSupported ? expect_return[enum_counter].headed : expect_return[enum_counter].headless;
+		if ( nRet != nExpectedRet )
 		{
-			FPRINTF("[Line : %d][%s] device_power_request_lock failed for enum = %s, error returned = %s\\n",
-				__LINE__, API_NAMESPACE, DeviceGetPowerLock(power_locks[enum_counter]), DeviceGetError(nRet));
+			FPRINTF("[Line : %d][%s] device_power_request_lock failed for enum = %s, error returned = %s, expected return = %s\\n",
+					__LINE__, API_NAMESPACE, DeviceGetPowerLock(expect_return[enum_counter].lock_type), DeviceGetError(nRet), DeviceGetError(nExpectedRet));
 			return 1;
 		}
 
 		// Target API
-		nRet  = device_power_release_lock(power_locks[enum_counter]);
-		if ( nRet != DEVICE_ERROR_NONE )
+		nRet  = device_power_release_lock(expect_return[enum_counter].lock_type);
+		if ( nRet != nExpectedRet )
 		{
-			FPRINTF("[Line : %d][%s] device_power_release_lock failed for enum = %s, error returned = %s\\n",
-				__LINE__, API_NAMESPACE, DeviceGetPowerLock(power_locks[enum_counter]), DeviceGetError(nRet));
+			FPRINTF("[Line : %d][%s] device_power_release_lock failed for enum = %s, error returned = %s, expected return = %s\\n",
+				__LINE__, API_NAMESPACE, DeviceGetPowerLock(expect_return[enum_counter].lock_type), DeviceGetError(nRet), DeviceGetError(nExpectedRet));
 			return 1;
 		}
 	}
diff --git a/src/utc/device/utc-system-device-power.c b/src/utc/device/utc-system-device-power.c
index c66b3b594..04ef83571 100644
--- a/src/utc/device/utc-system-device-power.c
+++ b/src/utc/device/utc-system-device-power.c
@@ -145,9 +145,6 @@ int utc_system_device_power_request_lock_n_2(void)
 				(locks[i] == POWER_LOCK_DISPLAY_DIM)) {
 				assert_eq(error, DEVICE_ERROR_NOT_SUPPORTED);
 				return 0;
-			} else if (locks[i] == POWER_LOCK_CPU) {
-				assert_eq(error, DEVICE_ERROR_NONE);
-				return 0;
 			}
 		}
 		assert_eq(error, DEVICE_ERROR_INVALID_PARAMETER);