github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-30 15:05:27 +00:00
Code Issues Projects Releases Wiki Activity

fix golint errors in pkg/util/strings

Browse Source 
Signed-off-by: Christian Schlotter <christi.schlotter@gmail.com>
This commit is contained in:
Christian Schlotter 2018-10-06 13:35:19 +02:00
parent 664d797245
commit 398039b36e
3 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
hack/.golint_failures
View File

@ -372,7 +372,6 @@ pkg/util/removeall
pkg/util/resourcecontainer
pkg/util/rlimit
pkg/util/selinux
pkg/util/strings
pkg/util/sysctl
pkg/util/sysctl/testing
pkg/util/system

9
pkg/util/strings/strings.go
View File

@ -22,7 +22,7 @@ import (
"unicode"
)
// Splits a fully qualified name and returns its namespace and name.
// SplitQualifiedName Splits a fully qualified name and returns its namespace and name.
// Assumes that the input 'str' has been validated.
func SplitQualifiedName(str string) (string, string) {
parts := strings.Split(str, "/")
@ -32,19 +32,18 @@ func SplitQualifiedName(str string) (string, string) {
return parts[0], parts[1]
}
// Joins 'namespace' and 'name' and returns a fully qualified name
// JoinQualifiedName joins 'namespace' and 'name' and returns a fully qualified name
// Assumes that the input is valid.
func JoinQualifiedName(namespace, name string) string {
return path.Join(namespace, name)
}
// Returns the first N slice of a string.
// ShortenString returns the first N slice of a string.
func ShortenString(str string, n int) string {
if len(str) <= n {
return str
} else {
return str[:n]
}
return str[:n]
}
// isVowel returns true if the rune is a vowel (case insensitive).

8
pkg/util/strings/strings_test.go
View File

@ -56,16 +56,16 @@ func TestJoinQualifiedName(t *testing.T) {
func TestShortenString(t *testing.T) {
testCases := []struct {
input string
out_len int
output string
input string
outLen int
output string
}{
{"kubernetes.io", 5, "kuber"},
{"blah", 34, "blah"},
{"kubernetes.io", 13, "kubernetes.io"},
}
for i, tc := range testCases {
res := ShortenString(tc.input, tc.out_len)
res := ShortenString(tc.input, tc.outLen)
if res != tc.output {
t.Errorf("case[%d]: expected %q, got %q", i, tc.output, res)
}