Commit8e131eb

committed

fix test and case insensitivity

1 parent62ada21 commit8e131ebCopy full SHA for 8e131eb

File tree

-6

lines changed

-6

lines changed

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,8 @@ func (o *sshConfigOptions) addOption(option string) error {`
`75`	`75`	`iferr!=nil {`
`76`	`76`	`returnerr`
`77`	`77`	`}`
`78`		`-ifo.removedKeys!=nil&&o.removedKeys[key] {`
	`78`	`+lowerKey:=strings.ToLower(key)`
	`79`	`+ifo.removedKeys!=nil&&o.removedKeys[lowerKey] {`
`79`	`80`	`// Key marked as removed, skip.`
`80`	`81`	`returnnil`
`81`	`82`	`}`
`@@ -87,7 +88,7 @@ func (o *sshConfigOptions) addOption(option string) error {`
`87`	`88`	`ifo.removedKeys==nil {`
`88`	`89`	`o.removedKeys=make(map[string]bool)`
`89`	`90`	`}`
`90`		`-o.removedKeys[key]=true`
	`91`	`+o.removedKeys[lowerKey]=true`
`91`	`92`	`}`
`92`	`93`	`returnnil`
`93`	`94`	`}`

Lines changed: 4 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -272,32 +272,32 @@ func Test_sshConfigOptions_addOption(t *testing.T) {`
`272`	`272`	`},`
`273`	`273`	`},`
`274`	`274`	`{`
`275`		`-Name:"Replace",`
	`275`	`+Name:"AddTwo",`
`276`	`276`	`Start: []string{`
`277`	`277`	`"foo bar",`
`278`	`278`	`},`
`279`	`279`	`Add: []string{"Foo baz"},`
`280`	`280`	`Expect: []string{`
	`281`	`+"foo bar",`
`281`	`282`	`"Foo baz",`
`282`	`283`	`},`
`283`	`284`	`},`
`284`	`285`	`{`
`285`		`-Name:"AddAndReplace",`
	`286`	`+Name:"AddAndRemove",`
`286`	`287`	`Start: []string{`
`287`		`-"a b",`
`288`	`288`	`"foo bar",`
`289`	`289`	`"buzz bazz",`
`290`	`290`	`},`
`291`	`291`	`Add: []string{`
`292`	`292`	`"b c",`
	`293`	`+"a=",// Remove all entries that start with "a", i.e. next line.`
`293`	`294`	`"A hello",`
`294`	`295`	`"hello world",`
`295`	`296`	`},`
`296`	`297`	`Expect: []string{`
`297`	`298`	`"foo bar",`
`298`	`299`	`"buzz bazz",`
`299`	`300`	`"b c",`
`300`		`-"A hello",`
`301`	`301`	`"hello world",`
`302`	`302`	`},`
`303`	`303`	`},`

Comments

(0)