diff options
| -rw-r--r-- | src/activity_log_unittest.cc | 5 | ||||
| -rw-r--r-- | src/gestures_unittest.cc | 3 |
2 files changed, 8 insertions, 0 deletions
diff --git a/src/activity_log_unittest.cc b/src/activity_log_unittest.cc index f1b4f37..aadf2f5 100644 --- a/src/activity_log_unittest.cc +++ b/src/activity_log_unittest.cc @@ -131,6 +131,10 @@ TEST(ActivityLogTest, SimpleTest) { TEST(ActivityLogTest, WrapAroundTest) { ActivityLog log(nullptr); +#ifndef __ANDROID__ + // Disabling this test with DISABLED_ prefix is not sufficient as just by compiling this + // section causes the hwasan crash at runtime in some other tests + // TODO(b/311110623): re-enable this section once we've tracked down the hwasan crash // overfill the buffer const size_t fill_size = (ActivityLog::kBufferSize * 3) / 2; for (size_t i = 0; i < fill_size; i++) @@ -140,6 +144,7 @@ TEST(ActivityLogTest, WrapAroundTest) { log.LogCallbackRequest(static_cast<stime_t>(fill_size)); string second_prefix = log.Encode().substr(0, prefix_length); EXPECT_NE(first_prefix, second_prefix); +#endif } TEST(ActivityLogTest, VersionTest) { diff --git a/src/gestures_unittest.cc b/src/gestures_unittest.cc index 77d4db0..6a22b2f 100644 --- a/src/gestures_unittest.cc +++ b/src/gestures_unittest.cc @@ -342,12 +342,15 @@ TEST(GesturesTest, SimpleTest) { gs_ptr->Initialize(GESTURES_DEVCLASS_POINTING_STICK); DeleteGestureInterpreter(gs_ptr); +#ifndef __ANDROID__ + // TODO(b/311110623): re-enable this section once we've tracked down the hwasan crash gs_ptr = NewGestureInterpreter(); EXPECT_NE(nullptr, gs_ptr); gs_ptr->Initialize(GESTURES_DEVCLASS_MULTITOUCH_MOUSE); std::string activity = gs_ptr->EncodeActivityLog(); EXPECT_NE(activity.size(), 0); DeleteGestureInterpreter(gs_ptr); +#endif EXPECT_EQ("1073741824", FingerState::FlagsString(1 << 30)); } |