Skip to content

Commit

Permalink
fix tests
Browse files Browse the repository at this point in the history
  • Loading branch information
mh-park committed Jul 13, 2017
1 parent efdcfc3 commit aeb1b8f
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
6 changes: 5 additions & 1 deletion cmd/query/app/builder/elastic_search.go
Original file line number Diff line number Diff line change
Expand Up @@ -62,5 +62,9 @@ func (e *esBuilder) NewSpanReader() (spanstore.Reader, error) {
}

func (e *esBuilder) NewDependencyReader() (dependencystore.Reader, error) {
return esDependencyStore.NewDependencyStore(), nil
client, err := e.getClient()
if err != nil {
return nil, err
}
return esDependencyStore.NewDependencyStore(client, e.logger), nil
}
3 changes: 3 additions & 0 deletions cmd/query/app/builder/elastic_search_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ func TestESBuilderFailure(t *testing.T) {
spanReader, err := esBuilder.NewSpanReader()
assert.Error(t, err)
assert.Nil(t, spanReader)
dependencyReader, err := esBuilder.NewDependencyReader()
assert.Error(t, err)
assert.Nil(t, dependencyReader)
})
}

Expand Down

0 comments on commit aeb1b8f

Please sign in to comment.