Fix test

Author: FrederikBolding

packages/snaps-controllers/src/services/webview/WebViewMessageStream.ts

@@ -65,6 +65,8 @@ export class WebViewMessageStream extends BasePostMessageStream {
       data,
     });
 
+    // To prevent XSS, we encode the message before injecting it.
+    // This adds significant performance overhead for larger messages.
     const bytes = new Uint8Array(stringToBytes(json));
 
     this.#webView.injectJavaScript(`window.postMessage([${bytes.toString()}])`);

packages/snaps-execution-environments/src/webview/WebViewExecutorStream.test.ts

@@ -1,5 +1,5 @@
 import { sleep } from '@metamask/snaps-utils/test-utils';
-import { bytesToBase64, stringToBytes } from '@metamask/utils';
+import { stringToBytes } from '@metamask/utils';
 
 import { WebViewExecutorStream } from './WebViewExecutorStream';
 
@@ -8,9 +8,8 @@ describe('WebViewExecutorStream', () => {
     const addEventListener = jest.fn();
     const postMessage = jest.fn().mockImplementation((message) => {
       const bytes = stringToBytes(message);
-      const base64 = bytesToBase64(bytes);
       addEventListener.mock.calls.forEach(([_type, listener]) => {
-        setTimeout(() => listener({ data: base64 }));
+        setTimeout(() => listener({ data: Array.from(bytes) }));
       });
     });
     const mockWindow = {