summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2017-04-25 20:36:00 +0200
committerandroid-build-merger <android-build-merger@google.com>2017-04-25 20:36:00 +0200
commit651be2a2af199f7fbf341c4d071bfce1d7ccaf27 (patch)
treee8d05ad5b5576ab57843f600476aa7e86846d6cc
parentMerge "recovery: Remove the include of adb.h." am: dff2276012 am: 2dad1a9d28 am: 36683d0fc8 (diff)
parentMerge "minadbd: Fix a failing test due to SIGPIPE." am: 7b1fffe095 am: f849ff34d4 (diff)
downloadandroid_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.tar
android_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.tar.gz
android_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.tar.bz2
android_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.tar.lz
android_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.tar.xz
android_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.tar.zst
android_bootable_recovery-651be2a2af199f7fbf341c4d071bfce1d7ccaf27.zip
-rw-r--r--minadbd/fuse_adb_provider_test.cpp20
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]);
}