diff options
author | Tao Bao <tbao@google.com> | 2017-04-25 20:34:00 +0200 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2017-04-25 20:34:00 +0200 |
commit | f70e39a90dc2843365e72f86d95cb2db537c97a0 (patch) | |
tree | 56e9841e097f80b3dd9d920c123fad4c55c0e005 | |
parent | Merge "recovery: Remove the include of adb.h." am: dff2276012 am: 2dad1a9d28 (diff) | |
parent | Merge "minadbd: Fix a failing test due to SIGPIPE." am: 7b1fffe095 (diff) | |
download | android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.tar android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.tar.gz android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.tar.bz2 android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.tar.lz android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.tar.xz android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.tar.zst android_bootable_recovery-f70e39a90dc2843365e72f86d95cb2db537c97a0.zip |
-rw-r--r-- | minadbd/fuse_adb_provider_test.cpp | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/minadbd/fuse_adb_provider_test.cpp b/minadbd/fuse_adb_provider_test.cpp index 0f2e881c7..31be2a64e 100644 --- a/minadbd/fuse_adb_provider_test.cpp +++ b/minadbd/fuse_adb_provider_test.cpp @@ -14,17 +14,17 @@ * limitations under the License. */ -#include "fuse_adb_provider.h" - -#include <gtest/gtest.h> - #include <errno.h> #include <fcntl.h> +#include <signal.h> #include <sys/socket.h> #include <string> +#include <gtest/gtest.h> + #include "adb_io.h" +#include "fuse_adb_provider.h" TEST(fuse_adb_provider, read_block_adb) { adb_data data = {}; @@ -46,9 +46,8 @@ TEST(fuse_adb_provider, read_block_adb) { uint32_t block = 1234U; const char expected_block[] = "00001234"; - ASSERT_EQ(0, read_block_adb(reinterpret_cast<void*>(&data), block, - reinterpret_cast<uint8_t*>(block_data), - sizeof(expected_data) - 1)); + ASSERT_EQ(0, read_block_adb(static_cast<void*>(&data), block, + reinterpret_cast<uint8_t*>(block_data), sizeof(expected_data) - 1)); // Check that read_block_adb requested the right block. char block_req[sizeof(expected_block)] = {}; @@ -80,9 +79,12 @@ TEST(fuse_adb_provider, read_block_adb_fail_write) { ASSERT_EQ(0, close(sockets[1])); + // write(2) raises SIGPIPE since the reading end has been closed. Ignore the signal to avoid + // failing the test. + signal(SIGPIPE, SIG_IGN); + char buf[1]; - ASSERT_EQ(-EIO, read_block_adb(reinterpret_cast<void*>(&data), 0, - reinterpret_cast<uint8_t*>(buf), 1)); + ASSERT_EQ(-EIO, read_block_adb(static_cast<void*>(&data), 0, reinterpret_cast<uint8_t*>(buf), 1)); close(sockets[0]); } |