Scrub ansi from test output and stderr (#19)

* scrub ansi from test output and stderr

* fix test case

Author: robherley

__tests__/renderer.test.ts

@@ -271,4 +271,36 @@ describe('renderer', () => {
 
     expect($('summary:contains(🖨️ Output)')).toHaveLength(0)
   })
+
+  it('scrubs ansi from stderr', async () => {
+    const renderer = await getRenderer()
+    const placeholder = 'no-ansi-please'
+    renderer.stderr = `\u001b[31m${placeholder}\u001b[0m`
+    await renderer.writeSummary()
+    const $ = await loadSummaryHTML()
+
+    expect($(`details:contains(${placeholder})`)).toHaveLength(1)
+    $(`details:contains(${placeholder})`).each((_, el) => {
+      const text = $(el).text()
+      if (text.includes(placeholder)) {
+        expect(text).not.toContain('\u001b')
+      }
+    })
+  })
+
+  it('scrubs ansi from test output', async () => {
+    const renderer = await getRenderer()
+    const placeholder = 'no-ansi-please'
+    renderer.packageResults[1].events[0].output = `\u001b[31m${placeholder}\u001b[0m`
+    await renderer.writeSummary()
+    const $ = await loadSummaryHTML()
+
+    expect($(`details:contains(${placeholder})`)).toHaveLength(1)
+    $(`details:contains(${placeholder})`).each((_, el) => {
+      const text = $(el).text()
+      if (text.includes(placeholder)) {
+        expect(text).not.toContain('\u001b')
+      }
+    })
+  })
 })

dist/index.js

@@ -1,6 +1,6 @@
 {
   "name": "go-test-action",
-  "version": "0.4.1",
+  "version": "0.5.0",
   "description": "GitHub Action to run go tests with rich summary output and annotations.",
   "main": "dist/index.js",
   "scripts": {

package.json

@@ -191,7 +191,7 @@ class Renderer {
 
     if (!this.omit.has(OmitOption.PackageOutput)) {
       details += `<details><summary>🖨️ Output</summary><pre><code>${
-        packageResult.output() || '(none)'
+        this.scrubAnsi(packageResult.output()) || '(none)'
       }</code></pre></details>`
     }
 
@@ -275,11 +275,15 @@ ${pieData}
 <summary>🚨 Standard Error Output</summary>
 
 \`\`\`
-${this.stderr}
+${this.scrubAnsi(this.stderr)}
 \`\`\`
 
 </details>`
   }
+
+  private scrubAnsi(input: string): string {
+    return input.replace(/\x1b\[[0-9;]*[a-zA-Z]/g, '')
+  }
 }
 
 export default Renderer