diff options
author | Abseil Team <absl-team@google.com> | 2020-08-03 14:34:37 (GMT) |
---|---|---|
committer | Derek Mauro <dmauro@google.com> | 2020-08-07 17:07:51 (GMT) |
commit | 48ec64092a38621210ecd835fde61d76861269c1 (patch) | |
tree | ddeb8bbadd118d867c404efdb7ef6d05ef14f5ce | |
parent | e6e2d3b7614ff4e6017d8968bd4c3f579133666e (diff) | |
download | googletest-48ec64092a38621210ecd835fde61d76861269c1.zip googletest-48ec64092a38621210ecd835fde61d76861269c1.tar.gz googletest-48ec64092a38621210ecd835fde61d76861269c1.tar.bz2 |
Googletest export
Fix DoAll to work with move-only sink arguments.
This changes types of the first n - 1 actions so that they only get a readonly
view of the arguments. The last action will accept move only objects.
PiperOrigin-RevId: 324600664
-rw-r--r-- | googlemock/docs/cheat_sheet.md | 2 | ||||
-rw-r--r-- | googlemock/include/gmock/gmock-actions.h | 17 | ||||
-rw-r--r-- | googlemock/test/gmock-generated-actions_test.cc | 9 |
3 files changed, 21 insertions, 7 deletions
diff --git a/googlemock/docs/cheat_sheet.md b/googlemock/docs/cheat_sheet.md index 85620f7..b47e00f 100644 --- a/googlemock/docs/cheat_sheet.md +++ b/googlemock/docs/cheat_sheet.md @@ -616,7 +616,7 @@ composite action - trying to do so will result in a run-time error. <!-- mdformat off(no multiline tables) --> | | | | :----------------------------- | :------------------------------------------ | -| `DoAll(a1, a2, ..., an)` | Do all actions `a1` to `an` and return the result of `an` in each invocation. The first `n - 1` sub-actions must return void. | +| `DoAll(a1, a2, ..., an)` | Do all actions `a1` to `an` and return the result of `an` in each invocation. The first `n - 1` sub-actions must return void and will receive a readonly view of the arguments. | | `IgnoreResult(a)` | Perform action `a` and ignore its result. `a` must not return void. | | `WithArg<N>(a)` | Pass the `N`-th (0-based) argument of the mock function to action `a` and perform it. | | `WithArgs<N1, N2, ..., Nk>(a)` | Pass the selected (0-based) arguments of the mock function to action `a` and perform it. | diff --git a/googlemock/include/gmock/gmock-actions.h b/googlemock/include/gmock/gmock-actions.h index 79054db..4f7c1dd 100644 --- a/googlemock/include/gmock/gmock-actions.h +++ b/googlemock/include/gmock/gmock-actions.h @@ -1032,8 +1032,13 @@ struct WithArgsAction { template <typename... Actions> struct DoAllAction { private: + template <typename T> + using NonFinalType = + typename std::conditional<std::is_scalar<T>::value, T, const T&>::type; + template <typename... Args, size_t... I> - std::vector<Action<void(Args...)>> Convert(IndexSequence<I...>) const { + std::vector<Action<void(NonFinalType<Args>...)>> Convert( + IndexSequence<I...>) const { return {std::get<I>(actions)...}; } @@ -1043,14 +1048,13 @@ struct DoAllAction { template <typename R, typename... Args> operator Action<R(Args...)>() const { // NOLINT struct Op { - std::vector<Action<void(Args...)>> converted; + std::vector<Action<void(NonFinalType<Args>...)>> converted; Action<R(Args...)> last; R operator()(Args... args) const { - auto tuple_args = std::forward_as_tuple(std::forward<Args>(args)...); for (auto& a : converted) { - a.Perform(tuple_args); + a.Perform(std::forward_as_tuple(std::forward<Args>(args)...)); } - return last.Perform(tuple_args); + return last.Perform(std::forward_as_tuple(std::forward<Args>(args)...)); } }; return Op{Convert<Args...>(MakeIndexSequence<sizeof...(Actions) - 1>()), @@ -1093,7 +1097,8 @@ struct DoAllAction { typedef internal::IgnoredValue Unused; // Creates an action that does actions a1, a2, ..., sequentially in -// each invocation. +// each invocation. All but the last action will have a readonly view of the +// arguments. template <typename... Action> internal::DoAllAction<typename std::decay<Action>::type...> DoAll( Action&&... action) { diff --git a/googlemock/test/gmock-generated-actions_test.cc b/googlemock/test/gmock-generated-actions_test.cc index ef39dff..3ec83e2 100644 --- a/googlemock/test/gmock-generated-actions_test.cc +++ b/googlemock/test/gmock-generated-actions_test.cc @@ -422,6 +422,15 @@ TEST(DoAllTest, TenActions) { EXPECT_EQ('g', g); } +TEST(DoAllTest, MoveOnlyArgs) { + bool ran_first = false; + Action<int(std::unique_ptr<int>)> a = + DoAll(InvokeWithoutArgs([&] { ran_first = true; }), + [](std::unique_ptr<int> p) { return *p; }); + EXPECT_EQ(7, a.Perform(std::make_tuple(std::unique_ptr<int>(new int(7))))); + EXPECT_TRUE(ran_first); +} + // The ACTION*() macros trigger warning C4100 (unreferenced formal // parameter) in MSVC with -W4. Unfortunately they cannot be fixed in // the macro definition, as the warnings are generated when the macro |