commit b6b33a84ed7badd0441cec1324ba3ef7f1d897e2
author Jonathan Lee <jonathanjlee@google.com> Sat Sep 06 00:23:47 2025 -0400
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> Sat Sep 06 07:30:32 2025 -0700
tree e8471d751be9fbd8ae805a6ee78bc83cdca776ab
parent 29cd444602d2153326db31b78d12b2a0f6c49e70

[gtest] Clean up single-arg `testing::Invoke()`s

Not needed and deprecated [0]. This CL should be a no-op.

[0]: https://chromium.googlesource.com/external/github.com/google/googletest.git/+/a05c0915074bcd1b82f232e081da9bb6c205c28d/googlemock/include/gmock/gmock-actions.h#2046

Bug: chromium:439838457
Change-Id: Ib93a80dbe2bb336a1e141422ca4a7de438e8ad4c
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/81907
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: Adam Langley <agl@google.com>
Auto-Submit: Jonathan Lee <jonathanjlee@google.com>

pki/path_builder_unittest.cc

diff --git a/pki/path_builder_unittest.cc b/pki/path_builder_unittest.cc
index 4464dcc..fd05e5b 100644
--- a/pki/path_builder_unittest.cc
+++ b/pki/path_builder_unittest.cc
@@ -1900,9 +1900,9 @@
 
   EXPECT_CALL(*target_issuers_req, GetNext(_))
       // First async batch: return oldintermediate_.
-      .WillOnce(Invoke(AppendCertToList(oldintermediate_)))
+      .WillOnce(AppendCertToList(oldintermediate_))
       // Second async batch: return newintermediate_.
-      .WillOnce(Invoke(AppendCertToList(newintermediate_)));
+      .WillOnce(AppendCertToList(newintermediate_));
   {
     ::testing::InSequence s;
     // oldintermediate_ does not create a valid path, so both sync and async
@@ -1991,11 +1991,11 @@
 
   EXPECT_CALL(*target_issuers_req, GetNext(_))
       // First async batch: return oldintermediate_.
-      .WillOnce(Invoke(AppendCertToList(oldintermediate_)))
+      .WillOnce(AppendCertToList(oldintermediate_))
       // Second async batch: return a different copy of oldintermediate_ again.
-      .WillOnce(Invoke(AppendCertToList(oldintermediate_dupe)))
+      .WillOnce(AppendCertToList(oldintermediate_dupe))
       // Third async batch: return newintermediate_.
-      .WillOnce(Invoke(AppendCertToList(newintermediate_)));
+      .WillOnce(AppendCertToList(newintermediate_));
 
   {
     ::testing::InSequence s;