From a656c31a70d7490a23ce0caf108aa9202f0e5104 Mon Sep 17 00:00:00 2001
From: John Knox <jknox@fb.com>
Date: Tue, 4 Dec 2018 03:25:28 -0800
Subject: [PATCH] Clean up connection test

Summary: Replace the console.error and lack of calling done(), with an explicit expect() call to make failure results clearer.

Reviewed By: danielbuechele

Differential Revision: D13254777

fbshipit-source-id: 602081323ad3cc1432ddb706f49f036f0fb8926d
---
 src/__tests__/server.device.js | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/__tests__/server.device.js b/src/__tests__/server.device.js
index 01fe94980..fe75c2ba3 100644
--- a/src/__tests__/server.device.js
+++ b/src/__tests__/server.device.js
@@ -55,11 +55,8 @@ test(
       // Make sure client stays connected for some time before passing test
       setTimeout(() => {
         testFinished = true;
-        if (disconnectedTooEarly) {
-          console.error('Client disconnected too early');
-        } else {
-          done();
-        }
+        expect(disconnectedTooEarly).toBe(false);
+        done();
       }, 5000);
     });
     server.addListener('removed-client', (id: string) => {