impl/memory: Disallow empty namespace.

service/datastore/datastore_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"
         "fmt"
         "io/ioutil"
         "os"
         "path/filepath"
         "runtime"
         "testing"
 
         "github.com/luci/gae/service/info"
         "github.com/luci/luci-go/common/errors"
         . "github.com/luci/luci-go/common/testing/assertions"
         . "github.com/smartystreets/goconvey/convey"
         "golang.org/x/net/context"
 )
 
 func fakeDatastoreFactory(c context.Context, wantTxn bool) RawInterface {
         i := info.Get(c)
-        return &fakeDatastore{
+        fds := fakeDatastore{
                 aid: i.FullyQualifiedAppID(),
-                ns:  i.GetNamespace(),
         }
+        fds.ns, _ = i.GetNamespace()
+        return &fds
 }
 
 type fakeDatastore struct {
         RawInterface
         aid string
         ns  string
 }
 
 func (f *fakeDatastore) mkKey(elems ...interface{}) *Key {
         return MakeKey(f.aid, f.ns, elems...)
@@ skipping 1536 matching lines @@
                         if err != nil {
                                 panic(fmt.Errorf("failed to find absolute path for `%s`", sameLevelDir))
                         }
 
                         ids, err := FindAndParseIndexYAML(abs)
                         So(err, ShouldBeNil)
                         So(ids[1].Kind, ShouldEqual, "Test Foo")
                 })
         })
 }