[cli/serve/integrationTests] log stdout from process when parsing fai…

…ls (#98699) (#98816) Co-authored-by: spalger <[email protected]> Co-authored-by: Spencer <[email protected]> Co-authored-by: spalger <[email protected]>
elastic · Apr 29, 2021 · 385cb54 · 385cb54
1 parent afea14d
commit 385cb54
Showing 1 changed file with 21 additions and 15 deletions.
diff --git a/src/cli/serve/integration_tests/invalid_config.test.ts b/src/cli/serve/integration_tests/invalid_config.test.ts
@@ -14,12 +14,11 @@ const INVALID_CONFIG_PATH = require.resolve('./__fixtures__/invalid_config.yml')
 
 interface LogEntry {
   message: string;
-  tags: string[];
+  tags?: string[];
   type: string;
 }
 
-// FLAKY: https://github.com/elastic/kibana/issues/32240
-describe.skip('cli invalid config support', function () {
+describe('cli invalid config support', function () {
   it(
     'exits with statusCode 64 and logs a single line when config is invalid',
     function () {
@@ -33,18 +32,25 @@ describe.skip('cli invalid config support', function () {
         }
       );
 
-      const [fatalLogLine] = stdout
-        .toString('utf8')
-        .split('\n')
-        .filter(Boolean)
-        .map((line) => JSON.parse(line) as LogEntry)
-        .filter((line) => line.tags.includes('fatal'))
-        .map((obj) => ({
-          ...obj,
-          pid: '## PID ##',
-          '@timestamp': '## @timestamp ##',
-          error: '## Error with stack trace ##',
-        }));
+      let fatalLogLine;
+      try {
+        [fatalLogLine] = stdout
+          .toString('utf8')
+          .split('\n')
+          .filter(Boolean)
+          .map((line) => JSON.parse(line) as LogEntry)
+          .filter((line) => line.tags?.includes('fatal'))
+          .map((obj) => ({
+            ...obj,
+            pid: '## PID ##',
+            '@timestamp': '## @timestamp ##',
+            error: '## Error with stack trace ##',
+          }));
+      } catch (e) {
+        throw new Error(
+          `error parsing log output:\n\n${e.stack}\n\nstdout: \n${stdout}\n\nstderr:\n${stderr}`
+        );
+      }
 
       expect(error).toBe(undefined);