Skip to content

Commit 4086dce

Browse files
committed
Adjust parallelization and reporting
1 parent 4f5527e commit 4086dce

File tree

2 files changed

+29
-34
lines changed

2 files changed

+29
-34
lines changed

typescript-sdk/apps/dojo/e2e/clean-reporter.js

Lines changed: 18 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
function getTimestamp() {
2+
return (process.env.CI || process.env.VERBOSE)
3+
? new Date().toLocaleTimeString('en-US', { hour12: false })
4+
: '';
5+
}
6+
7+
function logStamp(...args) {
8+
console.log(getTimestamp(), ...args);
9+
}
10+
111
class CleanReporter {
212
onBegin(config, suite) {
313
console.log(`\n🎭 Running ${suite.allTests().length} tests...\n`);
@@ -15,9 +25,9 @@ class CleanReporter {
1525
.trim();
1626

1727
if (result.status === "passed") {
18-
console.log(`✅ ${cleanSuite}: ${testName}`);
28+
logStamp(`✅ ${cleanSuite}: ${testName}`);
1929
} else if (result.status === "failed") {
20-
console.log(`❌ ${cleanSuite}: ${testName}`);
30+
logStamp(`❌ ${cleanSuite}: ${testName}`);
2131

2232
// Extract the most relevant error info
2333
const error = result.error || result.errors?.[0];
@@ -60,15 +70,17 @@ class CleanReporter {
6070

6171
onEnd(result) {
6272
console.log("\n" + "=".repeat(60));
63-
console.log(`📊 TEST SUMMARY`);
73+
logStamp(`📊 TEST SUMMARY`);
6474
console.log("=".repeat(60));
6575

6676
console.log(`\n🔍 FAILURE ANALYSIS:`);
6777
console.log(`• Most failures appear to be AI service related`);
6878
console.log(`• Check API keys and service availability`);
69-
console.log(
70-
`• Run 'pnpm exec playwright show-report' for detailed HTML report`
71-
);
79+
if (!process.env.CI) {
80+
console.log(
81+
`• Run 'pnpm exec playwright show-report' for detailed HTML report`
82+
);
83+
}
7284

7385
console.log("=".repeat(60) + "\n");
7486
}

typescript-sdk/apps/dojo/e2e/playwright.config.ts

Lines changed: 11 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ function getReporters(): ReporterDescription[] {
1111
uploadVideos: true,
1212
},
1313
];
14-
1514
const s3Reporter: ReporterDescription = [
1615
"./node_modules/playwright-slack-report/dist/src/SlackReporter.js",
1716
{
@@ -21,34 +20,19 @@ function getReporters(): ReporterDescription[] {
2120
layout: generateSimpleLayout, // Use our simple layout
2221
},
2322
];
24-
25-
let reporters: ReporterDescription[] = [];
23+
const githubReporter: ReporterDescription = ["github"];
24+
const htmlReporter: ReporterDescription = ["html", { open: "never" }];
25+
const cleanReporter: ReporterDescription = ["./clean-reporter.js"];
2626

2727
const addVideoAndSlack = process.env.SLACK_WEBHOOK_URL && process.env.AWS_S3_BUCKET_NAME;
28-
if (process.env.CI) {
29-
reporters = [
30-
["github"],
31-
["html", { open: "never" }],
32-
];
33-
if (addVideoAndSlack) {
34-
reporters.push(videoReporter, s3Reporter);
35-
}
36-
37-
return reporters;
38-
}
39-
40-
if (addVideoAndSlack) {
41-
return [
42-
videoReporter,
43-
s3Reporter,
44-
["html", { open: "never" }]
45-
];
46-
}
4728

4829
return [
49-
["./clean-reporter.js"],
50-
["html", { open: "never" }],
51-
];
30+
process.env.CI ? githubReporter : undefined,
31+
addVideoAndSlack ? videoReporter : undefined,
32+
addVideoAndSlack ? s3Reporter : undefined,
33+
htmlReporter,
34+
cleanReporter,
35+
].filter(Boolean) as ReporterDescription[];
5236
}
5337

5438
function getBaseUrl(): string {
@@ -61,10 +45,9 @@ function getBaseUrl(): string {
6145

6246
export default defineConfig({
6347
timeout: process.env.CI ? 300_000 : 120_000, // 5min in CI, 2min locally for AI tests
64-
workers: 1, // Serial execution to avoid race conditions and AI service conflicts
6548
testDir: "./tests",
66-
retries: process.env.CI ? 3 : 0, // More retries for flaky AI tests in CI, 0 for local
67-
fullyParallel: false, // Serial execution for deterministic AI test results
49+
retries: process.env.CI ? 1 : 0, // More retries for flaky AI tests in CI, 0 for local
50+
fullyParallel: true,
6851
use: {
6952
headless: true,
7053
viewport: { width: 1280, height: 720 },

0 commit comments

Comments
 (0)