Skip to content

Commit

Permalink
Remove unnecessary MATCHER_P. (#1729)
Browse files Browse the repository at this point in the history
  • Loading branch information
wujingyue authored Feb 7, 2024
1 parent f73ff1b commit 2123f16
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions test/test_linked_hash_map.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,7 @@ namespace nvfuser {
using testing::ElementsAre;
using testing::Eq;
using testing::Pair;
using testing::Property;

TEST(LinkedHashMapTest, PushBack) {
LinkedHashMap<std::string, int> map;
Expand Down Expand Up @@ -118,19 +119,15 @@ TEST(LinkedHashMapTest, EraseThenPushBack) {
EXPECT_THAT(map, ElementsAre(Pair("a", 1), Pair("b", 4)));
}

namespace {
MATCHER_P(DataIs, data, "") {
return arg.data() == data;
}
} // namespace

TEST(LinkedHashMapTest, MovableValue) {
LinkedHashMap<CopyableKey, MovableValue> map;
map.pushBack(CopyableKey("a"), MovableValue(1));
map.pushBack(CopyableKey("b"), MovableValue(2));
map.erase(CopyableKey("b"));

EXPECT_THAT(map, ElementsAre(Pair(CopyableKey("a"), DataIs(1))));
EXPECT_THAT(
map,
ElementsAre(Pair(CopyableKey("a"), Property(&MovableValue::data, 1))));
}

} // namespace nvfuser

0 comments on commit 2123f16

Please sign in to comment.