encapsulate wrappers from local midhermes (#119)

* encapsulate store wrappers and expect only interfaces to initialize midhermes

* fix indentation

Author: Lagovas

gohermes/midhermes/local/local_mid_hermes.go

@@ -25,6 +25,7 @@ import (
 	"errors"
 	"runtime"
 	"unsafe"
+	"github.com/cossacklabs/hermes-core/gohermes"
 )
 
 type LocalMidHermes struct {
@@ -39,12 +40,26 @@ func finalizeLocalMidHermes(mid_hermes *LocalMidHermes) {
 	mid_hermes.Close()
 }
 
-func NewLocalMidHermes(id []byte, private_key []byte, credentialStore *MidHermesCredentialStore, keyStore *MidHermesKeyStore, dataStore *MidHermesDataStore) (*LocalMidHermes, error) {
-	mh := &LocalMidHermes{credentialStore: credentialStore, keyStore: keyStore, dataStore: dataStore}
+func NewLocalMidHermes(id []byte, private_key []byte, credentialStore gohermes.CredentialStore, keyStore gohermes.KeyStore, dataStore gohermes.DataStore) (*LocalMidHermes, error) {
+	credentialWrapper, err := NewCredentialStore(credentialStore)
+	if err != nil{
+		return nil, err
+	}
+	keystoreWrapper, err := NewKeyStore(keyStore)
+	if err != nil{
+		return nil, err
+	}
+	datastoreWrapper, err := NewDataStore(dataStore)
+	if err != nil{
+		return nil, err
+	}
+	mh := &LocalMidHermes{credentialStore: credentialWrapper, keyStore: keystoreWrapper, dataStore: datastoreWrapper}
 	mh.mid_hermes = C.mid_hermes_create_with_services(
 		(*C.uint8_t)(unsafe.Pointer(&id[0])), C.size_t(len(id)),
 		(*C.uint8_t)(unsafe.Pointer(&private_key[0])), C.size_t(len(private_key)),
-		(*C.hermes_key_store_t)(unsafe.Pointer(keyStore.GetHermesKeyStore())), (*C.hermes_data_store_t)(unsafe.Pointer(dataStore.GetHermesDataStore())), (*C.hermes_credential_store_t)(unsafe.Pointer(credentialStore.GetHermesCredentialStore())))
+		(*C.hermes_key_store_t)(unsafe.Pointer(keystoreWrapper.GetHermesKeyStore())),
+		(*C.hermes_data_store_t)(unsafe.Pointer(datastoreWrapper.GetHermesDataStore())),
+		(*C.hermes_credential_store_t)(unsafe.Pointer(credentialWrapper.GetHermesCredentialStore())))
 	if nil == mh.mid_hermes {
 		return mh, errors.New("LocalMidHermes object creation error")
 	}

gohermes/midhermes/local/local_mid_hermes_test.go

@@ -106,23 +106,7 @@ func (store *SimpleDataStore) DeleteBlock(id []byte, oldMac []byte) error { retu
 func (store *SimpleDataStore) Close() error { return nil }
 
 func TestNewLocalMidHermes(t *testing.T) {
-	credentialStore := &SimpleCredentialStore{}
-	midhermesCredentialStore, err := NewCredentialStore(credentialStore)
-	if err != nil {
-		t.Fatal(err)
-	}
-
-	midhermesKeyStore, err := NewKeyStore(&SimpleKeyStore{})
-	if err != nil {
-		t.Fatal(err)
-	}
-
-	midhermesDataStore, err := NewDataStore(&SimpleDataStore{})
-	if err != nil {
-		t.Fatal(err)
-	}
-
-	hermesWrapper, err := NewLocalMidHermes(userId, userPrivateKey, midhermesCredentialStore, midhermesKeyStore, midhermesDataStore)
+	hermesWrapper, err := NewLocalMidHermes(userId, userPrivateKey, &SimpleCredentialStore{}, &SimpleKeyStore{}, &SimpleDataStore{})
 	if err != nil {
 		t.Fatal(err)
 	}