aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPéter Szilágyi <peterke@gmail.com>2017-01-17 19:25:36 +0800
committerPéter Szilágyi <peterke@gmail.com>2017-01-17 19:25:36 +0800
commit230530f5ea71f45c5d89f62126e60d24635136d5 (patch)
treee1f64e3903b85ac8be400bdcec94b5cf9bbc0a24
parent26d385c18b5eb003d9a69ff618c78acbe594db44 (diff)
downloadgo-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.tar
go-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.tar.gz
go-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.tar.bz2
go-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.tar.lz
go-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.tar.xz
go-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.tar.zst
go-tangerine-230530f5ea71f45c5d89f62126e60d24635136d5.zip
accounts, mobile: make account manager API a bit more uniform
-rw-r--r--accounts/account_manager.go6
-rw-r--r--accounts/accounts_test.go8
-rw-r--r--mobile/accounts.go2
3 files changed, 8 insertions, 8 deletions
diff --git a/accounts/account_manager.go b/accounts/account_manager.go
index 6460b6e85..01dd62e25 100644
--- a/accounts/account_manager.go
+++ b/accounts/account_manager.go
@@ -113,9 +113,9 @@ func (am *Manager) Accounts() []Account {
return am.cache.accounts()
}
-// DeleteAccount deletes the key matched by account if the passphrase is correct.
-// If a contains no filename, the address must match a unique key.
-func (am *Manager) DeleteAccount(a Account, passphrase string) error {
+// Delete deletes the key matched by account if the passphrase is correct.
+// If the account contains no filename, the address must match a unique key.
+func (am *Manager) Delete(a Account, passphrase string) error {
// Decrypting the key isn't really necessary, but we do
// it anyway to check the password and zero out the key
// immediately afterwards.
diff --git a/accounts/accounts_test.go b/accounts/accounts_test.go
index f276059e2..b3ab87d50 100644
--- a/accounts/accounts_test.go
+++ b/accounts/accounts_test.go
@@ -53,14 +53,14 @@ func TestManager(t *testing.T) {
if err := am.Update(a, "foo", "bar"); err != nil {
t.Errorf("Update error: %v", err)
}
- if err := am.DeleteAccount(a, "bar"); err != nil {
- t.Errorf("DeleteAccount error: %v", err)
+ if err := am.Delete(a, "bar"); err != nil {
+ t.Errorf("Delete error: %v", err)
}
if common.FileExist(a.File) {
- t.Errorf("account file %s should be gone after DeleteAccount", a.File)
+ t.Errorf("account file %s should be gone after Delete", a.File)
}
if am.HasAddress(a.Address) {
- t.Errorf("HasAccount(%x) should've returned true after DeleteAccount", a.Address)
+ t.Errorf("HasAccount(%x) should've returned true after Delete", a.Address)
}
}
diff --git a/mobile/accounts.go b/mobile/accounts.go
index 47c3a5c21..621be4d7a 100644
--- a/mobile/accounts.go
+++ b/mobile/accounts.go
@@ -103,7 +103,7 @@ func (am *AccountManager) GetAccounts() *Accounts {
// DeleteAccount deletes the key matched by account if the passphrase is correct.
// If a contains no filename, the address must match a unique key.
func (am *AccountManager) DeleteAccount(account *Account, passphrase string) error {
- return am.manager.DeleteAccount(accounts.Account{
+ return am.manager.Delete(accounts.Account{
Address: account.account.Address,
File: account.account.File,
}, passphrase)