From 2123f1657ffed92d0090c42f6c35a256c061cba6 Mon Sep 17 00:00:00 2001
From: Jingyue Wu <wujingyue@gmail.com>
Date: Tue, 6 Feb 2024 17:14:59 -0800
Subject: [PATCH] Remove unnecessary `MATCHER_P`. (#1729)

---
 test/test_linked_hash_map.cpp | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/test/test_linked_hash_map.cpp b/test/test_linked_hash_map.cpp
index 8261a358d7c..83ee8696127 100644
--- a/test/test_linked_hash_map.cpp
+++ b/test/test_linked_hash_map.cpp
@@ -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;
@@ -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