Merge pull request #24 from takker99/fix-connecting

🐛 socketの接続・遮断前にlistenerを登録しないといけなかった

Author: takker99

browser/websocket/socket.ts

@@ -12,11 +12,12 @@ export function makeSocket() {
  */
 export async function connect(socket: Socket): Promise<void> {
   if (socket.connected) return;
-  socket.connect();
 
-  return await new Promise<void>((resolve) =>
+  const waiting = new Promise<void>((resolve) =>
     socket.once("connect", () => resolve())
   );
+  socket.connect();
+  await waiting;
 }
 
 /** websocketを切断する
@@ -25,14 +26,15 @@ export async function connect(socket: Socket): Promise<void> {
  */
 export async function disconnect(socket: Socket): Promise<void> {
   if (socket.disconnected) return;
-  socket.disconnect();
 
-  return await new Promise<void>((resolve) => {
+  const waiting = new Promise<void>((resolve) => {
     const onDisconnect = (reason: Socket.DisconnectReason) => {
       if (reason !== "io client disconnect") return;
       resolve();
       socket.off("disconnect", onDisconnect);
     };
     socket.on("disconnect", onDisconnect);
   });
+  socket.disconnect();
+  await waiting;
 }