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

Unified Diff: service/datastore/datastore.go

Issue 1574353004: GitHub #8: Seed indexes from index.yml (Closed) Base URL: https://github.com/luci/gae@master
Patch Set: Fix missing parallel call in test Created 4 years, 11 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
« no previous file with comments | « no previous file | service/datastore/datastore_test.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: service/datastore/datastore.go
diff --git a/service/datastore/datastore.go b/service/datastore/datastore.go
index 68fae10a88300f0fe0773b21bfb45a09654af4a1..208547c6b730e00db226e9d5b47db2d2dd9e0f24 100644
--- a/service/datastore/datastore.go
+++ b/service/datastore/datastore.go
@@ -6,9 +6,17 @@ package datastore
import (
"fmt"
+ "io"
+ "io/ioutil"
+ "os"
+ "path/filepath"
"reflect"
+ "runtime"
+ "strings"
"github.com/luci/luci-go/common/errors"
+
+ "gopkg.in/yaml.v2"
)
type datastoreImpl struct {
@@ -348,3 +356,77 @@ func (d *datastoreImpl) DeleteMulti(keys []*Key) (err error) {
func (d *datastoreImpl) Raw() RawInterface {
return d.RawInterface
}
+
+// ParseIndexYAML parses the contents of a index YAML file into a list of
+// IndexDefinitions.
+func ParseIndexYAML(content io.Reader) ([]*IndexDefinition, error) {
+ serialized, err := ioutil.ReadAll(content)
+ if err != nil {
+ return nil, err
+ }
+
+ var m map[string][]*IndexDefinition
+ if err := yaml.Unmarshal(serialized, &m); err != nil {
+ return nil, err
+ }
+
+ if _, ok := m["indexes"]; !ok {
+ return nil, fmt.Errorf("datastore: missing key `indexes`: %v", m)
+ }
+ return m["indexes"], nil
+}
+
+// getCallingTestFilePath looks up the call stack and returns the path of the
+// first test file encountered. If no test file is found, getCallingTestFilePath
+// returns a non-nil error.
+func getCallingTestFilePath() (string, error) {
+ for skip := 1; ; skip += 1 {
+ _, path, _, ok := runtime.Caller(skip)
iannucci 2016/01/15 03:06:08 consider using https://golang.org/pkg/runtime/#Cal
+
+ if !ok {
+ return "", fmt.Errorf("datastore: failed to determine source file name")
+ }
+
+ if filename := filepath.Base(path); strings.HasSuffix(filename, "_test.go") {
+ return path, nil
+ }
+ }
+}
+
+// FindAndParseIndexYAML walks up from the directory of the calling test file
+// until it finds a `index.yaml` or `index.yml` file.
+// If an index YAML file is found, it opens and parses the file,
+// and returns all the indexes found.
+//
+// FindAndParseIndexYAML returns a non-nil error if the root of the drive is
+// reached without finding an index YAML file, or if there was
+// an error reading the found index YAML file.
+func FindAndParseIndexYAML() ([]*IndexDefinition, error) {
+ path, err := getCallingTestFilePath()
+ if err != nil {
+ return nil, err
+ }
+
+ isRoot := func(currentDir string) bool {
+ parentDir := filepath.Dir(currentDir)
+ return os.IsPathSeparator(currentDir[len(currentDir)-1]) && os.IsPathSeparator(parentDir[len(parentDir)-1])
+ }
+
+ currentDir := filepath.Dir(path)
+
+ for {
+ for _, filename := range []string{"index.yml", "index.yaml"} {
+ file, err := os.Open(filepath.Join(currentDir, filename))
+ if err == nil {
+ defer file.Close()
+ return ParseIndexYAML(file)
+ }
+ }
+
+ if isRoot(currentDir) {
+ return nil, fmt.Errorf("datastore: failed to find index YAML file")
+ }
+
+ currentDir = filepath.Dir(currentDir)
+ }
+}
« no previous file with comments | « no previous file | service/datastore/datastore_test.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698