Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1395)

Side by Side Diff: service/datastore/properties_test.go

Issue 1269113005: A transparent cache for datastore, backed by memcache. (Closed) Base URL: https://github.com/luci/gae.git@add_meta
Patch Set: fix comments Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « service/datastore/properties.go ('k') | service/datastore/serialize.go » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package datastore 5 package datastore
6 6
7 import ( 7 import (
8 "math" 8 "math"
9 "testing" 9 "testing"
10 "time" 10 "time"
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 So(err, ShouldBeNil) 153 So(err, ShouldBeNil)
154 154
155 v, err := pm.GetMeta("foo") 155 v, err := pm.GetMeta("foo")
156 So(err, ShouldBeNil) 156 So(err, ShouldBeNil)
157 So(v, ShouldEqual, 100) 157 So(v, ShouldEqual, 100)
158 So(pm.GetMetaDefault("foo", 100), ShouldEqual, 1 00) 158 So(pm.GetMetaDefault("foo", 100), ShouldEqual, 1 00)
159 So(pm.GetMetaDefault("bar", 100), ShouldEqual, 1 00) 159 So(pm.GetMetaDefault("bar", 100), ShouldEqual, 1 00)
160 160
161 npm, err := pm.Save(false) 161 npm, err := pm.Save(false)
162 So(err, ShouldBeNil) 162 So(err, ShouldBeNil)
163 » » » » So(len(npm), ShouldEqual, 1) 163 » » » » So(len(npm), ShouldEqual, 0)
164 }) 164 })
165 165
166 Convey("errors", func() { 166 Convey("errors", func() {
167 Convey("too many values", func() { 167 Convey("too many values", func() {
168 pm := PropertyMap{ 168 pm := PropertyMap{
169 "$bad": {MkProperty(100), MkProp erty(200)}, 169 "$bad": {MkProperty(100), MkProp erty(200)},
170 } 170 }
171 _, err := pm.GetMeta("bad") 171 _, err := pm.GetMeta("bad")
172 So(err.Error(), ShouldContainSubstring, "too many values") 172 So(err.Error(), ShouldContainSubstring, "too many values")
173 }) 173 })
174 174
175 Convey("weird value", func() { 175 Convey("weird value", func() {
176 pm := PropertyMap{} 176 pm := PropertyMap{}
177 err := pm.SetMeta("sup", complex(100, 20 )) 177 err := pm.SetMeta("sup", complex(100, 20 ))
178 So(err.Error(), ShouldContainSubstring, "bad type") 178 So(err.Error(), ShouldContainSubstring, "bad type")
179 }) 179 })
180 }) 180 })
181 }) 181 })
182 }) 182 })
183 } 183 }
OLDNEW
« no previous file with comments | « service/datastore/properties.go ('k') | service/datastore/serialize.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698