diff options
Diffstat (limited to 'cmds/idmap2/tests/FileUtilsTests.cpp')
| -rw-r--r-- | cmds/idmap2/tests/FileUtilsTests.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cmds/idmap2/tests/FileUtilsTests.cpp b/cmds/idmap2/tests/FileUtilsTests.cpp index 0c6439ab8c0c..6584ee32a509 100644 --- a/cmds/idmap2/tests/FileUtilsTests.cpp +++ b/cmds/idmap2/tests/FileUtilsTests.cpp @@ -39,7 +39,7 @@ TEST(FileUtilsTests, FindFilesFindEverythingNonRecursive) { [](unsigned char type ATTRIBUTE_UNUSED, const std::string& path ATTRIBUTE_UNUSED) -> bool { return true; }); ASSERT_THAT(v, NotNull()); - ASSERT_EQ(v->size(), 4u); + ASSERT_EQ(v->size(), 4U); ASSERT_EQ( std::set<std::string>(v->begin(), v->end()), std::set<std::string>({root + "/.", root + "/..", root + "/overlay", root + "/target"})); @@ -48,10 +48,10 @@ TEST(FileUtilsTests, FindFilesFindEverythingNonRecursive) { TEST(FileUtilsTests, FindFilesFindApkFilesRecursive) { const auto& root = GetTestDataPath(); auto v = utils::FindFiles(root, true, [](unsigned char type, const std::string& path) -> bool { - return type == DT_REG && path.size() > 4 && !path.compare(path.size() - 4, 4, ".apk"); + return type == DT_REG && path.size() > 4 && path.compare(path.size() - 4, 4, ".apk") == 0; }); ASSERT_THAT(v, NotNull()); - ASSERT_EQ(v->size(), 4u); + ASSERT_EQ(v->size(), 4U); ASSERT_EQ(std::set<std::string>(v->begin(), v->end()), std::set<std::string>({root + "/target/target.apk", root + "/overlay/overlay.apk", root + "/overlay/overlay-static-1.apk", |