impl/memory: Disallow empty namespace.

impl/memory/datastore_query_execution_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.
 
 package memory
 
 import (
         "fmt"
         "strings"
         "testing"
@@ skipping 476 matching lines @@
 func TestQueryExecution(t *testing.T) {
         t.Parallel()
 
         Convey("Test query execution", t, func() {
                 c, err := info.Get(Use(context.Background())).Namespace("ns")
                 if err != nil {
                         panic(err)
                 }
 
                 So(info.Get(c).FullyQualifiedAppID(), ShouldEqual, "dev~app")
-                So(info.Get(c).GetNamespace(), ShouldEqual, "ns")
+                ns, has := info.Get(c).GetNamespace()
+                So(ns, ShouldEqual, "ns")
+                So(has, ShouldBeTrue)
 
                 data := ds.Get(c)
                 testing := data.Testable()
 
                 for _, tc := range queryExecutionTests {
                         Convey(tc.name, func() {
                                 for i, stage := range tc.test {
                                         // outside of Convey, since these must always happen
                                         testing.CatchupIndexes()
 
@@ skipping 129 matching lines @@
 
         v, ok := actual.(error)
         if !ok {
                 return fmt.Sprintf("type of 'actual' must be error, not %T", actual)
         }
         if v == nil || v == ds.Stop {
                 return ""
         }
         return fmt.Sprintf("expected success value, not %v", v)
 }