Added check for multiple overrides in manifes files.

This commit is contained in:
Milutin Jovanović 2017-06-15 17:35:58 -04:00
Родитель 24c37f5f33
Коммит 319d04e4f3
3 изменённых файлов: 13 добавлений и 0 удалений

Просмотреть файл

@ -193,6 +193,9 @@ func fromRawManifest(raw rawManifest) (*Manifest, error) {
if err != nil {
return nil, err
}
if _, exists := m.Ovr[name]; exists {
return nil, errors.Errorf("multiple overrides specified for %s, can only specify one", name)
}
m.Ovr[name] = prj
}

Просмотреть файл

@ -107,6 +107,7 @@ func TestReadManifestErrors(t *testing.T) {
}{
{"multiple constraints", "manifest/error1.toml"},
{"multiple dependencies", "manifest/error2.toml"},
{"multiple overrides", "manifest/error3.toml"},
}
for _, tst := range tests {

9
testdata/manifest/error3.toml поставляемый Normal file
Просмотреть файл

@ -0,0 +1,9 @@
ignored = ["github.com/foo/bar"]
[[override]]
name = "github.com/golang/dep"
branch = "master"
[[override]]
name = "github.com/golang/dep"
branch = "master"