impl/memory: Disallow empty namespace.

impl/memory/datastore_query_execution_test.go

Index: impl/memory/datastore_query_execution_test.go
diff --git a/impl/memory/datastore_query_execution_test.go b/impl/memory/datastore_query_execution_test.go
index 41f4a898b2356b8466cb60cbffd3699ff14d83cc..3a57b07c4bcb987913287609b9828cbe59d45265 100644
--- a/impl/memory/datastore_query_execution_test.go
+++ b/impl/memory/datastore_query_execution_test.go
@@ -85,7 +85,7 @@ var stage2Data = []ds.PropertyMap{
 		"Val", 3, 4, 2, 1, Next,
 		"Extra", "nuts",
 	),
-	pmap("$key", ds.MakeKey("dev~app", "", "Kind", "id")),
+	pmap("$key", ds.MakeKey("dev~app", "jim", "Kind", "id")),

[iannucci, 2016/04/22 19:13:49]

s/jim/kat

 	pmap("$key", ds.MakeKey("dev~app", "bob", "Kind", "id")),
 }
 
@@ -351,16 +351,16 @@ var queryExecutionTests = []qExTest{
 					stage1Data[2],
 				}},
 				{q: nq("__namespace__"), get: []ds.PropertyMap{
-					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", 1)),
 					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "bob")),

[iannucci, 2016/04/22 19:13:49]

lost test for default namespace :/. Could we add a quick ad-hoc one?

+					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "jim")),
 					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "ns")),
 				}},
 				{q: nq("__namespace__").Offset(1), get: []ds.PropertyMap{
-					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "bob")),
+					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "jim")),
 					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "ns")),
 				}},
 				{q: nq("__namespace__").Offset(1).Limit(1), get: []ds.PropertyMap{
-					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "bob")),
+					pmap("$key", ds.MakeKey("dev~app", "", "__namespace__", "jim")),
 				}},
 			},