Add new tests to presubmit, fix errors

monitoring/go/buildbot_ingest/bycommit.go

 package buildbot_ingest
 
 /*
         Loads data from build masters and pushes it into InfluxDB.
 */
 import (
         "fmt"
         "path"
         "sync"
         "time"
@@ skipping 123 matching lines @@
         if len(results) == 0 {
                 return -1, nil
         }
 
         // Error checking.
         if len(results) != 1 {
                 return -1, fmt.Errorf("Query returned incorrect number of series: %q", q)
         }
         series := results[0]
         if series.Name != SERIES_BUILDBOT_BYCOMMIT {
-                return -1, fmt.Errorf("Query returned the wrong series: %q; expected: %s got: %S", q, SERIES_BUILDBOT_BYCOMMIT, series.Name)
+                return -1, fmt.Errorf("Query returned the wrong series: %q; expected: %s got: %s", q, SERIES_BUILDBOT_BYCOMMIT, series.Name)
         }
         if len(series.Columns) != 2 {
                 return -1, fmt.Errorf("Query returned incorrect number of columns: %q, %v", q, series.Columns)
         }
         if len(series.Points) != 1 {
                 return -1, fmt.Errorf("Query returned more than one point: %q", q)
         }
         p := series.Points[0]
         if len(p) != len(series.Columns) {
                 return -1, fmt.Errorf("Number of columns does not match number of fields in datapoint: %v, %v", p, series.Columns)
@@ skipping 98 matching lines @@
                                                 if err != nil {
                                                         glog.Error(err)
                                                 }
                                         }
                                 }(m, b, builds)
                         }
                 }
                 wg.Wait()
         }
 }