Deflake some AsyncSSLSocket tests.

Summary: Fulfilling starts destroying the evb in another thread which races with the socket closing on the evb.

Reviewed By: siyengar, ngoyal

Differential Revision: D5755271

fbshipit-source-id: ace37eee63e684c97ca0fe503293eee83514e0ac

diff --git a/folly/io/async/test/AsyncSSLSocketTest2.cpp b/folly/io/async/test/AsyncSSLSocketTest2.cpp
index ae9ef53fa8c4bb28fbf780f6b1eeb00019a28ade..c50b6a7db89459b9e978e54f7ee62686fb415479 100644 (file)
--- a/folly/io/async/test/AsyncSSLSocketTest2.cpp
+++ b/folly/io/async/test/AsyncSSLSocketTest2.cpp
@@ -206,13 +206,13 @@ class ConnectClient : public AsyncSocket::ConnectCallback {
   }
 
   void connectSuccess() noexcept override {
-    promise_.setValue(true);
     socket_.reset();
+    promise_.setValue(true);
   }
 
   void connectErr(const AsyncSocketException& /* ex */) noexcept override {
-    promise_.setValue(false);
     socket_.reset();
+    promise_.setValue(false);
   }
 
   void setCtx(std::shared_ptr<SSLContext> ctx) {