Fix use-after-move in AeadTicketCipherTest.

Summary: This "worked" before since we only cared about the time_point, but was still wrong.

Reviewed By: zalecodez

Differential Revision: D51535310

fbshipit-source-id: b370352a741d0d004cd76d98474ede45d592a4b1

fizz/server/test/AeadTicketCipherTest.cpp

@@ -141,6 +141,13 @@ class AeadTicketCipherTest : public Test {
       policy_.setHandshakeValidity(*handshakeValidity);
     }
   }
+
+  static ResumptionState makeState(
+      std::chrono::system_clock::time_point handshakeTime) {
+    ResumptionState state;
+    state.handshakeTime = handshakeTime;
+    return state;
+  }
 };
 
 TEST_F(AeadTicketCipherTest, TestEncryptNoTicketSecrets) {
@@ -178,9 +185,7 @@ TEST_F(AeadTicketCipherTest, TestHandshakeExpiration) {
         res.handshakeTime = time;
         return res;
       }));
-  ResumptionState state;
-  state.handshakeTime = time;
-  auto result = cipher_.encrypt(std::move(state)).get();
+  auto result = cipher_.encrypt(makeState(time)).get();
   EXPECT_TRUE(result.has_value());
   EXPECT_TRUE(IOBufEqualTo()(result->first, toIOBuf(ticket1)));
   EXPECT_EQ(result->second, std::chrono::seconds(2));
@@ -209,25 +214,23 @@ TEST_F(AeadTicketCipherTest, TestTicketLifetime) {
 
   // At handshake time, expect ticket validity.
   EXPECT_CALL(*clock_, getCurrentTime()).WillOnce(Return(time));
-  ResumptionState state;
-  state.handshakeTime = time;
-  auto result = cipher_.encrypt(std::move(state)).get();
+  auto result = cipher_.encrypt(makeState(time)).get();
   EXPECT_TRUE(result.has_value());
   EXPECT_TRUE(IOBufEqualTo()(result->first, toIOBuf(ticket1)));
   EXPECT_EQ(result->second, std::chrono::seconds(2));
 
   // At 3 seconds in, expect 1 second (remaining handshake validity)
   EXPECT_CALL(*clock_, getCurrentTime())
       .WillOnce(Return(time + std::chrono::seconds(3)));
-  auto result2 = cipher_.encrypt(std::move(state)).get();
+  auto result2 = cipher_.encrypt(makeState(time)).get();
   EXPECT_TRUE(result2.has_value());
   EXPECT_TRUE(IOBufEqualTo()(result2->first, toIOBuf(ticket1)));
   EXPECT_EQ(result2->second, std::chrono::seconds(1));
 
   // 5 seconds in, no longer valid. Expect none.
   EXPECT_CALL(*clock_, getCurrentTime())
       .WillOnce(Return(time + std::chrono::seconds(5)));
-  auto result3 = cipher_.encrypt(std::move(state)).get();
+  auto result3 = cipher_.encrypt(makeState(time)).get();
   EXPECT_FALSE(result3.has_value());
 }
 
@@ -238,9 +241,8 @@ TEST_F(AeadTicketCipherTest, TestEncryptExpiredHandshakeTicket) {
   auto time = std::chrono::system_clock::now();
   EXPECT_CALL(*clock_, getCurrentTime()).WillOnce(Return(time));
 
-  ResumptionState state;
-  state.handshakeTime = time - std::chrono::seconds(5);
-  auto result = cipher_.encrypt(std::move(state)).get();
+  auto result =
+      cipher_.encrypt(makeState(time - std::chrono::seconds(5))).get();
   EXPECT_FALSE(result.has_value());
 }
 
@@ -254,11 +256,10 @@ TEST_F(AeadTicketCipherTest, TestEncryptTicketFromFuture) {
   EXPECT_CALL(codec_, _encode(_)).WillOnce(InvokeWithoutArgs([]() {
     return IOBuf::copyBuffer("encodedticket");
   }));
-  ResumptionState state;
   // Ticket was created in the future. Validity period should be equal
   // to maximum (as we can't be sure how old it really is)
-  state.handshakeTime = time + std::chrono::seconds(5);
-  auto result = cipher_.encrypt(std::move(state)).get();
+  auto result =
+      cipher_.encrypt(makeState(time + std::chrono::seconds(5))).get();
   EXPECT_TRUE(result.has_value());
   EXPECT_TRUE(IOBufEqualTo()(result->first, toIOBuf(ticket1)));
   EXPECT_EQ(result->second, std::chrono::seconds(2));