auto fix tests

Author: mydea

dev-packages/e2e-tests/lib/getTestMatrix.ts

@@ -101,7 +101,7 @@ function addIncludesForTestApp(
 }
 
 function getSentryDependencies(appName: string): string[] {
-  const packageJson = getPackageJson(appName) || {};
+  const packageJson = getPackageJson(appName);
 
   const dependencies = {
     ...packageJson.devDependencies,

packages/browser/test/sdk.test.ts

@@ -89,7 +89,7 @@ describe('init', () => {
     expect(initAndBindSpy).toHaveBeenCalledTimes(1);
 
     const optionsPassed = initAndBindSpy.mock.calls[0]?.[1];
-    expect(optionsPassed?.integrations?.length).toBeGreaterThan(0);
+    expect(optionsPassed?.integrations.length).toBeGreaterThan(0);
   });
 
   test("doesn't install default integrations if told not to", () => {

packages/core/test/lib/envelope.test.ts

@@ -116,7 +116,7 @@ describe('createSpanEnvelope', () => {
 
     const spanEnvelope = createSpanEnvelope([span]);
 
-    const spanItem = spanEnvelope[1]?.[0]?.[1];
+    const spanItem = spanEnvelope[1][0]?.[1];
     expect(spanItem).toEqual({
       data: {
         'sentry.origin': 'manual',
@@ -207,7 +207,7 @@ describe('createSpanEnvelope', () => {
 
     expect(beforeSendSpan).toHaveBeenCalled();
 
-    const spanItem = spanEnvelope[1]?.[0]?.[1];
+    const spanItem = spanEnvelope[1][0]?.[1];
     expect(spanItem).toEqual({
       data: {
         'sentry.origin': 'manual',
@@ -242,7 +242,7 @@ describe('createSpanEnvelope', () => {
 
     expect(beforeSendSpan).toHaveBeenCalled();
 
-    const spanItem = spanEnvelope[1]?.[0]?.[1];
+    const spanItem = spanEnvelope[1][0]?.[1];
     expect(spanItem).toEqual({
       data: {
         'sentry.origin': 'manual',

packages/core/test/lib/transports/multiplexed.test.ts

@@ -118,7 +118,7 @@ describe('makeMultiplexedTransport', () => {
     const makeTransport = makeMultiplexedTransport(
       createTestTransport((url, _, env) => {
         expect(url).toBe(DSN2_URL);
-        expect(env[0]?.dsn).toBe(DSN2);
+        expect(env[0].dsn).toBe(DSN2);
       }),
       () => [DSN2],
     );
@@ -134,7 +134,7 @@ describe('makeMultiplexedTransport', () => {
       createTestTransport((url, release, env) => {
         expect(url).toBe(DSN2_URL);
         expect(release).toBe('[email protected]');
-        expect(env[0]?.dsn).toBe(DSN2);
+        expect(env[0].dsn).toBe(DSN2);
       }),
       () => [{ dsn: DSN2, release: '[email protected]' }],
     );
@@ -150,7 +150,7 @@ describe('makeMultiplexedTransport', () => {
       createTestTransport((url, release, env) => {
         expect(url).toBe('http://google.com');
         expect(release).toBe('[email protected]');
-        expect(env[0]?.dsn).toBe(DSN2);
+        expect(env[0].dsn).toBe(DSN2);
       }),
       () => [{ dsn: DSN2, release: '[email protected]' }],
     );

packages/core/test/lib/transports/offline.test.ts

@@ -328,7 +328,7 @@ describe('makeOfflineTransport', () => {
       // When it gets shifted out of the store, the sent_at header should be updated
       const envelopes = getSentEnvelopes().map(parseEnvelope) as EventEnvelope[];
       expect(envelopes[0]?.[0]).toBeDefined();
-      const sent_at = new Date(envelopes[0]![0]?.sent_at);
+      const sent_at = new Date(envelopes[0]![0].sent_at);
 
       expect(sent_at.getTime()).toBeGreaterThan(testStartTime.getTime());
     },

packages/core/test/utils-hoist/envelope.test.ts

@@ -71,7 +71,7 @@ describe('envelope', () => {
         measurements: { inp: { value: expect.any(Number), unit: expect.any(String) } },
       };
 
-      expect(spanEnvelopeItem[0]?.type).toBe('span');
+      expect(spanEnvelopeItem[0].type).toBe('span');
       expect(spanEnvelopeItem[1]).toMatchObject(expectedObj);
     });
   });

packages/core/test/utils-hoist/misc.test.ts

@@ -215,7 +215,7 @@ describe('addExceptionMechanism', () => {
 
     addExceptionMechanism(event);
 
-    expect(event.exception.values[0]?.mechanism).toEqual(defaultMechanism);
+    expect(event.exception.values[0].mechanism).toEqual(defaultMechanism);
   });
 
   it('prefers current values to defaults', () => {
@@ -226,7 +226,7 @@ describe('addExceptionMechanism', () => {
 
     addExceptionMechanism(event);
 
-    expect(event.exception.values[0]?.mechanism).toEqual(nonDefaultMechanism);
+    expect(event.exception.values[0].mechanism).toEqual(nonDefaultMechanism);
   });
 
   it('prefers incoming values to current values', () => {
@@ -239,7 +239,7 @@ describe('addExceptionMechanism', () => {
     addExceptionMechanism(event, newMechanism);
 
     // the new `handled` value took precedence
-    expect(event.exception.values[0]?.mechanism).toEqual({ type: 'instrument', handled: true, synthetic: true });
+    expect(event.exception.values[0].mechanism).toEqual({ type: 'instrument', handled: true, synthetic: true });
   });
 
   it('merges data values', () => {
@@ -251,7 +251,7 @@ describe('addExceptionMechanism', () => {
 
     addExceptionMechanism(event, newMechanism);
 
-    expect(event.exception.values[0]?.mechanism.data).toEqual({
+    expect(event.exception.values[0].mechanism.data).toEqual({
       function: 'addEventListener',
       handler: 'organizeShoes',
       target: 'closet',

packages/feedback/src/core/components/Actor.test.ts

@@ -22,7 +22,7 @@ describe('Actor', () => {
     const actorComponent = feedback!.createWidget();
 
     expect(actorComponent.el).toBeInstanceOf(HTMLButtonElement);
-    expect(actorComponent.el?.textContent).toBe(TRIGGER_LABEL);
+    expect(actorComponent.el.textContent).toBe(TRIGGER_LABEL);
   });
 
   it('renders the correct aria label for the button', () => {
@@ -43,19 +43,19 @@ describe('Actor', () => {
     // aria label is the same as trigger label when the trigger label isn't empty
     const actorDefault = feedback!.createWidget({ triggerLabel: 'Button' });
 
-    expect(actorDefault.el?.textContent).toBe('Button');
-    expect(actorDefault.el?.ariaLabel).toBe('Button');
+    expect(actorDefault.el.textContent).toBe('Button');
+    expect(actorDefault.el.ariaLabel).toBe('Button');
 
     // aria label is default text when trigger label is empty and aria isn't configured
     const actorIcon = feedback!.createWidget({ triggerLabel: '' });
 
-    expect(actorIcon.el?.textContent).toBe('');
-    expect(actorIcon.el?.ariaLabel).toBe(TRIGGER_LABEL);
+    expect(actorIcon.el.textContent).toBe('');
+    expect(actorIcon.el.ariaLabel).toBe(TRIGGER_LABEL);
 
     // aria label is the triggerAriaLabel if it's configured
     const actorAria = feedback!.createWidget({ triggerLabel: 'Button', triggerAriaLabel: 'Aria' });
 
-    expect(actorAria.el?.textContent).toBe('Button');
-    expect(actorAria.el?.ariaLabel).toBe('Aria');
+    expect(actorAria.el.textContent).toBe('Button');
+    expect(actorAria.el.ariaLabel).toBe('Aria');
   });
 });

packages/node/test/integration/transactions.test.ts

@@ -517,7 +517,7 @@ describe('Integration | Transactions', () => {
 
     // Checking the spans here, as they are circular to the transaction...
     const runArgs = beforeSendTransaction.mock.calls[0] as unknown as [TransactionEvent, unknown];
-    const spans = runArgs[0]?.spans || [];
+    const spans = runArgs[0].spans || [];
 
     // note: Currently, spans do not have any context/span added to them
     // This is the same behavior as for the "regular" SDKs

packages/opentelemetry/test/integration/transactions.test.ts

@@ -399,7 +399,7 @@ describe('Integration | Transactions', () => {
 
     // Checking the spans here, as they are circular to the transaction...
     const runArgs = beforeSendTransaction.mock.calls[0] as unknown as [TransactionEvent, unknown];
-    const spans = runArgs[0]?.spans || [];
+    const spans = runArgs[0].spans || [];
 
     // note: Currently, spans do not have any context/span added to them
     // This is the same behavior as for the "regular" SDKs