diff options
| author | Tao Bao <tbao@google.com> | 2016-12-22 18:33:33 +0100 |
|---|---|---|
| committer | android-build-merger <android-build-merger@google.com> | 2016-12-22 18:33:33 +0100 |
| commit | 5a108b6e00190885295d3fd40b98aebf89379533 (patch) | |
| tree | 0fdc5c6c8d29396b07e99e84750340ad7bd3c896 /tests/unit | |
| parent | Merge "tests: Add testcase for ZipUtil." am: aa305e9ce8 am: 6df1c4579e (diff) | |
| parent | Merge "tests: Replace the O_RDONLY in access(2)." am: 3b9645288e (diff) | |
| download | android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.tar android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.tar.gz android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.tar.bz2 android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.tar.lz android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.tar.xz android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.tar.zst android_bootable_recovery-5a108b6e00190885295d3fd40b98aebf89379533.zip | |
Diffstat (limited to '')
| -rw-r--r-- | tests/unit/zip_test.cpp | 21 |
1 files changed, 15 insertions, 6 deletions
diff --git a/tests/unit/zip_test.cpp b/tests/unit/zip_test.cpp index ef0ee4c1d..4a1a49b97 100644 --- a/tests/unit/zip_test.cpp +++ b/tests/unit/zip_test.cpp @@ -15,7 +15,6 @@ */ #include <errno.h> -#include <fcntl.h> #include <unistd.h> #include <memory> @@ -42,10 +41,10 @@ TEST(ZipTest, ExtractPackageRecursive) { // Make sure all the files are extracted correctly. std::string path(td.path); - ASSERT_EQ(0, access((path + "/a.txt").c_str(), O_RDONLY)); - ASSERT_EQ(0, access((path + "/b.txt").c_str(), O_RDONLY)); - ASSERT_EQ(0, access((path + "/b/c.txt").c_str(), O_RDONLY)); - ASSERT_EQ(0, access((path + "/b/d.txt").c_str(), O_RDONLY)); + ASSERT_EQ(0, access((path + "/a.txt").c_str(), F_OK)); + ASSERT_EQ(0, access((path + "/b.txt").c_str(), F_OK)); + ASSERT_EQ(0, access((path + "/b/c.txt").c_str(), F_OK)); + ASSERT_EQ(0, access((path + "/b/d.txt").c_str(), F_OK)); // The content of the file is the same as expected. std::string content1; @@ -54,7 +53,16 @@ TEST(ZipTest, ExtractPackageRecursive) { std::string content2; ASSERT_TRUE(android::base::ReadFileToString(path + "/b/d.txt", &content2)); - ASSERT_EQ(kBTxtContents, content2); + ASSERT_EQ(kDTxtContents, content2); + + CloseArchive(handle); + + // Clean up. + ASSERT_EQ(0, unlink((path + "/a.txt").c_str())); + ASSERT_EQ(0, unlink((path + "/b.txt").c_str())); + ASSERT_EQ(0, unlink((path + "/b/c.txt").c_str())); + ASSERT_EQ(0, unlink((path + "/b/d.txt").c_str())); + ASSERT_EQ(0, rmdir((path + "/b").c_str())); } TEST(ZipTest, OpenFromMemory) { @@ -76,6 +84,7 @@ TEST(ZipTest, OpenFromMemory) { ASSERT_NE(-1, tmp_binary.fd); ASSERT_EQ(0, ExtractEntryToFile(handle, &binary_entry, tmp_binary.fd)); + CloseArchive(handle); sysReleaseMap(&map); } |