Verbosity is fine, try the clean reporter in CI

Author: maxkorp

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

@@ -11,7 +11,6 @@ function getReporters(): ReporterDescription[] {
       uploadVideos: true,
     },
   ];
-
   const s3Reporter: ReporterDescription = [
       "./node_modules/playwright-slack-report/dist/src/SlackReporter.js",
       {
@@ -21,34 +20,19 @@ function getReporters(): ReporterDescription[] {
         layout: generateSimpleLayout, // Use our simple layout
       },
     ];
-
-  let reporters: ReporterDescription[] = [];
+  const githubReporter: ReporterDescription = ["github"];
+  const htmlReporter: ReporterDescription = ["html", { open: "never" }];
+  const cleanReporter: ReporterDescription = ["./clean-reporter.js"];
 
   const addVideoAndSlack = process.env.SLACK_WEBHOOK_URL && process.env.AWS_S3_BUCKET_NAME;
-  if (process.env.CI) {
-    reporters = [
-      ["github"],
-      ["html", { open: "never" }],
-    ];
-    if (addVideoAndSlack) {
-      reporters.push(videoReporter, s3Reporter);
-    }
-
-    return reporters;
-  }
-
-  if (addVideoAndSlack) {
-    return [
-      videoReporter,
-      s3Reporter,
-      ["html", { open: "never" }]
-    ];
-  }
 
   return [
-    ["./clean-reporter.js"],
-    ["html", { open: "never" }],
-  ];
+    process.env.CI ? githubReporter : undefined,
+    addVideoAndSlack ? videoReporter : undefined,
+    addVideoAndSlack ? s3Reporter : undefined,
+    htmlReporter,
+    cleanReporter,
+  ].filter(Boolean) as ReporterDescription[];
 }
 
 function getBaseUrl(): string {