another test run

Author: nicohrubec

.github/workflows/build.yml

@@ -1180,11 +1180,11 @@ jobs:
                   per_page: 100
                 });
 
-                const failureAnnotations = annotations.filter(a => a.annotation_level === 'failure');
-                if (failureAnnotations.length > 0) {
-                  const testNames = failureAnnotations.map(a => {
-                    const firstLine = a.message.split('\n')[0];
-                    return `- \`${a.path}\`: ${firstLine}`;
+                const testAnnotations = annotations.filter(a => a.annotation_level === 'failure' && a.path !== '.github');
+                if (testAnnotations.length > 0) {
+                  const testNames = testAnnotations.map(a => {
+                    // The title field contains the full test name (e.g. "file.test.ts > Suite > test name")
+                    return a.title ? `- ${a.title}` : `- \`${a.path}\``;
                   });
                   // Deduplicate and limit to 20 entries to keep issues readable
                   testName = [...new Set(testNames)].slice(0, 20).join('\n');

dev-packages/browser-integration-tests/playwright.config.ts

@@ -30,7 +30,7 @@ const config: PlaywrightTestConfig = {
     },
   ],
 
-  reporter: process.env.CI ? [['list'], ['junit', { outputFile: 'results.junit.xml' }]] : 'list',
+  reporter: process.env.CI ? [['list'], ['github'], ['junit', { outputFile: 'results.junit.xml' }]] : 'list',
 
   globalSetup: require.resolve('./playwright.setup.ts'),
   globalTeardown: require.resolve('./playwright.teardown.ts'),

dev-packages/node-integration-tests/suites/public-api/setTag/with-primitives/test.ts

@@ -1,12 +1,12 @@
-import { afterAll, test } from 'vitest';
+import { afterAll, expect, test } from 'vitest';
 import { cleanupChildProcesses, createRunner } from '../../../../utils/runner';
 
 afterAll(() => {
   cleanupChildProcesses();
 });
 
-// TEMPORARY: Guaranteed failure to verify test name extraction in CI
-test('TEMPORARY - should fail to test annotation extraction', () => {
+// TEMPORARY: Guaranteed failure to verify test name extraction in CI (Node 18 only)
+test.skipIf(!process.version.startsWith('v18'))('TEMPORARY - should fail to test annotation extraction', () => {
   expect(true).toBe(false);
 });
 

dev-packages/test-utils/src/playwright-config.ts

@@ -37,7 +37,7 @@ export function getPlaywrightConfig(
     /* In dev mode some apps are flaky, so we allow retry there... */
     retries: testEnv === 'development' ? 3 : 0,
     /* Reporter to use. See https://playwright.dev/docs/test-reporters */
-    reporter: process.env.CI ? [['line'], ['junit', { outputFile: 'results.junit.xml' }]] : 'list',
+    reporter: process.env.CI ? [['line'], ['github'], ['junit', { outputFile: 'results.junit.xml' }]] : 'list',
     /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
     use: {
       /* Maximum time each action such as `click()` can take. Defaults to 0 (no limit). */

packages/core/test/lib/session.test.ts

@@ -4,8 +4,8 @@ import type { SessionContext } from '../../src/types-hoist/session';
 import { timestampInSeconds } from '../../src/utils/time';
 
 describe('Session', () => {
-  // TEMPORARY: Guaranteed failure to verify test name extraction in CI
-  it('TEMPORARY - should fail to test annotation extraction', () => {
+  // TEMPORARY: Guaranteed failure to verify test name extraction in CI (Node 18 only)
+  it.skipIf(process.env.NODE_VERSION !== '18')('TEMPORARY - should fail to test annotation extraction', () => {
     expect(true).toBe(false);
   });
 

vite/vite.config.ts

@@ -19,7 +19,9 @@ export default defineConfig({
         'vite.config.*',
       ],
     },
-    reporters: process.env.CI ? ['default', 'github-actions', ['junit', { classnameTemplate: '{filepath}' }]] : ['default'],
+    reporters: process.env.CI
+      ? ['default', 'github-actions', ['junit', { classnameTemplate: '{filepath}' }]]
+      : ['default'],
     outputFile: {
       junit: 'vitest.junit.xml',
     },