Deserialize newParticipant on the front end

gh-250
douglasnaphas · Mar 28, 2022 · ee72295 · ee72295
1 parent 4a5238e
commit ee72295
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 9 deletions.
diff --git a/frontend/src/components/RosterPage.js b/frontend/src/components/RosterPage.js
@@ -115,15 +115,24 @@ class RosterPage extends Component {
         `gamename=${encodeURIComponent(gameName)}`
     );
     webSocket.addEventListener("message", (event) => {
-      if (event.data.newParticipant) {
-        this.setState((state, props) => {
-          return {
-            participants: state.participants.concat([
-              event.data.newParticipant,
-            ]),
-          };
-        });
+      if (!event) {
+        return;
       }
+      if (!event.data) {
+        return;
+      }
+      const eventData = JSON.parse(event.data);
+      if (!eventData) {
+        return;
+      }
+      if (!eventData.newParticipant) {
+        return;
+      }
+      this.setState((state, props) => {
+        return {
+          participants: state.participants.concat([eventData.newParticipant]),
+        };
+      });
     });
   }
   componentWillUnmount() {

diff --git a/frontend/src/components/RosterPage.test.js b/frontend/src/components/RosterPage.test.js
@@ -219,7 +219,7 @@ describe("RosterPage", () => {
     );
     expect(mockWebSocketConstructorCalls).toEqual(1);
     const testMessage = new MessageEvent("message", {
-      data: { newParticipant: "Em Three" },
+      data: `{"newParticipant":"Em Three"}`,
     });
     let table = await screen.findByRole("table");
     await findByText(table, "Je Teste");