github.com/luckypickle/go-ethereum-vet@v1.14.2/accounts/keystore/account_cache_test.go (about)

     1  // Copyright 2017 The go-ethereum Authors
     2  // This file is part of the go-ethereum library.
     3  //
     4  // The go-ethereum library is free software: you can redistribute it and/or modify
     5  // it under the terms of the GNU Lesser General Public License as published by
     6  // the Free Software Foundation, either version 3 of the License, or
     7  // (at your option) any later version.
     8  //
     9  // The go-ethereum library is distributed in the hope that it will be useful,
    10  // but WITHOUT ANY WARRANTY; without even the implied warranty of
    11  // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
    12  // GNU Lesser General Public License for more details.
    13  //
    14  // You should have received a copy of the GNU Lesser General Public License
    15  // along with the go-ethereum library. If not, see <http://www.gnu.org/licenses/>.
    16  
    17  package keystore
    18  
    19  import (
    20  	"fmt"
    21  	"io/ioutil"
    22  	"math/rand"
    23  	"os"
    24  	"path/filepath"
    25  	"reflect"
    26  	"sort"
    27  	"testing"
    28  	"time"
    29  
    30  	"github.com/cespare/cp"
    31  	"github.com/davecgh/go-spew/spew"
    32  	"github.com/luckypickle/go-ethereum-vet/accounts"
    33  	"github.com/luckypickle/go-ethereum-vet/common"
    34  )
    35  
    36  var (
    37  	cachetestDir, _   = filepath.Abs(filepath.Join("testdata", "keystore"))
    38  	cachetestAccounts = []accounts.Account{
    39  		{
    40  			Address: common.HexToAddress("7ef5a6135f1fd6a02593eedc869c6d41d934aef8"),
    41  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(cachetestDir, "UTC--2016-03-22T12-57-55.920751759Z--7ef5a6135f1fd6a02593eedc869c6d41d934aef8")},
    42  		},
    43  		{
    44  			Address: common.HexToAddress("f466859ead1932d743d622cb74fc058882e8648a"),
    45  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(cachetestDir, "aaa")},
    46  		},
    47  		{
    48  			Address: common.HexToAddress("289d485d9771714cce91d3393d764e1311907acc"),
    49  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(cachetestDir, "zzz")},
    50  		},
    51  	}
    52  )
    53  
    54  func TestWatchNewFile(t *testing.T) {
    55  	t.Parallel()
    56  
    57  	dir, ks := tmpKeyStore(t, false)
    58  	defer os.RemoveAll(dir)
    59  
    60  	// Ensure the watcher is started before adding any files.
    61  	ks.Accounts()
    62  	time.Sleep(1000 * time.Millisecond)
    63  
    64  	// Move in the files.
    65  	wantAccounts := make([]accounts.Account, len(cachetestAccounts))
    66  	for i := range cachetestAccounts {
    67  		wantAccounts[i] = accounts.Account{
    68  			Address: cachetestAccounts[i].Address,
    69  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(dir, filepath.Base(cachetestAccounts[i].URL.Path))},
    70  		}
    71  		if err := cp.CopyFile(wantAccounts[i].URL.Path, cachetestAccounts[i].URL.Path); err != nil {
    72  			t.Fatal(err)
    73  		}
    74  	}
    75  
    76  	// ks should see the accounts.
    77  	var list []accounts.Account
    78  	for d := 200 * time.Millisecond; d < 5*time.Second; d *= 2 {
    79  		list = ks.Accounts()
    80  		if reflect.DeepEqual(list, wantAccounts) {
    81  			// ks should have also received change notifications
    82  			select {
    83  			case <-ks.changes:
    84  			default:
    85  				t.Fatalf("wasn't notified of new accounts")
    86  			}
    87  			return
    88  		}
    89  		time.Sleep(d)
    90  	}
    91  	t.Errorf("got %s, want %s", spew.Sdump(list), spew.Sdump(wantAccounts))
    92  }
    93  
    94  func TestWatchNoDir(t *testing.T) {
    95  	t.Parallel()
    96  
    97  	// Create ks but not the directory that it watches.
    98  	rand.Seed(time.Now().UnixNano())
    99  	dir := filepath.Join(os.TempDir(), fmt.Sprintf("eth-keystore-watch-test-%d-%d", os.Getpid(), rand.Int()))
   100  	ks := NewKeyStore(dir, LightScryptN, LightScryptP)
   101  
   102  	list := ks.Accounts()
   103  	if len(list) > 0 {
   104  		t.Error("initial account list not empty:", list)
   105  	}
   106  	time.Sleep(100 * time.Millisecond)
   107  
   108  	// Create the directory and copy a key file into it.
   109  	os.MkdirAll(dir, 0700)
   110  	defer os.RemoveAll(dir)
   111  	file := filepath.Join(dir, "aaa")
   112  	if err := cp.CopyFile(file, cachetestAccounts[0].URL.Path); err != nil {
   113  		t.Fatal(err)
   114  	}
   115  
   116  	// ks should see the account.
   117  	wantAccounts := []accounts.Account{cachetestAccounts[0]}
   118  	wantAccounts[0].URL = accounts.URL{Scheme: KeyStoreScheme, Path: file}
   119  	for d := 200 * time.Millisecond; d < 8*time.Second; d *= 2 {
   120  		list = ks.Accounts()
   121  		if reflect.DeepEqual(list, wantAccounts) {
   122  			// ks should have also received change notifications
   123  			select {
   124  			case <-ks.changes:
   125  			default:
   126  				t.Fatalf("wasn't notified of new accounts")
   127  			}
   128  			return
   129  		}
   130  		time.Sleep(d)
   131  	}
   132  	t.Errorf("\ngot  %v\nwant %v", list, wantAccounts)
   133  }
   134  
   135  func TestCacheInitialReload(t *testing.T) {
   136  	cache, _ := newAccountCache(cachetestDir)
   137  	accounts := cache.accounts()
   138  	if !reflect.DeepEqual(accounts, cachetestAccounts) {
   139  		t.Fatalf("got initial accounts: %swant %s", spew.Sdump(accounts), spew.Sdump(cachetestAccounts))
   140  	}
   141  }
   142  
   143  func TestCacheAddDeleteOrder(t *testing.T) {
   144  	cache, _ := newAccountCache("testdata/no-such-dir")
   145  	cache.watcher.running = true // prevent unexpected reloads
   146  
   147  	accs := []accounts.Account{
   148  		{
   149  			Address: common.HexToAddress("095e7baea6a6c7c4c2dfeb977efac326af552d87"),
   150  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "-309830980"},
   151  		},
   152  		{
   153  			Address: common.HexToAddress("2cac1adea150210703ba75ed097ddfe24e14f213"),
   154  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "ggg"},
   155  		},
   156  		{
   157  			Address: common.HexToAddress("8bda78331c916a08481428e4b07c96d3e916d165"),
   158  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "zzzzzz-the-very-last-one.keyXXX"},
   159  		},
   160  		{
   161  			Address: common.HexToAddress("d49ff4eeb0b2686ed89c0fc0f2b6ea533ddbbd5e"),
   162  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "SOMETHING.key"},
   163  		},
   164  		{
   165  			Address: common.HexToAddress("7ef5a6135f1fd6a02593eedc869c6d41d934aef8"),
   166  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "UTC--2016-03-22T12-57-55.920751759Z--7ef5a6135f1fd6a02593eedc869c6d41d934aef8"},
   167  		},
   168  		{
   169  			Address: common.HexToAddress("f466859ead1932d743d622cb74fc058882e8648a"),
   170  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "aaa"},
   171  		},
   172  		{
   173  			Address: common.HexToAddress("289d485d9771714cce91d3393d764e1311907acc"),
   174  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: "zzz"},
   175  		},
   176  	}
   177  	for _, a := range accs {
   178  		cache.add(a)
   179  	}
   180  	// Add some of them twice to check that they don't get reinserted.
   181  	cache.add(accs[0])
   182  	cache.add(accs[2])
   183  
   184  	// Check that the account list is sorted by filename.
   185  	wantAccounts := make([]accounts.Account, len(accs))
   186  	copy(wantAccounts, accs)
   187  	sort.Sort(accountsByURL(wantAccounts))
   188  	list := cache.accounts()
   189  	if !reflect.DeepEqual(list, wantAccounts) {
   190  		t.Fatalf("got accounts: %s\nwant %s", spew.Sdump(accs), spew.Sdump(wantAccounts))
   191  	}
   192  	for _, a := range accs {
   193  		if !cache.hasAddress(a.Address) {
   194  			t.Errorf("expected hasAccount(%x) to return true", a.Address)
   195  		}
   196  	}
   197  	if cache.hasAddress(common.HexToAddress("fd9bd350f08ee3c0c19b85a8e16114a11a60aa4e")) {
   198  		t.Errorf("expected hasAccount(%x) to return false", common.HexToAddress("fd9bd350f08ee3c0c19b85a8e16114a11a60aa4e"))
   199  	}
   200  
   201  	// Delete a few keys from the cache.
   202  	for i := 0; i < len(accs); i += 2 {
   203  		cache.delete(wantAccounts[i])
   204  	}
   205  	cache.delete(accounts.Account{Address: common.HexToAddress("fd9bd350f08ee3c0c19b85a8e16114a11a60aa4e"), URL: accounts.URL{Scheme: KeyStoreScheme, Path: "something"}})
   206  
   207  	// Check content again after deletion.
   208  	wantAccountsAfterDelete := []accounts.Account{
   209  		wantAccounts[1],
   210  		wantAccounts[3],
   211  		wantAccounts[5],
   212  	}
   213  	list = cache.accounts()
   214  	if !reflect.DeepEqual(list, wantAccountsAfterDelete) {
   215  		t.Fatalf("got accounts after delete: %s\nwant %s", spew.Sdump(list), spew.Sdump(wantAccountsAfterDelete))
   216  	}
   217  	for _, a := range wantAccountsAfterDelete {
   218  		if !cache.hasAddress(a.Address) {
   219  			t.Errorf("expected hasAccount(%x) to return true", a.Address)
   220  		}
   221  	}
   222  	if cache.hasAddress(wantAccounts[0].Address) {
   223  		t.Errorf("expected hasAccount(%x) to return false", wantAccounts[0].Address)
   224  	}
   225  }
   226  
   227  func TestCacheFind(t *testing.T) {
   228  	dir := filepath.Join("testdata", "dir")
   229  	cache, _ := newAccountCache(dir)
   230  	cache.watcher.running = true // prevent unexpected reloads
   231  
   232  	accs := []accounts.Account{
   233  		{
   234  			Address: common.HexToAddress("095e7baea6a6c7c4c2dfeb977efac326af552d87"),
   235  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(dir, "a.key")},
   236  		},
   237  		{
   238  			Address: common.HexToAddress("2cac1adea150210703ba75ed097ddfe24e14f213"),
   239  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(dir, "b.key")},
   240  		},
   241  		{
   242  			Address: common.HexToAddress("d49ff4eeb0b2686ed89c0fc0f2b6ea533ddbbd5e"),
   243  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(dir, "c.key")},
   244  		},
   245  		{
   246  			Address: common.HexToAddress("d49ff4eeb0b2686ed89c0fc0f2b6ea533ddbbd5e"),
   247  			URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(dir, "c2.key")},
   248  		},
   249  	}
   250  	for _, a := range accs {
   251  		cache.add(a)
   252  	}
   253  
   254  	nomatchAccount := accounts.Account{
   255  		Address: common.HexToAddress("f466859ead1932d743d622cb74fc058882e8648a"),
   256  		URL:     accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Join(dir, "something")},
   257  	}
   258  	tests := []struct {
   259  		Query      accounts.Account
   260  		WantResult accounts.Account
   261  		WantError  error
   262  	}{
   263  		// by address
   264  		{Query: accounts.Account{Address: accs[0].Address}, WantResult: accs[0]},
   265  		// by file
   266  		{Query: accounts.Account{URL: accs[0].URL}, WantResult: accs[0]},
   267  		// by basename
   268  		{Query: accounts.Account{URL: accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Base(accs[0].URL.Path)}}, WantResult: accs[0]},
   269  		// by file and address
   270  		{Query: accs[0], WantResult: accs[0]},
   271  		// ambiguous address, tie resolved by file
   272  		{Query: accs[2], WantResult: accs[2]},
   273  		// ambiguous address error
   274  		{
   275  			Query: accounts.Account{Address: accs[2].Address},
   276  			WantError: &AmbiguousAddrError{
   277  				Addr:    accs[2].Address,
   278  				Matches: []accounts.Account{accs[2], accs[3]},
   279  			},
   280  		},
   281  		// no match error
   282  		{Query: nomatchAccount, WantError: ErrNoMatch},
   283  		{Query: accounts.Account{URL: nomatchAccount.URL}, WantError: ErrNoMatch},
   284  		{Query: accounts.Account{URL: accounts.URL{Scheme: KeyStoreScheme, Path: filepath.Base(nomatchAccount.URL.Path)}}, WantError: ErrNoMatch},
   285  		{Query: accounts.Account{Address: nomatchAccount.Address}, WantError: ErrNoMatch},
   286  	}
   287  	for i, test := range tests {
   288  		a, err := cache.find(test.Query)
   289  		if !reflect.DeepEqual(err, test.WantError) {
   290  			t.Errorf("test %d: error mismatch for query %v\ngot %q\nwant %q", i, test.Query, err, test.WantError)
   291  			continue
   292  		}
   293  		if a != test.WantResult {
   294  			t.Errorf("test %d: result mismatch for query %v\ngot %v\nwant %v", i, test.Query, a, test.WantResult)
   295  			continue
   296  		}
   297  	}
   298  }
   299  
   300  func waitForAccounts(wantAccounts []accounts.Account, ks *KeyStore) error {
   301  	var list []accounts.Account
   302  	for d := 200 * time.Millisecond; d < 8*time.Second; d *= 2 {
   303  		list = ks.Accounts()
   304  		if reflect.DeepEqual(list, wantAccounts) {
   305  			// ks should have also received change notifications
   306  			select {
   307  			case <-ks.changes:
   308  			default:
   309  				return fmt.Errorf("wasn't notified of new accounts")
   310  			}
   311  			return nil
   312  		}
   313  		time.Sleep(d)
   314  	}
   315  	return fmt.Errorf("\ngot  %v\nwant %v", list, wantAccounts)
   316  }
   317  
   318  // TestUpdatedKeyfileContents tests that updating the contents of a keystore file
   319  // is noticed by the watcher, and the account cache is updated accordingly
   320  func TestUpdatedKeyfileContents(t *testing.T) {
   321  	t.Parallel()
   322  
   323  	// Create a temporary kesytore to test with
   324  	rand.Seed(time.Now().UnixNano())
   325  	dir := filepath.Join(os.TempDir(), fmt.Sprintf("eth-keystore-watch-test-%d-%d", os.Getpid(), rand.Int()))
   326  	ks := NewKeyStore(dir, LightScryptN, LightScryptP)
   327  
   328  	list := ks.Accounts()
   329  	if len(list) > 0 {
   330  		t.Error("initial account list not empty:", list)
   331  	}
   332  	time.Sleep(100 * time.Millisecond)
   333  
   334  	// Create the directory and copy a key file into it.
   335  	os.MkdirAll(dir, 0700)
   336  	defer os.RemoveAll(dir)
   337  	file := filepath.Join(dir, "aaa")
   338  
   339  	// Place one of our testfiles in there
   340  	if err := cp.CopyFile(file, cachetestAccounts[0].URL.Path); err != nil {
   341  		t.Fatal(err)
   342  	}
   343  
   344  	// ks should see the account.
   345  	wantAccounts := []accounts.Account{cachetestAccounts[0]}
   346  	wantAccounts[0].URL = accounts.URL{Scheme: KeyStoreScheme, Path: file}
   347  	if err := waitForAccounts(wantAccounts, ks); err != nil {
   348  		t.Error(err)
   349  		return
   350  	}
   351  
   352  	// needed so that modTime of `file` is different to its current value after forceCopyFile
   353  	time.Sleep(1000 * time.Millisecond)
   354  
   355  	// Now replace file contents
   356  	if err := forceCopyFile(file, cachetestAccounts[1].URL.Path); err != nil {
   357  		t.Fatal(err)
   358  		return
   359  	}
   360  	wantAccounts = []accounts.Account{cachetestAccounts[1]}
   361  	wantAccounts[0].URL = accounts.URL{Scheme: KeyStoreScheme, Path: file}
   362  	if err := waitForAccounts(wantAccounts, ks); err != nil {
   363  		t.Errorf("First replacement failed")
   364  		t.Error(err)
   365  		return
   366  	}
   367  
   368  	// needed so that modTime of `file` is different to its current value after forceCopyFile
   369  	time.Sleep(1000 * time.Millisecond)
   370  
   371  	// Now replace file contents again
   372  	if err := forceCopyFile(file, cachetestAccounts[2].URL.Path); err != nil {
   373  		t.Fatal(err)
   374  		return
   375  	}
   376  	wantAccounts = []accounts.Account{cachetestAccounts[2]}
   377  	wantAccounts[0].URL = accounts.URL{Scheme: KeyStoreScheme, Path: file}
   378  	if err := waitForAccounts(wantAccounts, ks); err != nil {
   379  		t.Errorf("Second replacement failed")
   380  		t.Error(err)
   381  		return
   382  	}
   383  
   384  	// needed so that modTime of `file` is different to its current value after ioutil.WriteFile
   385  	time.Sleep(1000 * time.Millisecond)
   386  
   387  	// Now replace file contents with crap
   388  	if err := ioutil.WriteFile(file, []byte("foo"), 0644); err != nil {
   389  		t.Fatal(err)
   390  		return
   391  	}
   392  	if err := waitForAccounts([]accounts.Account{}, ks); err != nil {
   393  		t.Errorf("Emptying account file failed")
   394  		t.Error(err)
   395  		return
   396  	}
   397  }
   398  
   399  // forceCopyFile is like cp.CopyFile, but doesn't complain if the destination exists.
   400  func forceCopyFile(dst, src string) error {
   401  	data, err := ioutil.ReadFile(src)
   402  	if err != nil {
   403  		return err
   404  	}
   405  	return ioutil.WriteFile(dst, data, 0644)
   406  }