Milo: Increase test coverage for appengine/buildbot

milo/appengine/buildbot/pubsub_test.go

 // Copyright 2016 The LUCI Authors. All rights reserved.
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
 package buildbot
 
 import (
         "bytes"
         "compress/zlib"
         "encoding/base64"
@@ skipping 79 matching lines @@
                 c = auth.WithState(c, &authtest.FakeState{
                         Identity:       identity.AnonymousIdentity,
                         IdentityGroups: []string{"all"},
                 })
                 // Update the service config so that the settings are loaded.
                 err := common.UpdateServiceConfig(c)
                 So(err, ShouldBeNil)
 
                 rand.Seed(5)
 
+                Convey("Remove source changes", func() {
+                        m := &buildbotMaster{
+                                Name: "fake",
+                                Builders: map[string]*buildbotBuilder{
+                                        "fake builder": {
+                                                PendingBuildStates: []*buildbotPending{
+                                                        {
+                                                                Source: buildbotSourceStamp{
+                                                                        Changes: []buildbotChange{{Comments: "foo"}},
+                                                                },
+                                                        },
+                                                },
+                                        },
+                                },
+                        }
+                        So(putDSMasterJSON(c, m, false), ShouldBeNil)
+                        lm, _, err := getMasterJSON(c, "fake")
+                        So(err, ShouldBeNil)
+                        So(lm.Builders["fake builder"].PendingBuildStates[0].Source.Changes[0].Comments, ShouldResemble, "")
+                })
+
                 Convey("Save build entry", func() {
                         build := &buildbotBuild{
                                 Master:      "Fake Master",
                                 Buildername: "Fake buildername",
                                 Number:      1234,
                                 Currentstep: "this is a string",
                                 Finished:    true,
                         }
                         err := ds.Put(c, build)
                         ds.GetTestable(c).CatchupIndexes()
@@ skipping 354 matching lines @@
                                 So(m.Project.Title, ShouldEqual, "some title")
                                 So(m.Slaves["testslave"].Name, ShouldEqual, "testslave")
                                 So(len(m.Slaves["testslave"].Runningbuilds), ShouldEqual, 0)
                                 So(len(m.Slaves["testslave"].RunningbuildsMap), ShouldEqual, 1)
                                 So(m.Slaves["testslave"].RunningbuildsMap["Fake buildername"][0],
                                         ShouldEqual, 2222)
                         })
                 })
         })
 }