From 85a39e026750cfcdf9898a7f5f4df2b2cf75bb0b Mon Sep 17 00:00:00 2001
From: Yury Semikhatsky <yurys@chromium.org>
Date: Thu, 2 Jan 2025 14:28:19 -0800
Subject: [PATCH] chore(bidi): print all tests in csv report

---
 tests/bidi/csvReporter.ts | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/tests/bidi/csvReporter.ts b/tests/bidi/csvReporter.ts
index f227550ef3..2b83b55694 100644
--- a/tests/bidi/csvReporter.ts
+++ b/tests/bidi/csvReporter.ts
@@ -47,19 +47,18 @@ class CsvReporter implements Reporter {
         for (const test of file.allTests()) {
           // Report fixme tests as failing.
           const fixme = test.annotations.find(a => a.type === 'fixme');
-          if (test.ok() && !fixme)
-            continue;
           const row = [];
           row.push(csvEscape(`${file.title} :: ${test.title}`));
           row.push(test.expectedStatus);
           row.push(test.outcome());
+          let errorMessage = '';
           if (fixme) {
-            row.push('fixme' + (fixme.description ? `: ${fixme.description}` : ''));
+            errorMessage = 'fixme' + (fixme.description ? `: ${fixme.description}` : '');
           } else {
             const result = test.results.find(r => r.error);
-            const errorMessage = stripAnsi(result?.error?.message.replace(/\s+/g, ' ').trim().substring(0, 1024));
-            row.push(csvEscape(errorMessage ?? ''));
+            errorMessage = result?.error?.message.replace(/\s+/g, ' ').trim().substring(0, 1024) ?? '';
           }
+          row.push(csvEscape(stripAnsi(errorMessage)));
           rows.push(row);
         }
       }