Fix removal of event listeners for runSocket

The code previously used was only working with the patched Sentry method, not when Sentry was inactive. Since a Websocket is inheriting from EventTarget, the method signature usually requires two arguments.

See https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/removeEventListener

Author: MrSerth

app/assets/javascripts/editor/execution.js

@@ -49,21 +49,25 @@ CodeOceanEditorWebsocket = {
         // Remove event listeners for Promise handling.
         // This is especially useful in case of an error, where a `close` event might follow the `error` event.
         const teardown = () => {
-          this.websocket.websocket.removeEventListener(closeListener);
-          this.websocket.websocket.removeEventListener(errorListener);
+          this.websocket.websocket.removeEventListener('close', closeListener);
+          this.websocket.websocket.removeEventListener('error', errorListener);
         };
 
-        // We are using event listeners (and not `onError` or `onClose`) here, since these listeners should never be overwritten.
-        // With `onError` or `onClose`, a new assignment would overwrite a previous one.
-        const closeListener = this.websocket.websocket.addEventListener('close', () => {
+        const closeListener = () => {
           resolve();
           teardown();
-        });
-        const errorListener = this.websocket.websocket.addEventListener('error', (error) => {
+        }
+
+        const errorListener = (error) => {
           reject(error);
           teardown();
           this.websocket.killWebSocket(); // In case of error, ensure we always close the connection.
-        });
+        }
+
+        // We are using event listeners (and not `onError` or `onClose`) here, since these listeners should never be overwritten.
+        // With `onError` or `onClose`, a new assignment would overwrite a previous one.
+        this.websocket.websocket.addEventListener('close', closeListener);
+        this.websocket.websocket.addEventListener('error', errorListener);
       });
     });
   },