diff --git a/window_scene/session/screen/src/screen_session.cpp b/window_scene/session/screen/src/screen_session.cpp index 3c9195f5f3378be80f8d1b4b58bc8e91e9fa9d93..2fbc7a8a503b024f0e002d18254082569e81e3c7 100644 --- a/window_scene/session/screen/src/screen_session.cpp +++ b/window_scene/session/screen/src/screen_session.cpp @@ -2185,7 +2185,7 @@ void ScreenSession::SetForceCloseHdr(bool isForceCloseHdr) return; } if (lastCloseHdrStatus_ == isForceCloseHdr) { - TLOGD(WmsLogTag::DMS, "lastCloseHdrStatus_ and isForceCloseHdr are the same."); + TLOGE(WmsLogTag::DMS, "lastCloseHdrStatus_ and isForceCloseHdr are the same."); return; } lastCloseHdrStatus_ = isForceCloseHdr; diff --git a/window_scene/test/dms_unittest/dual_display_sensor_fold_state_manager_test.cpp b/window_scene/test/dms_unittest/dual_display_sensor_fold_state_manager_test.cpp index b28bf2c8beeeab00fce88857db67fd4f8689957a..3550f6c60385701e7d8afb1050eb75d3c307ef65 100644 --- a/window_scene/test/dms_unittest/dual_display_sensor_fold_state_manager_test.cpp +++ b/window_scene/test/dms_unittest/dual_display_sensor_fold_state_manager_test.cpp @@ -78,6 +78,9 @@ namespace { */ HWTEST_F(DualDisplaySensorFoldStateManagerTest, DualDisplaySensorFoldStateManager, TestSize.Level1) { + if (!FoldScreenStateInternel::IsDualDisplayFoldDevice()) { + GTEST_SKIP(); + } std::shared_ptr screenPowerTaskScheduler = nullptr; DualDisplaySensorFoldStateManager mgr = DualDisplaySensorFoldStateManager(screenPowerTaskScheduler); ASSERT_EQ(mgr.packageNames_.size(), 1); diff --git a/window_scene/test/dms_unittest/screen_session_test.cpp b/window_scene/test/dms_unittest/screen_session_test.cpp index fa5b82ac5dd0678455ce785543c1d3527c201914..6f26d804be0a89be126ad03df04f1026b0cad8e2 100644 --- a/window_scene/test/dms_unittest/screen_session_test.cpp +++ b/window_scene/test/dms_unittest/screen_session_test.cpp @@ -4027,11 +4027,13 @@ HWTEST_F(ScreenSessionTest, GetChildCount, TestSize.Level1) HWTEST_F(ScreenSessionTest, SetForceCloseHdr, TestSize.Level1) { GTEST_LOG_(INFO) << "ScreenSessionTest: SetForceCloseHdr start"; - sptr session = sptr::MakeSptr(); + LOG_SetCallback(MyLogCallback); + ScreenProperty newScreenProperty; + sptr session = sptr::MakeSptr(0,newScreenProperty,0); ASSERT_NE(session, nullptr); session->SetForceCloseHdr(true); session->SetForceCloseHdr(false); - EXPECT_TRUE(g_errLog.find("Start get screen status.") != std::string::npos); + EXPECT_FALSE(g_errLog.find("Start get screen status.") != std::string::npos); GTEST_LOG_(INFO) << "ScreenSessionTest: SetForceCloseHdr end"; } @@ -4043,11 +4045,13 @@ HWTEST_F(ScreenSessionTest, SetForceCloseHdr, TestSize.Level1) HWTEST_F(ScreenSessionTest, SetForceCloseHdr01, TestSize.Level1) { GTEST_LOG_(INFO) << "ScreenSessionTest: SetForceCloseHdr start"; - sptr session = sptr::MakeSptr(); + LOG_SetCallback(MyLogCallback); + ScreenProperty newScreenProperty; + sptr session = sptr::MakeSptr(0,newScreenProperty,0); ASSERT_NE(session, nullptr); session->SetForceCloseHdr(false); session->SetForceCloseHdr(false); - EXPECT_TRUE(g_errLog.find("Start get screen status.") != std::string::npos); + EXPECT_TRUE(g_errLog.find("lastCloseHdrStatus_ and isForceCloseHdr are the same.") != std::string::npos); GTEST_LOG_(INFO) << "ScreenSessionTest: SetForceCloseHdr end"; }