FIX issues#49. Find and FindOne returns nil objects

Author: dpordomingo

tests/common_test.go

@@ -95,10 +95,7 @@ func (s *BaseTestSuite) resultOrError(res interface{}, err error) bool {
 	}
 
 	if err == nil && res == nil {
-		s.Fail(
-			`FindOne should return an error or a document, but nothing was returned
-			TODO: https://github.com/src-d/go-kallax/issues/49`,
-		)
+		s.Fail("FindOne should return an error or a document, but nothing was returned")
 		return false
 	}
 

tests/query_test.go

@@ -29,9 +29,10 @@ func (s *QuerySuite) TestQuery() {
 
 	query := NewQueryFixtureQuery()
 	query.Where(kallax.Eq(Schema.QueryFixture.ID, doc.ID))
+
 	s.NotPanics(func() {
 		s.Equal("bar", store.MustFindOne(query).Foo)
-	}, "TODO: https://github.com/src-d/go-kallax/issues/49")
+	})
 
 	notID := kallax.NewID()
 	queryErr := NewQueryFixtureQuery()

tests/resultset_test.go

@@ -136,7 +136,6 @@ func (s *ResulsetSuite) TestResultSetForEachError() {
 	})
 }
 
-// TODO: https://github.com/src-d/go-kallax/issues/49
 func (s *ResulsetSuite) TestForEachAndCount() {
 	store := NewResultSetFixtureStore(s.db)
 
@@ -151,7 +150,7 @@ func (s *ResulsetSuite) TestForEachAndCount() {
 	manualCount := 0
 	rs.ForEach(func(doc *ResultSetFixture) error {
 		manualCount++
-		s.NotNil(doc, "TODO: https://github.com/src-d/go-kallax/issues/49")
+		s.NotNil(doc)
 		return nil
 	})
 	s.Equal(2, manualCount)

tests/store_test.go

@@ -86,15 +86,15 @@ func (s *StoreSuite) TestStoreInsertUpdateMustFind() {
 	s.Nil(err)
 	s.NotPanics(func() {
 		s.Equal("foo", store.MustFindOne(NewStoreWithConstructFixtureQuery()).Foo)
-	}, "TODO: https://github.com/src-d/go-kallax/issues/49")
+	})
 
 	doc.Foo = "bar"
 	updatedRows, err := store.Update(doc)
 	s.Nil(err)
 	s.True(updatedRows > 0)
 	s.NotPanics(func() {
 		s.Equal("bar", store.MustFindOne(NewStoreWithConstructFixtureQuery()).Foo)
-	}, "TODO: https://github.com/src-d/go-kallax/issues/49")
+	})
 }
 
 func (s *StoreSuite) TestStoreSave() {
@@ -107,15 +107,15 @@ func (s *StoreSuite) TestStoreSave() {
 	s.True(doc.IsPersisted())
 	s.NotPanics(func() {
 		s.Equal("foo", store.MustFindOne(NewStoreWithConstructFixtureQuery()).Foo)
-	}, "TODO: https://github.com/src-d/go-kallax/issues/49")
+	})
 
 	doc.Foo = "bar"
 	updated, err = store.Save(doc)
 	s.Nil(err)
 	s.True(updated)
 	s.NotPanics(func() {
 		s.Equal("bar", store.MustFindOne(NewStoreWithConstructFixtureQuery()).Foo)
-	}, "TODO: https://github.com/src-d/go-kallax/issues/49")
+	})
 }
 
 func (s *StoreSuite) TestMultiKeySort() {
@@ -169,16 +169,13 @@ func (s *StoreSuite) TestMultiKeySort() {
 	s.Len(documents, 4)
 	success := true
 	for _, doc := range documents {
-		if !s.NotNil(doc, "TODO: https://github.com/src-d/go-kallax/issues/49") {
+		if !s.NotNil(doc) {
 			success = false
 		}
 	}
 
 	if !success {
-		s.Fail(
-			`Testcase aborted. All retrieved Documents should be not-nil
-			TODO: https://github.com/src-d/go-kallax/issues/49`,
-		)
+		s.Fail("Testcase aborted. All retrieved Documents should be not-nil")
 		return
 	}
 
@@ -188,7 +185,6 @@ func (s *StoreSuite) TestMultiKeySort() {
 	s.Equal("2001-2012", documents[3].Name)
 }
 
-// TODO: https://github.com/src-d/go-kallax/issues/49
 func (s *StoreSuite) TestFindOne() {
 	store := NewStoreWithConstructFixtureStore(s.db)
 
@@ -198,7 +194,7 @@ func (s *StoreSuite) TestFindOne() {
 	query := NewStoreWithConstructFixtureQuery()
 	docFound, err := store.FindOne(query)
 	s.resultOrError(docFound, err)
-	if s.NotNil(docFound, "TODO: https://github.com/src-d/go-kallax/issues/49") {
+	if s.NotNil(docFound) {
 		s.Equal(docInserted.Foo, docFound.Foo)
 	}
 }