Milo: Fix project importer so it doesn't error out on a bad config

milo/common/config_test.go

 // Copyright 2016 The LUCI Authors.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //      http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
 package common
 
 import (
         "errors"
-        "strings"
         "testing"
 
         "github.com/luci/gae/impl/memory"
         memcfg "github.com/luci/luci-go/common/config/impl/memory"
         "github.com/luci/luci-go/common/logging/gologger"
         "github.com/luci/luci-go/luci_config/server/cfgclient/backend/testconfig"
 
         "golang.org/x/net/context"
 
         . "github.com/smartystreets/goconvey/convey"
@@ skipping 41 matching lines @@
                                 So(p.ID, ShouldEqual, "foo")
                         })
 
                         Convey("Check Console config updated", func() {
                                 cs, err := GetConsole(c, "foo", "default")
                                 So(err, ShouldBeNil)
                                 So(cs.Name, ShouldEqual, "default")
                                 So(cs.RepoURL, ShouldEqual, "https://chromium.googlesource.com/foo/bar")
                         })
                 })
-
-                Convey("Reject duplicate configs.", func() {
-                        c = testconfig.WithCommonClient(c, memcfg.New(mockedConfigs))
-                        _, err := UpdateServiceConfig(c)
-                        So(err, ShouldBeNil)
-                        mockedConfigs["projects/bar.git"] = memcfg.ConfigSet{"luci-milo.cfg": barCfg}
-
-                        err = UpdateProjectConfigs(c)
-                        So(strings.HasPrefix(err.Error(), "Duplicate project ID"), ShouldEqual, true)
-                })
         })
 }
 
 var fooCfg = `
 ID: "foo"
 Consoles: {
         Name: "default"
         RepoURL: "https://chromium.googlesource.com/foo/bar"
         Branch: "master"
         Builders: {
                 Name: "buildbucket/luci.foo.something/bar"
                 Category: "main|something"
                 ShortName: "s"
         }
         Builders: {
                 Name: "buildbucket/luci.foo.other/baz"
                 Category: "main|other"
                 ShortName: "o"
         }
 }
 `
 
-var barCfg = `
-ID: "foo"
-`
-
 var settingsCfg = `
 buildbot: {
         internal_reader: "googlers"
 }
 `
 
 var mockedConfigs = map[string]memcfg.ConfigSet{
-        "projects/foo.git": {
+        "projects/foo": {
                 "luci-milo.cfg": fooCfg,
         },
 }