Fix miscellaneous prod bugs.

service/datastore/pls_test.go

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // adapted from github.com/golang/appengine/datastore
 
 package datastore
 
 import (
         "bytes"
@@ skipping 1696 matching lines @@
                         type OKDefaults struct {
                                 When   string `gae:"$when,tomorrow"`
                                 Amount int64  `gae:"$amt,100"`
                         }
                         pls := GetPLS(&OKDefaults{})
                         pm, err := pls.Save(true)
                         So(err, ShouldBeNil)
                         So(pm, ShouldResemble, PropertyMap{
                                 "$when": {mpNI("tomorrow")},
                                 "$amt":  {mpNI(100)},
+                                "$kind": {mp("OKDefaults")},

[iannucci, 2015/08/15 21:50:54]

technically when you SetMeta on a PropertyMap, it sets them to 'noindex'. Not
that it matters, because they'll never be serialized. But that's why the other
lines are mpNI

[dnj (Google), 2015/08/16 06:11:06]

Switched to NI.

                         })
 
                         v, err := pm.GetMeta("when")
                         So(err, ShouldBeNil)
                         So(v, ShouldEqual, "tomorrow")
 
                         v, err = pm.GetMeta("amt")
                         So(err, ShouldBeNil)
                         So(v, ShouldEqual, int64(100))
                 })
@@ skipping 29 matching lines @@
 
                 Convey("Bad default meta type", func() {
                         type BadDefault struct {
                                 Val time.Time `gae:"$meta,tomorrow"`
                         }
                         pls := GetPLS(&BadDefault{})
                         So(pls.Problem().Error(), ShouldContainSubstring, "bad type")
                 })
         })
 }