mirror of
https://github.com/rancher/rke.git
synced 2025-06-23 22:17:11 +00:00
4613cce211
Previously, we declared the Env slice with the size of the `c.Services.KubeAPI.ExtraEnv` field, which leads to empty strings at the beginning of the Env slice because we use Golang's append function to add new elements, and Docker is not happy with that. We fix this issue by declaring the Env variable as an empty slice. We also enhance the `getUniqStringList` function to properly trim leading and trailing spaces in each element and to ignore empty strings. We add unit tests for the `getUniqStringList` function and update the integration tests.
53 lines
1.0 KiB
Go
53 lines
1.0 KiB
Go
package cluster
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func Test_getUniqStringList(t *testing.T) {
|
|
type args struct {
|
|
l []string
|
|
}
|
|
tests := []struct {
|
|
name string
|
|
args args
|
|
want []string
|
|
}{
|
|
{
|
|
"contain strings with only spaces",
|
|
args{
|
|
[]string{" ", "key1=value1", " ", "key2=value2"},
|
|
},
|
|
[]string{"key1=value1", "key2=value2"},
|
|
},
|
|
{
|
|
"contain strings with trailing or leading spaces",
|
|
args{
|
|
[]string{" key1=value1", "key1=value1 ", " key2=value2 "},
|
|
},
|
|
[]string{"key1=value1", "key2=value2"},
|
|
},
|
|
{
|
|
"contain duplicated strings",
|
|
args{
|
|
[]string{"", "key1=value1", "key1=value1", "key2=value2"},
|
|
},
|
|
[]string{"key1=value1", "key2=value2"},
|
|
},
|
|
{
|
|
"contain empty string",
|
|
args{
|
|
[]string{"", "key1=value1", "", "key2=value2"},
|
|
},
|
|
[]string{"key1=value1", "key2=value2"},
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
assert.Equalf(t, tt.want, getUniqStringList(tt.args.l), "getUniqStringList(%v)", tt.args.l)
|
|
})
|
|
}
|
|
}
|