fix(browser-integration-tests): Replace deprecated envelope helpers in viewHierarchy test

cursoragent · claude · cursoragent · commit 5816421e7551 · 2026-04-13T16:38:03.000Z
Replace unreliable getMultipleSentryEnvelopeRequests and envelopeParser
with waitForErrorRequest and properFullEnvelopeRequestParser for more
reliable test execution.

Co-Authored-By: Claude Sonnet 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/dev-packages/browser-integration-tests/suites/integrations/viewHierarchy/test.ts b/dev-packages/browser-integration-tests/suites/integrations/viewHierarchy/test.ts
@@ -1,7 +1,7 @@
 import { expect } from '@playwright/test';
 import type { ViewHierarchyData } from '@sentry/core';
 import { sentryTest } from '../../../utils/fixtures';
-import { getMultipleSentryEnvelopeRequests, envelopeParser } from '../../../utils/helpers';
+import { properFullEnvelopeRequestParser, waitForErrorRequest } from '../../../utils/helpers';
 
 sentryTest('Captures view hierarchy as attachment', async ({ getLocalTestUrl, page }) => {
   const bundle = process.env.PW_BUNDLE;
@@ -11,18 +11,21 @@ sentryTest('Captures view hierarchy as attachment', async ({ getLocalTestUrl, pa
 
   const url = await getLocalTestUrl({ testDir: __dirname });
 
-  const [, events] = await Promise.all([
-    page.goto(url),
-    getMultipleSentryEnvelopeRequests<ViewHierarchyData>(
-      page,
-      1,
-      {},
-      req => envelopeParser(req)?.[4] as ViewHierarchyData,
-    ),
-  ]);
-
-  expect(events).toHaveLength(1);
-  const event: ViewHierarchyData = events[0];
+  const req = await Promise.all([page.goto(url), waitForErrorRequest(page)]).then(([, req]) => req);
+
+  const envelope = properFullEnvelopeRequestParser(req);
+  const items = envelope[1];
+
+  expect(items).toHaveLength(2);
+
+  const attachmentHeader = items[1][0];
+  expect(attachmentHeader.type).toBe('attachment');
+  expect(attachmentHeader.filename).toBe('view-hierarchy.json');
+
+  const attachmentData = items[1][1];
+  const event: ViewHierarchyData = JSON.parse(
+    typeof attachmentData === 'string' ? attachmentData : new TextDecoder().decode(attachmentData as Uint8Array),
+  );
 
   expect(event.rendering_system).toBe('DOM');
   expect(event.positioning).toBe('absolute');
