MockQuicSocketDriver fix ::good() impl

Summary:
- fixes the `::good()` impl for `MockQuicSocketDriver`

- after `::close`, `::good()` should return false

- used by next diff in stack

Reviewed By: sharmafb

Differential Revision: D54918964

fbshipit-source-id: 5c173561378b72b1fe2b845eba86a9c730ce90c1

proxygen/lib/http/session/test/MockQuicSocketDriver.h

@@ -1194,7 +1194,6 @@ class MockQuicSocketDriver : public folly::EventBase::LoopCallback {
   }
 
   void closeImpl(folly::Optional<QuicError> errorCode) {
-
     closeConnection();
     if (errorCode) {
       quic::ApplicationErrorCode* err =
@@ -1208,6 +1207,7 @@ class MockQuicSocketDriver : public folly::EventBase::LoopCallback {
         QuicError(LocalErrorCode::NO_ERROR, "Closing socket with no error")));
     sock_->connCb_ = nullptr;
     sock_->setupCb_ = nullptr;
+    sockGood_ = false;
   }
   folly::Optional<quic::ApplicationErrorCode> getConnErrorCode() {
     return streams_[kConnectionStreamId].error;