From 19a3bbce512dc2c4959769134fddd52b27989003 Mon Sep 17 00:00:00 2001
From: Piotr Nycz <piotrwn1@gmail.com>
Date: Tue, 22 Oct 2019 18:41:35 +0200
Subject: [PATCH] Added tests verifying that temporaries are accepted by
 ReturnRef

Issue no 2527
---
 googlemock/test/gmock-actions_test.cc | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/googlemock/test/gmock-actions_test.cc b/googlemock/test/gmock-actions_test.cc
index f63c8c5a..99d273aa 100644
--- a/googlemock/test/gmock-actions_test.cc
+++ b/googlemock/test/gmock-actions_test.cc
@@ -646,6 +646,30 @@ TEST(ReturnRefTest, IsCovariant) {
   EXPECT_EQ(&derived, &a.Perform(std::make_tuple()));
 }
 
+namespace
+{
+template <typename T, typename = decltype(ReturnRef(std::declval<T&&>()))>
+bool CanCallReturnRef(T&&) { return true; }
+bool CanCallReturnRef(Unused) { return false; }
+}
+
+// Tests that ReturnRef(v) is not working with temporaries (T&&)
+TEST(ReturnRefTest, WillNotAcceptTemporaryAkaRValueRef) {
+  int value = 13;
+  EXPECT_TRUE(CanCallReturnRef(value));
+  EXPECT_FALSE(CanCallReturnRef(std::move(value)));
+  EXPECT_FALSE(CanCallReturnRef(value + 1));
+  EXPECT_FALSE(CanCallReturnRef(123));
+}
+
+// Tests that ReturnRef(v) is not working with const temporaries (const T&&)
+TEST(ReturnRefTest, WillNotAcceptConstTemporaryAkaContRValueRef) {
+  const int value = 42;
+  EXPECT_TRUE(CanCallReturnRef(value));
+  EXPECT_FALSE(CanCallReturnRef(std::move(value)));
+}
+
+
 // Tests that ReturnRefOfCopy(v) works for reference types.
 TEST(ReturnRefOfCopyTest, WorksForReference) {
   int n = 42;