test: More Electron versions (#1273)

Author: timfish

test/e2e/test-apps/anr/anr-main-native/test.ts

@@ -7,81 +7,85 @@ import {
   UUID_MATCHER,
 } from '../../..';
 
-electronTestRunner(__dirname, { skip: (electronVersion) => electronVersion.major < 28 }, async (ctx) => {
-  await ctx
-    .includeSessionEnvelopes()
-    .ignoreExpectationOrder()
-    .expect({
-      envelope: sessionEnvelope({
-        sid: UUID_MATCHER,
-        init: true,
-        started: ISO_DATE_MATCHER,
-        timestamp: ISO_DATE_MATCHER,
-        status: 'abnormal',
-        errors: 0,
-        duration: expect.any(Number),
-        abnormal_mechanism: 'anr_foreground',
-        attrs: expect.objectContaining({
-          release: 'anr-main-native@1.0.0',
-          environment: 'development',
+electronTestRunner(
+  __dirname,
+  { skip: (electronVersion) => electronVersion.major < 28 || electronVersion.major > 37 },
+  async (ctx) => {
+    await ctx
+      .includeSessionEnvelopes()
+      .ignoreExpectationOrder()
+      .expect({
+        envelope: sessionEnvelope({
+          sid: UUID_MATCHER,
+          init: true,
+          started: ISO_DATE_MATCHER,
+          timestamp: ISO_DATE_MATCHER,
+          status: 'abnormal',
+          errors: 0,
+          duration: expect.any(Number),
+          abnormal_mechanism: 'anr_foreground',
+          attrs: expect.objectContaining({
+            release: 'anr-main-native@1.0.0',
+            environment: 'development',
+          }),
         }),
-      }),
-    })
-    .expect({
-      envelope: eventEnvelopeNoLiveContext({
-        contexts: {
-          trace: undefined,
-        },
-        breadcrumbs: undefined,
-        platform: 'node',
-        level: 'error',
-        exception: {
-          values: [
-            {
-              type: 'EventLoopBlocked',
-              value: 'Event Loop Blocked for at least 1000 ms',
-              thread_id: '0',
-              stacktrace: {
-                frames: expect.arrayContaining([
-                  {
-                    filename: expect.stringContaining('app:///src/main.'),
-                    function: 'longWork',
-                    colno: expect.any(Number),
-                    lineno: expect.any(Number),
-                    in_app: true,
-                  },
-                ]),
+      })
+      .expect({
+        envelope: eventEnvelopeNoLiveContext({
+          contexts: {
+            trace: undefined,
+          },
+          breadcrumbs: undefined,
+          platform: 'node',
+          level: 'error',
+          exception: {
+            values: [
+              {
+                type: 'EventLoopBlocked',
+                value: 'Event Loop Blocked for at least 1000 ms',
+                thread_id: '0',
+                stacktrace: {
+                  frames: expect.arrayContaining([
+                    {
+                      filename: expect.stringContaining('app:///src/main.'),
+                      function: 'longWork',
+                      colno: expect.any(Number),
+                      lineno: expect.any(Number),
+                      in_app: true,
+                    },
+                  ]),
+                },
+                mechanism: { type: 'ANR' },
               },
-              mechanism: { type: 'ANR' },
-            },
-          ],
-        },
-        threads: {
-          values: [
-            {
-              crashed: true,
-              current: true,
-              id: '0',
-              main: true,
-              name: 'main',
-            },
-          ],
-        },
-        tags: {
-          'event.environment': 'javascript',
-          'event.origin': 'electron',
-          'event.process': 'browser',
-        },
-        debug_meta: {
-          images: [
-            {
-              type: 'sourcemap',
-              code_file: expect.stringContaining('app:///src/main.'),
-              debug_id: 'aaaaaaaa-aaaa-4aaa-aaaa-aaaaaaaaaa',
-            },
-          ],
-        },
-      }),
-    })
-    .run();
-});
+            ],
+          },
+          threads: {
+            values: [
+              {
+                crashed: true,
+                current: true,
+                id: '0',
+                main: true,
+                name: 'main',
+              },
+            ],
+          },
+          tags: {
+            'event.environment': 'javascript',
+            'event.origin': 'electron',
+            'event.process': 'browser',
+          },
+          debug_meta: {
+            images: [
+              {
+                type: 'sourcemap',
+                code_file: expect.stringContaining('app:///src/main.'),
+                debug_id: 'aaaaaaaa-aaaa-4aaa-aaaa-aaaaaaaaaa',
+              },
+            ],
+          },
+        }),
+      })
+      .run();
+  },
+);

test/e2e/test-apps/sessions/native-crash-main/src/main.js

@@ -20,17 +20,16 @@ app.on('ready', () => {
   });
 
   mainWindow.loadFile(path.join(__dirname, 'index.html'));
-});
-
-// We only crash on the first run
-// The second run is where the crash is uploaded
 
-if (process.env.APP_FIRST_RUN) {
-  setTimeout(() => {
-    process.crash();
-  }, 1000);
-} else {
-  setTimeout(() => {
-    app.quit();
-  }, 4000);
-}
+  // We only crash on the first run
+  // The second run is where the crash is uploaded
+  if (process.env.APP_FIRST_RUN) {
+    setTimeout(() => {
+      process.crash();
+    }, 1000);
+  } else {
+    setTimeout(() => {
+      app.quit();
+    }, 4000);
+  }
+});

test/e2e/versions.json

@@ -1 +1 @@
-["23.3.13","24.8.8","25.9.8","26.6.10","27.3.11","28.3.3","29.4.6","30.5.1","31.7.7","32.3.3","33.4.11","34.5.8","35.7.2","36.7.1","37.2.2"]
+["23.3.13","24.8.8","25.9.8","26.6.10","27.3.11","28.3.3","29.4.6","30.5.1","31.7.7","32.3.3","33.4.11","34.5.8","35.7.5","36.9.5","37.10.3","38.7.2","39.2.7"]