Skip to content

Commit fea5dc6

Browse files
committed
SF: temporary disable FlagManagerTest.returnsOverrideFalse
Since we are left we a single server flag, we can't have a unique flag for this test which is required since the flag implementation uses static variables. We'll re-enable once we have a new server flag. Change-Id: I49826eb0fbb76a389c063b6234b2a2131e358c75 Test: presubmit Bug: 325334658
1 parent 4238fc9 commit fea5dc6

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

services/surfaceflinger/tests/unittests/FlagManagerTest.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,10 @@ TEST_F(FlagManagerTest, returnsOverrideReadonly) {
114114
EXPECT_TRUE(mFlagManager.add_sf_skipped_frames_to_trace());
115115
}
116116

117-
TEST_F(FlagManagerTest, returnsOverrideFalse) {
117+
// disabling this test since we need to use a unique flag for this test,
118+
// but we only one server flag currently. Re-enable once we have a new flag
119+
// and change this test to use a unique flag.
120+
TEST_F(FlagManagerTest, DISABLED_returnsOverrideFalse) {
118121
mFlagManager.markBootCompleted();
119122

120123
SET_FLAG_FOR_TEST(flags::refresh_rate_overlay_on_external_display, true);

0 commit comments

Comments
 (0)