Merge pull request #103574 from liggitt/restore-long-printing

Restore ability to print long strings
This commit is contained in:
Kubernetes Prow Robot 2021-07-08 14:02:18 -07:00 committed by GitHub
commit 97d6e4a66a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 15 deletions

View File

@ -32,8 +32,6 @@ import (
"k8s.io/apimachinery/pkg/watch"
)
const maxStringLength = 100
var _ ResourcePrinter = &HumanReadablePrinter{}
type printHandler struct {
@ -213,21 +211,16 @@ func printTable(table *metav1.Table, output io.Writer, options PrintOptions) err
switch val := cell.(type) {
case string:
print := val
more := 0
// cut to maxStringLength
if len(val) > maxStringLength {
more = len(print) - maxStringLength
print = print[:maxStringLength]
}
// and also check for newlines
truncated := false
// truncate at newlines
newline := strings.Index(print, "\n")
if newline >= 0 {
more = more + len(print) - newline
truncated = true
print = print[:newline]
}
fmt.Fprint(output, print)
if more > 0 {
fmt.Fprintf(output, " + %d more...", more)
if truncated {
fmt.Fprint(output, "...")
}
default:
fmt.Fprint(output, val)

View File

@ -740,7 +740,7 @@ func TestStringPrinting(t *testing.T) {
{Cells: []interface{}{"test1", "20h", "This is first line\nThis is second line\nThis is third line\nand another one\n"}},
},
expected: `NAME AGE DESCRIPTION
test1 20h This is first line + 56 more...
test1 20h This is first line...
`,
},
// lengthy string
@ -754,7 +754,7 @@ test1 20h This is first line + 56 more...
{Cells: []interface{}{"test1", "20h", "This is first line which is long and goes for on and on and on an on and on and on and on and on and on and on and on and on and on and on"}},
},
expected: `NAME AGE DESCRIPTION
test1 20h This is first line which is long and goes for on and on and on an on and on and on and on and on and + 38 more...
test1 20h This is first line which is long and goes for on and on and on an on and on and on and on and on and on and on and on and on and on and on
`,
},
// lengthy string + newline
@ -768,7 +768,7 @@ test1 20h This is first line which is long and goes for on and on and on an
{Cells: []interface{}{"test1", "20h", "This is first\n line which is long and goes for on and on and on an on and on and on and on and on and on and on and on and on and on and on"}},
},
expected: `NAME AGE DESCRIPTION
test1 20h This is first + 126 more...
test1 20h This is first...
`,
},
}