From 44c12101e5a6e85e1ecf7f9f402bac30891f61fb Mon Sep 17 00:00:00 2001 From: Hannes Roth Date: Wed, 5 Mar 2014 17:39:47 -0800 Subject: [PATCH] (Wangle) Remove some early returns in tests Summary: I guess they sneaked in when testing. Also remove an `EXPECT_NE` that depended on `std::move` actually moving. Test Plan: All the tests. Reviewed By: hans@fb.com FB internal diff: D1204971 --- folly/wangle/test/FutureTest.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/folly/wangle/test/FutureTest.cpp b/folly/wangle/test/FutureTest.cpp index e356d0de..e5906058 100644 --- a/folly/wangle/test/FutureTest.cpp +++ b/folly/wangle/test/FutureTest.cpp @@ -171,14 +171,10 @@ TEST(Promise, setValue) { pod = Promise(); fpod = pod.getFuture(); - return; pod.setValue(std::move(f2)); Foo f3 = fpod.value(); EXPECT_EQ(f.name, f3.name); EXPECT_EQ(f.value, f3.value); - EXPECT_NE(f.name, f2.name); - - return; Promise> mov; auto fmov = mov.getFuture(); -- 2.34.1