From 8a96fc18c4fc1cb40bc8541638a0a69767b3a292 Mon Sep 17 00:00:00 2001
From: Victor Farazdagi <simple.square@gmail.com>
Date: Sat, 18 Mar 2017 21:23:58 +0300
Subject: [PATCH] geth/params: stop transforming passed DataDir

---
 geth/jail/jail_test.go                   |  2 +-
 geth/params/config.go                    |  7 ++---
 geth/params/config_test.go               | 36 +++++-------------------
 geth/params/testdata/config.testnet.json |  2 +-
 geth/utils.go                            |  6 ++--
 5 files changed, 14 insertions(+), 39 deletions(-)

diff --git a/geth/jail/jail_test.go b/geth/jail/jail_test.go
index ed389df05..e3e01dbff 100644
--- a/geth/jail/jail_test.go
+++ b/geth/jail/jail_test.go
@@ -11,8 +11,8 @@ import (
 
 	"github.com/ethereum/go-ethereum/common"
 	"github.com/status-im/status-go/geth"
-	"github.com/status-im/status-go/geth/params"
 	"github.com/status-im/status-go/geth/jail"
+	"github.com/status-im/status-go/geth/params"
 )
 
 const (
diff --git a/geth/params/config.go b/geth/params/config.go
index 590f43cd6..9aed07cb0 100644
--- a/geth/params/config.go
+++ b/geth/params/config.go
@@ -2,12 +2,12 @@ package params
 
 import (
 	"encoding/json"
+	"errors"
 	"io/ioutil"
+	"math/big"
 	"os"
 	"path/filepath"
 	"strings"
-	"math/big"
-	"errors"
 
 	"github.com/ethereum/go-ethereum/common"
 	"github.com/ethereum/go-ethereum/core"
@@ -195,9 +195,6 @@ func (c *NodeConfig) populateChainConfig() {
 		c.ChainConfig.EIP158Block = params.TestnetChainConfig.EIP158Block
 		c.ChainConfig.ChainId = params.TestnetChainConfig.ChainId
 
-		if len(c.DataDir) > 0 {
-			c.DataDir = filepath.Join(c.DataDir, "testnet")
-		}
 		c.Genesis = core.DefaultTestnetGenesisBlock()
 	} else {
 		// Homestead fork
diff --git a/geth/params/config_test.go b/geth/params/config_test.go
index 805f1f697..4957e71e9 100644
--- a/geth/params/config_test.go
+++ b/geth/params/config_test.go
@@ -7,9 +7,9 @@ import (
 	"strings"
 	"testing"
 
+	gethparams "github.com/ethereum/go-ethereum/params"
 	"github.com/status-im/status-go/geth"
 	"github.com/status-im/status-go/geth/params"
-	gethparams "github.com/ethereum/go-ethereum/params"
 )
 
 var loadConfigTestCases = []struct {
@@ -60,40 +60,18 @@ var loadConfigTestCases = []struct {
 		},
 	},
 	{
-		`testnet subdirectory not used (while we are on Network = 3)`,
+		`check static DataDir passing`,
 		`{
 			"NetworkId": 3,
-			"DataDir": "$TMPDIR"
+			"DataDir": "/storage/emulated/0/ethereum/"
 		}`,
 		func(t *testing.T, dataDir string, nodeConfig *params.NodeConfig, err error) {
 			if err != nil {
 				t.Fatalf("unexpected error: %v", err)
 			}
-			if nodeConfig.DataDir != filepath.Join(dataDir, "testnet") {
-				t.Fatal("'testnet' subdirectory not used")
-			}
-
-			if !strings.Contains(nodeConfig.LightEthConfig.Genesis, "\"chainId\": 3") {
-				t.Fatal("wrong genesis")
-			}
-		},
-	},
-	{
-		`testnet subdirectory used (while we are on Network != 3)`,
-		`{
-			"NetworkId": 1,
-			"DataDir": "$TMPDIR"
-		}`,
-		func(t *testing.T, dataDir string, nodeConfig *params.NodeConfig, err error) {
-			if err != nil {
-				t.Fatalf("unexpected error: %v", err)
-			}
-			if nodeConfig.DataDir != dataDir {
-				t.Fatal("'testnet' subdirectory used")
-			}
-
-			if strings.Contains(nodeConfig.LightEthConfig.Genesis, "\"chainId\": 3") {
-				t.Fatal("wrong genesis")
+			expectedDataDir := "/storage/emulated/0/ethereum/"
+			if nodeConfig.DataDir != expectedDataDir {
+				t.Fatalf("incorrect DataDir used, expected: %v, got: %v", expectedDataDir, nodeConfig.DataDir)
 			}
 		},
 	},
@@ -139,7 +117,7 @@ var loadConfigTestCases = []struct {
 				t.Fatal("wrong WSEnabled")
 			}
 
-			if nodeConfig.IPCEnabled != true{
+			if nodeConfig.IPCEnabled != true {
 				t.Fatal("wrong IPCEnabled")
 			}
 			if nodeConfig.LightEthConfig.DatabaseCache != 64 {
diff --git a/geth/params/testdata/config.testnet.json b/geth/params/testdata/config.testnet.json
index 1b66faa67..db59f3556 100755
--- a/geth/params/testdata/config.testnet.json
+++ b/geth/params/testdata/config.testnet.json
@@ -2,7 +2,7 @@
     "TestNet": true,
     "NetworkId": 3,
     "DataDir": "$TMPDIR",
-    "Name": "status",
+    "Name": "StatusIM",
     "Version": "$VERSION",
     "APIModules": "db,eth,net,web3,shh,personal,admin",
     "HTTPHost": "localhost",
diff --git a/geth/utils.go b/geth/utils.go
index 156539ca3..288b08a45 100644
--- a/geth/utils.go
+++ b/geth/utils.go
@@ -157,18 +157,18 @@ func PrepareTestNode() (err error) {
 	}
 
 	syncRequired := false
-	if _, err := os.Stat(filepath.Join(TestDataDir, "testnet")); os.IsNotExist(err) {
+	if _, err := os.Stat(TestDataDir); os.IsNotExist(err) {
 		syncRequired = true
 	}
 
 	// prepare node directory
-	if err := os.MkdirAll(filepath.Join(TestDataDir, "testnet", "keystore"), os.ModePerm); err != nil {
+	if err := os.MkdirAll(filepath.Join(TestDataDir, "keystore"), os.ModePerm); err != nil {
 		glog.V(logger.Warn).Infoln("make node failed:", err)
 		return err
 	}
 
 	// import test account (with test ether on it)
-	dst := filepath.Join(TestDataDir, "testnet", "keystore", "test-account.pk")
+	dst := filepath.Join(TestDataDir, "keystore", "test-account.pk")
 	if _, err := os.Stat(dst); os.IsNotExist(err) {
 		err = CopyFile(dst, filepath.Join(RootDir, "data", "test-account.pk"))
 		if err != nil {