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

Unified Diff: impl/memory/datastore_query_test.go

Issue 1369353003: Make impl/memory have an AppID. (Closed) Base URL: https://github.com/luci/gae.git@fix_consistent
Patch Set: Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: impl/memory/datastore_query_test.go
diff --git a/impl/memory/datastore_query_test.go b/impl/memory/datastore_query_test.go
index 7c268e1a8767150fddc785cef0df3ce028f1e57e..ad517716b1565f3f458e3a2bb78a2c7f66b66a0f 100644
--- a/impl/memory/datastore_query_test.go
+++ b/impl/memory/datastore_query_test.go
@@ -116,7 +116,7 @@ var queryTests = []queryTest{
End(curs("Foo", 20, "__key__", key("Something", 20)))),
nil,
&reducedQuery{
- "ns", "Foo", map[string]stringset.Set{}, []dstore.IndexColumn{
+ "dev~app", "ns", "Foo", map[string]stringset.Set{}, []dstore.IndexColumn{
{Property: "Foo"},
{Property: "__key__"},
},
@@ -140,7 +140,7 @@ func TestQueries(t *testing.T) {
Convey("non-ancestor queries in a transaction", func() {
fq, err := nq().Finalize()
So(err, ShouldErrLike, nil)
- _, err = reduce(fq, "ns", true)
+ _, err = reduce(fq, "dev~app", "ns", true)
So(err, ShouldErrLike, "must include an Ancestor")
})
@@ -151,7 +151,7 @@ func TestQueries(t *testing.T) {
}
fq, err := q.Finalize()
So(err, ShouldErrLike, nil)
- _, err = reduce(fq, "ns", false)
+ _, err = reduce(fq, "dev~app", "ns", false)
So(err, ShouldErrLike, "query is too large")
})
@@ -163,7 +163,7 @@ func TestQueries(t *testing.T) {
if err == nil {
err = fq.Valid("s~aid", "ns")
if err == nil {
- rq, err = reduce(fq, "ns", false)
+ rq, err = reduce(fq, "dev~app", "ns", false)
}
}
So(err, ShouldErrLike, tc.err)

Powered by Google App Engine
This is Rietveld 408576698