From 517787cb34176fd7eda5466627d04dedbba694cc Mon Sep 17 00:00:00 2001
From: Tomasz Janiszewski <janiszt@gmail.com>
Date: Tue, 16 Aug 2022 11:50:56 +0200
Subject: [PATCH] Revert "Drop control characters from output"

This reverts commit bc70670a401d5123cafab4aa5fbc988e6bf742f6.
---
 parser/gotest/internal/collector/collector.go |  17 +++--------------
 testdata/035-report.xml                       |  12 ++----------
 testdata/035-whitespace.txt                   | Bin 2470 -> 2224 bytes
 testdata/src/whitespace/whitespace_test.go    |   6 ------
 4 files changed, 5 insertions(+), 30 deletions(-)

diff --git a/parser/gotest/internal/collector/collector.go b/parser/gotest/internal/collector/collector.go
index 1d856084..92d5e1ff 100644
--- a/parser/gotest/internal/collector/collector.go
+++ b/parser/gotest/internal/collector/collector.go
@@ -4,24 +4,13 @@ package collector
 
 import (
 	"sort"
-	"strings"
 	"time"
-	"unicode"
 )
 
 // line is a single line of output captured at some point in time.
 type line struct {
 	Timestamp time.Time
-	text      string
-}
-
-func (l line) SafeText() string {
-	return strings.Map(func(r rune) rune {
-		if unicode.IsControl(r) && !unicode.IsSpace(r) {
-			return -1
-		}
-		return r
-	}, l.text)
+	Text      string
 }
 
 // Output stores output lines grouped by id. Output can be retrieved for one or
@@ -65,7 +54,7 @@ func (o *Output) Contains(id int) bool {
 func (o *Output) Get(id int) []string {
 	var lines []string
 	for _, line := range o.m[id] {
-		lines = append(lines, line.SafeText())
+		lines = append(lines, line.Text)
 	}
 	return lines
 }
@@ -82,7 +71,7 @@ func (o *Output) GetAll(ids ...int) []string {
 	})
 	var lines []string
 	for _, line := range output {
-		lines = append(lines, line.SafeText())
+		lines = append(lines, line.Text)
 	}
 	return lines
 }
diff --git a/testdata/035-report.xml b/testdata/035-report.xml
index 70b972c5..db1a68cc 100644
--- a/testdata/035-report.xml
+++ b/testdata/035-report.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<testsuites tests="11">
-	<testsuite name="package/whitespace" tests="11" failures="0" errors="0" id="0" hostname="hostname" time="0.001" timestamp="2022-01-01T00:00:00Z">
+<testsuites tests="9">
+	<testsuite name="package/whitespace" tests="9" failures="0" errors="0" id="0" hostname="hostname" time="0.001" timestamp="2022-01-01T00:00:00Z">
 		<properties>
 			<property name="go.version" value="1.0"></property>
 		</properties>
@@ -45,10 +45,6 @@ one-newline
 two-newlines
 two-newlines
 two-newlines]]></system-out>
-		</testcase>
-		<testcase name="TestControl" classname="package/whitespace" time="0.000">
-			<system-out><![CDATA[    whitespace_test.go:49: log control
-printf control]]></system-out>
 		</testcase>
 		<testcase name="TestSubTests" classname="package/whitespace" time="0.000"></testcase>
 		<testcase name="TestSubTests/TestFlat" classname="package/whitespace" time="0.000">
@@ -93,9 +89,5 @@ two-newlines
 two-newlines
 two-newlines]]></system-out>
 		</testcase>
-		<testcase name="TestSubTests/TestControl" classname="package/whitespace" time="0.000">
-			<system-out><![CDATA[    whitespace_test.go:49: log control
-printf control]]></system-out>
-		</testcase>
 	</testsuite>
 </testsuites>
diff --git a/testdata/035-whitespace.txt b/testdata/035-whitespace.txt
index 544a4041dfaaca3c810925bfca270d0550046c7e..7d6590ca4d88e751e4be187624dae8c52f9378fe 100644
GIT binary patch
delta 28
kcmZ1`yg_ioU*^dTEF7B)S(Y$QKEcd4xsK!N<R(rP0HX&AwEzGB

delta 158
zcmdlWxJ-D%UuJga{JfH){G7=b*(7Z(trT+d(-{<!A!1wwMVWaeX$ZcquC79WV{ovQ
zLP%<H2~?+ohJl`eL9yoKdN%RRtSn2ICpWN3BO5DEz}U$v*q=;3!7dNi)6Ag@0FX2_
A$^ZZW

diff --git a/testdata/src/whitespace/whitespace_test.go b/testdata/src/whitespace/whitespace_test.go
index 269db29f..36639bcc 100644
--- a/testdata/src/whitespace/whitespace_test.go
+++ b/testdata/src/whitespace/whitespace_test.go
@@ -45,15 +45,9 @@ func TestWithNewlinesFlat(t *testing.T) {
 	fmt.Println("two-newlines\ntwo-newlines\ntwo-newlines")
 }
 
-func TestControl(t *testing.T) {
-	t.Logf("log\x00 control")
-	fmt.Printf("printf\x00 control\n")
-}
-
 func TestSubTests(t *testing.T) {
 	t.Run("TestFlat", TestFlat)
 	t.Run("TestWithSpace", TestWithSpace)
 	t.Run("TestWithTab", TestWithTab)
 	t.Run("TestWithNewlinesFlat", TestWithNewlinesFlat)
-	t.Run("TestControl", TestControl)
 }