lint

Author: billyvg

dev-packages/browser-integration-tests/suites/feedback/captureFeedbackAndReplay/hasSampling/test.ts

@@ -36,10 +36,7 @@ sentryTest('should capture feedback (@sentry-internal/feedback import)', async (
 
   const url = await getLocalTestPath({ testDir: __dirname });
 
-  await Promise.all([
-    page.goto(url),
-    page.getByText('Report a Bug').click(),
-  ]);
+  await Promise.all([page.goto(url), page.getByText('Report a Bug').click()]);
 
   await page.locator('[name="name"]').fill('Jane Doe');
   await page.locator('[name="email"]').fill('[email protected]');

dev-packages/browser-integration-tests/suites/feedback/captureFeedbackAndReplay/noSampling/test.ts

@@ -1,28 +1,30 @@
 import { expect } from '@playwright/test';
 
 import { sentryTest } from '../../../../utils/fixtures';
-import { envelopeRequestParser, getEnvelopeType } from '../../../../utils/helpers';
-import { getCustomRecordingEvents, getReplayEvent, waitForReplayRequest } from '../../../../utils/replayHelpers';
+import { getReplayEvent, waitForReplayRequest } from '../../../../utils/replayHelpers';
 
-sentryTest('should capture feedback with no replay sampling when Form opens (@sentry-internal/feedback import)', async ({ getLocalTestPath, page }) => {
-  if (process.env.PW_BUNDLE) {
-    sentryTest.skip();
-  }
+sentryTest(
+  'should capture feedback with no replay sampling when Form opens (@sentry-internal/feedback import)',
+  async ({ getLocalTestPath, page }) => {
+    if (process.env.PW_BUNDLE) {
+      sentryTest.skip();
+    }
 
-  const reqPromise0 = waitForReplayRequest(page, 0);
-  await page.route('https://dsn.ingest.sentry.io/**/*', route => {
-    return route.fulfill({
-      status: 200,
-      contentType: 'application/json',
-      body: JSON.stringify({ id: 'test-id' }),
+    const reqPromise0 = waitForReplayRequest(page, 0);
+    await page.route('https://dsn.ingest.sentry.io/**/*', route => {
+      return route.fulfill({
+        status: 200,
+        contentType: 'application/json',
+        body: JSON.stringify({ id: 'test-id' }),
+      });
     });
-  });
 
-  const url = await getLocalTestPath({ testDir: __dirname });
+    const url = await getLocalTestPath({ testDir: __dirname });
 
-  const [,,replayReq] = await Promise.all([page.goto(url), page.getByText('Report a Bug').click(), reqPromise0]);
+    const [, , replayReq] = await Promise.all([page.goto(url), page.getByText('Report a Bug').click(), reqPromise0]);
 
-  const replayEvent = getReplayEvent(replayReq);
-  expect(replayEvent.segment_id).toBe(0);
-  expect(replayEvent.replay_type).toBe('buffer');
-});
+    const replayEvent = getReplayEvent(replayReq);
+    expect(replayEvent.segment_id).toBe(0);
+    expect(replayEvent.replay_type).toBe('buffer');
+  },
+);

packages/feedback/src/widget/createWidget.ts

@@ -129,7 +129,7 @@ export function createWidget({
   /**
    * Internal handler when dialog is opened
    */
-  function handleOpenDialog({includeReplay}: {includeReplay?: boolean} = {}) {
+  function handleOpenDialog({ includeReplay }: { includeReplay?: boolean } = {}) {
     // Flush replay if it exists
     if (includeReplay) {
       const client = getClient();
@@ -138,8 +138,8 @@ export function createWidget({
         return;
       }
       try {
-      // @ts-expect-error Not sure how best to type this w/o
-      // making @sentry/replay a dep
+        // @ts-expect-error Not sure how best to type this w/o
+        // making @sentry/replay a dep
         void replay.flush();
       } catch (err) {
         DEBUG_BUILD && logger.error(err);
@@ -179,7 +179,7 @@ export function createWidget({
         if (options.onFormOpen) {
           options.onFormOpen();
         }
-        handleOpenDialog({includeReplay: options.includeReplay});
+        handleOpenDialog({ includeReplay: options.includeReplay });
         return;
       }
 
@@ -232,7 +232,7 @@ export function createWidget({
       if (options.onFormOpen) {
         options.onFormOpen();
       }
-      handleOpenDialog({includeReplay: options.includeReplay});
+      handleOpenDialog({ includeReplay: options.includeReplay });
     } catch (err) {
       // TODO: Error handling?
       logger.error(err);