Merge pull request #98 from mgatny/master

cbusbey · cbusbey · commit 827248e34996 · 2016-04-19T10:01:53.000-05:00
Ensure new sqlstore resets cache immediately
diff --git a/sqlstore.go b/sqlstore.go
@@ -49,6 +49,7 @@ func newSQLStore(sessionID SessionID, driver string, dataSourceName string) (sto
 		sqlDriver:         driver,
 		sqlDataSourceName: dataSourceName,
 	}
+	store.cache.Reset()
 
 	if store.db, err = sql.Open(store.sqlDriver, store.sqlDataSourceName); err != nil {
 		return nil, err
diff --git a/store_test.go b/store_test.go
@@ -164,6 +164,8 @@ func (suite *MessageStoreTestSuite) TestMessageStore_GetMessages_VariousRanges()
 }
 
 func (suite *MessageStoreTestSuite) TestMessageStore_CreationTime() {
+	assert.False(suite.T(), suite.msgStore.CreationTime().IsZero())
+
 	t0 := time.Now()
 	suite.msgStore.Reset()
 	t1 := time.Now()

Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,7 @@ func newSQLStore(sessionID SessionID, driver string, dataSourceName string) (sto`
`49`	`49`	`sqlDriver: driver,`
`50`	`50`	`sqlDataSourceName: dataSourceName,`
`51`	`51`	`}`
	`52`	`+ store.cache.Reset()`
`52`	`53`
`53`	`54`	`if store.db, err = sql.Open(store.sqlDriver, store.sqlDataSourceName); err != nil {`
`54`	`55`	`return nil, err`
Original file line number	Diff line number	Diff line change
`@@ -164,6 +164,8 @@ func (suite *MessageStoreTestSuite) TestMessageStore_GetMessages_VariousRanges()`
`164`	`164`	`}`
`165`	`165`
`166`	`166`	`func (suite *MessageStoreTestSuite) TestMessageStore_CreationTime() {`
	`167`	`+ assert.False(suite.T(), suite.msgStore.CreationTime().IsZero())`
	`168`	`+`
`167`	`169`	`t0 := time.Now()`
`168`	`170`	`suite.msgStore.Reset()`
`169`	`171`	`t1 := time.Now()`