Remove redundant check for event_id response field

Author: AndrewFerr

src/embedded.ts

@@ -311,8 +311,9 @@ export class RoomWidgetClient extends MatrixClient {
             throw e;
         }
 
+        // This also checks for an event id on the response
         room.updatePendingEvent(event, EventStatus.SENT, response.event_id);
-        return this.validateSendEventResponse(response);
+        return { event_id: response.event_id! };
     }
 
     public async sendStateEvent(
@@ -322,7 +323,10 @@ export class RoomWidgetClient extends MatrixClient {
         stateKey = "",
     ): Promise<ISendEventResponse> {
         const response = await this.widgetApi.sendStateEvent(eventType, stateKey, content, roomId);
-        return this.validateSendEventResponse(response);
+        if (response.event_id === undefined) {
+            throw new Error("'event_id' absent from response to an event request");
+        }
+        return { event_id: response.event_id };
     }
 
     /**
@@ -351,13 +355,6 @@ export class RoomWidgetClient extends MatrixClient {
         return this.validateSendDelayedEventResponse(response);
     }
 
-    private validateSendEventResponse(response: ISendEventFromWidgetResponseData): ISendEventResponse {
-        if (response.event_id === undefined) {
-            throw new Error("'event_id' absent from response to an event request");
-        }
-        return { event_id: response.event_id };
-    }
-
     private validateSendDelayedEventResponse(response: ISendEventFromWidgetResponseData): SendDelayedEventResponse {
         if (response.delay_id === undefined) {
             throw new Error("'delay_id' absent from response to a delayed event request");