config: url, breaking change to support multiple insteadOf keys in co… …nfiguration

config/config_test.go

···
 	}
 
 	cfg.URLs["ssh://[e-mail]/"] = &URL{
-		Name:      "ssh://[e-mail]/",
-		InsteadOf: "https://github.com/",
+		Name:       "ssh://[e-mail]/",
+		InsteadOfs: []string{"https://github.com/"},
 	}
 
 	b, err := cfg.Marshal()

config/url.go

···
 	Name string
 	// Any URL that starts with this value will be rewritten to start, instead, with <base>.
 	// When more than one insteadOf strings match a given URL, the longest match is used.
-	InsteadOf string
+	InsteadOfs []string
 
 	// raw representation of the subsection, filled by marshal or unmarshal are
 	// called.
···
 
 // Validate validates fields of branch
 func (b *URL) Validate() error {
-	if b.InsteadOf == "" {
+	if len(b.InsteadOfs) == 0 {
 		return errURLEmptyInsteadOf
 	}
 
···
 	u.raw = s
 
 	u.Name = s.Name
-	u.InsteadOf = u.raw.Option(insteadOfKey)
+	u.InsteadOfs = u.raw.OptionAll(insteadOfKey)
 	return nil
 }
 
···
 	}
 
 	u.raw.Name = u.Name
-	u.raw.SetOption(insteadOfKey, u.InsteadOf)
+	u.raw.SetOption(insteadOfKey, u.InsteadOfs...)
 
 	return u.raw
 }
 
 func findLongestInsteadOfMatch(remoteURL string, urls map[string]*URL) *URL {
 	var longestMatch *URL
+	var longestMatchLength int
+
 	for _, u := range urls {
-		if !strings.HasPrefix(remoteURL, u.InsteadOf) {
-			continue
-		}
+		for _, currentInsteadOf := range u.InsteadOfs {
+			if !strings.HasPrefix(remoteURL, currentInsteadOf) {
+				continue
+			}
+
+			lengthCurrentInsteadOf := len(currentInsteadOf)
 
-		// according to spec if there is more than one match, take the logest
-		if longestMatch == nil || len(longestMatch.InsteadOf) < len(u.InsteadOf) {
-			longestMatch = u
+			// according to spec if there is more than one match, take the longest
+			if longestMatch == nil || longestMatchLength < lengthCurrentInsteadOf {
+				longestMatch = u
+				longestMatchLength = lengthCurrentInsteadOf
+			}
 		}
 	}
 
···
 }
 
 func (u *URL) ApplyInsteadOf(url string) string {
-	if !strings.HasPrefix(url, u.InsteadOf) {
-		return url
+	for _, j := range u.InsteadOfs {
+		if strings.HasPrefix(url, j) {
+			return u.Name + url[len(j):]
+		}
 	}
 
-	return u.Name + url[len(u.InsteadOf):]
+	return url
 }

config/url_test.go

···
 
 func (b *URLSuite) TestValidateInsteadOf() {
 	goodURL := URL{
-		Name:      "ssh://github.com",
-		InsteadOf: "http://github.com",
+		Name:       "ssh://github.com",
+		InsteadOfs: []string{"http://github.com"},
 	}
 	badURL := URL{}
 	b.Nil(goodURL.Validate())
···
 
 	cfg := NewConfig()
 	cfg.URLs["ssh://[e-mail]/"] = &URL{
-		Name:      "ssh://[e-mail]/",
-		InsteadOf: "https://github.com/",
+		Name:       "ssh://[e-mail]/",
+		InsteadOfs: []string{"https://github.com/"},
+	}
+
+	actual, err := cfg.Marshal()
+	b.Nil(err)
+	b.Equal(string(expected), string(actual))
+}
+
+func (b *URLSuite) TestMarshalMultipleInsteadOf() {
+	expected := []byte(`[core]
+	bare = false
+[url "ssh://[e-mail]/"]
+	insteadOf = https://github.com/
+	insteadOf = https://google.com/
+`)
+
+	cfg := NewConfig()
+	cfg.URLs["ssh://[e-mail]/"] = &URL{
+		Name:       "ssh://[e-mail]/",
+		InsteadOfs: []string{"https://github.com/", "https://google.com/"},
 	}
 
 	actual, err := cfg.Marshal()
···
 	b.NoError(err)
 	url := cfg.URLs["ssh://[e-mail]/"]
 	b.Equal("ssh://[e-mail]/", url.Name)
-	b.Equal("https://github.com/", url.InsteadOf)
+	b.Equal("https://github.com/", url.InsteadOfs[0])
+}
+
+func (b *URLSuite) TestUnmarshalMultipleInsteadOf() {
+	input := []byte(`[core]
+	bare = false
+[url "ssh://[e-mail]/"]
+	insteadOf = https://github.com/
+	insteadOf = https://google.com/
+`)
+
+	cfg := NewConfig()
+	err := cfg.Unmarshal(input)
+	b.Nil(err)
+	url := cfg.URLs["ssh://[e-mail]/"]
+	b.Equal("ssh://[e-mail]/", url.Name)
+
+	b.Equal("ssh://[e-mail]/foobar", url.ApplyInsteadOf("https://github.com/foobar"))
+	b.Equal("ssh://[e-mail]/foobar", url.ApplyInsteadOf("https://google.com/foobar"))
+}
+
+func (b *URLSuite) TestUnmarshalDuplicateUrls() {
+	input := []byte(`[core]
+	bare = false
+[url "ssh://[e-mail]/"]
+	insteadOf = https://github.com/
+[url "ssh://[e-mail]/"]
+	insteadOf = https://google.com/
+`)
+
+	cfg := NewConfig()
+	err := cfg.Unmarshal(input)
+	b.Nil(err)
+	url := cfg.URLs["ssh://[e-mail]/"]
+	b.Equal("ssh://[e-mail]/", url.Name)
+
+	b.Equal("ssh://[e-mail]/foobar", url.ApplyInsteadOf("https://github.com/foobar"))
+	b.Equal("ssh://[e-mail]/foobar", url.ApplyInsteadOf("https://google.com/foobar"))
 }
 
 func (b *URLSuite) TestApplyInsteadOf() {
 	urlRule := URL{
-		Name:      "ssh://github.com",
-		InsteadOf: "http://github.com",
+		Name:       "ssh://github.com",
+		InsteadOfs: []string{"http://github.com"},
 	}
 
 	b.Equal("http://google.com", urlRule.ApplyInsteadOf("http://google.com"))
 	b.Equal("ssh://github.com/myrepo", urlRule.ApplyInsteadOf("http://github.com/myrepo"))
 }
+
+func (b *URLSuite) TestFindLongestInsteadOfMatch() {
+	urlRules := map[string]*URL{
+		"ssh://github.com": &URL{
+			Name:       "ssh://github.com",
+			InsteadOfs: []string{"http://github.com"},
+		},
+		"ssh://somethingelse.com": &URL{
+			Name:       "ssh://somethingelse.com",
+			InsteadOfs: []string{"http://github.com/foobar"},
+		},
+	}
+
+	longestUrl := findLongestInsteadOfMatch("http://github.com/foobar/bingbash.git", urlRules)
+
+	b.Equal("ssh://somethingelse.com", longestUrl.Name)
+}