Skip to content

Commit f755b7b

Browse files
committed
Add diag for inputflinger tests
In some cases diag may give us better error reporting. We can't add diag to production code because it would slow down the code, but having it on tests shouldn't have any impact. Bug: 378308551 Test: TEST=inputflinger_tests; m $TEST && $ANDROID_HOST_OUT/nativetest64/$TEST/$TEST Flag: TEST_ONLY Change-Id: I91349ee7e880685519690a47580a18f240d00b9f
1 parent 3001fd6 commit f755b7b

1 file changed

Lines changed: 16 additions & 9 deletions

File tree

services/inputflinger/tests/Android.bp

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,11 @@ cc_test {
4040
// defaults rather than including them as shared or static libraries. By doing so, the tests
4141
// will always run against the compiled version of the inputflinger code rather than the
4242
// version on the device.
43+
"libinputdispatcher_defaults",
4344
"libinputflinger_base_defaults",
45+
"libinputflinger_defaults",
4446
"libinputreader_defaults",
4547
"libinputreporter_defaults",
46-
"libinputdispatcher_defaults",
47-
"libinputflinger_defaults",
4848
],
4949
srcs: [
5050
":inputdispatcher_common_test_sources",
@@ -62,16 +62,18 @@ cc_test {
6262
"HardwareStateConverter_test.cpp",
6363
"InputDeviceMetricsCollector_test.cpp",
6464
"InputDeviceMetricsSource_test.cpp",
65+
"InputDispatcher_test.cpp",
6566
"InputMapperTest.cpp",
66-
"InputProcessor_test.cpp",
6767
"InputProcessorConverter_test.cpp",
68-
"InputDispatcher_test.cpp",
68+
"InputProcessor_test.cpp",
6969
"InputReader_test.cpp",
7070
"InputTraceSession.cpp",
7171
"InputTracingTest.cpp",
7272
"InstrumentedInputReader.cpp",
7373
"JoystickInputMapper_test.cpp",
74+
"KeyboardInputMapper_test.cpp",
7475
"LatencyTracker_test.cpp",
76+
"MultiTouchInputMapper_test.cpp",
7577
"MultiTouchMotionAccumulator_test.cpp",
7678
"NotifyArgs_test.cpp",
7779
"PointerChoreographer_test.cpp",
@@ -82,14 +84,12 @@ cc_test {
8284
"SlopController_test.cpp",
8385
"SwitchInputMapper_test.cpp",
8486
"SyncQueue_test.cpp",
85-
"TimerProvider_test.cpp",
8687
"TestInputListener.cpp",
88+
"TimerProvider_test.cpp",
8789
"TouchpadInputMapper_test.cpp",
88-
"VibratorInputMapper_test.cpp",
89-
"MultiTouchInputMapper_test.cpp",
90-
"KeyboardInputMapper_test.cpp",
9190
"UinputDevice.cpp",
9291
"UnwantedInteractionBlocker_test.cpp",
92+
"VibratorInputMapper_test.cpp",
9393
],
9494
aidl: {
9595
include_dirs: [
@@ -109,7 +109,14 @@ cc_test {
109109
undefined: true,
110110
all_undefined: true,
111111
diag: {
112+
cfi: true,
113+
integer_overflow: true,
114+
memtag_heap: true,
112115
undefined: true,
116+
misc_undefined: [
117+
"all",
118+
"bounds",
119+
],
113120
},
114121
},
115122
static_libs: [
@@ -121,8 +128,8 @@ cc_test {
121128
unit_test: true,
122129
},
123130
test_suites: [
124-
"device-tests",
125131
"device-platinum-tests",
132+
"device-tests",
126133
],
127134
native_coverage: false,
128135
}

0 commit comments

Comments
 (0)