ACPR/android_system_core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge "liblog: cleanup TODOs in tests"

Browse source
This commit is contained in:
Tom Cherry 2020-04-11 00:18:57 +00:00 • committed by Gerrit Code Review
commit ac0cee539f
1 changed files with 3 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
liblog/tests/liblog_test.cpp
View file

@ -97,10 +97,7 @@ static void RunLogTests(log_id_t log_buffer, FWrite write_messages, FCheck check
ASSERT_EQ(log_buffer, log_msg.id());
ASSERT_EQ(pid, log_msg.entry.pid);
// TODO: Should this be an assert?
if (log_msg.msg() == nullptr) {
continue;
}
ASSERT_NE(nullptr, log_msg.msg());
check_message(log_msg, &found);
}
@ -121,10 +118,7 @@ static void RunLogTests(log_id_t log_buffer, FWrite write_messages, FCheck check
ASSERT_EQ(log_buffer, log_msg.id());
ASSERT_EQ(pid, log_msg.entry.pid);
// TODO: Should this be an assert?
if (log_msg.msg() == nullptr) {
continue;
}
ASSERT_NE(nullptr, log_msg.msg());
found = false;
check_message(log_msg, &found);
@ -1031,7 +1025,7 @@ TEST(liblog, __android_log_buf_print__maxtag) {
#endif
}
// TODO: This test is tautological. android_logger_list_read() calls recv() with
// Note: This test is tautological. android_logger_list_read() calls recv() with
// LOGGER_ENTRY_MAX_PAYLOAD as its size argument, so it's not possible for this test to read a
// payload larger than that size.
TEST(liblog, too_big_payload) {
@ -1982,7 +1976,6 @@ TEST(liblog,
#endif
}
// TODO: Do we need to check that we didn't actually write anything if we return a failure here?
TEST(liblog,
android_errorWriteWithInfoLog__android_logger_list_read__null_data) {
#ifdef __ANDROID__