functional_tests: add address book tests

3 files changed, 274 insertions, 1 deletions

diff --git a/tests/functional_tests/address_book.py b/tests/functional_tests/address_book.py
new file mode 100755
index 000000000..77a2d200b
--- /dev/null
+++ b/tests/functional_tests/address_book.py
@@ -0,0 +1,238 @@
+#!/usr/bin/env python3
+#encoding=utf-8
+
+# Copyright (c) 2019 The Monero Project
+# 
+# All rights reserved.
+# 
+# Redistribution and use in source and binary forms, with or without modification, are
+# permitted provided that the following conditions are met:
+# 
+# 1. Redistributions of source code must retain the above copyright notice, this list of
+#    conditions and the following disclaimer.
+# 
+# 2. Redistributions in binary form must reproduce the above copyright notice, this list
+#    of conditions and the following disclaimer in the documentation and/or other
+#    materials provided with the distribution.
+# 
+# 3. Neither the name of the copyright holder nor the names of its contributors may be
+#    used to endorse or promote products derived from this software without specific
+#    prior written permission.
+# 
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
+# EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+# MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
+# THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+# PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+# STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
+# THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+import os
+import time
+import errno
+
+"""Test wallet address book RPC
+"""
+
+from framework.wallet import Wallet
+
+class AddressBookTest():
+    def run_test(self):
+      self.create()
+      self.test_address_book()
+
+    def create(self):
+        print 'Creating wallet'
+        wallet = Wallet()
+        # close the wallet if any, will throw if none is loaded
+        try: wallet.close_wallet()
+        except: pass
+        seed = 'velvet lymph giddy number token physics poetry unquoted nibs useful sabotage limits benches lifestyle eden nitrogen anvil fewest avoid batch vials washing fences goat unquoted'
+        res = wallet.restore_deterministic_wallet(seed = seed)
+        assert res.address == '42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm'
+        assert res.seed == seed
+
+    def test_address_book(self):
+        print 'Testing address book'
+        wallet = Wallet()
+
+        # empty at start
+        res = wallet.get_address_book()
+        assert not 'entries' in res or (res.entries) == 0
+        ok = False
+        try: wallet.get_address_book([0])
+        except: ok = True
+        assert ok
+        ok = False
+        try: wallet.delete_address_book(0)
+        except: ok = True
+        assert ok
+
+        # add one
+        res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', description = 'self')
+        assert res.index == 0
+        for get_all in [True, False]:
+            res = wallet.get_address_book() if get_all else wallet.get_address_book([0])
+            assert len(res.entries) == 1
+            e = res.entries[0]
+            assert e.index == 0
+            assert e.address == '42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm'
+            assert e.payment_id == '' or e.payment_id == '0' * 16 or e.payment_id == '0' * 64
+            assert e.description == 'self'
+
+        # add a duplicate
+        res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', description = 'self')
+        assert res.index == 1
+        res = wallet.get_address_book()
+        assert len(res.entries) == 2
+        assert res.entries[0].index == 0
+        assert res.entries[1].index == 1
+        assert res.entries[0].address == res.entries[1].address
+        assert res.entries[0].payment_id == res.entries[1].payment_id
+        assert res.entries[0].description == res.entries[1].description
+        e = res.entries[1]
+        res = wallet.get_address_book([1])
+        assert len(res.entries) == 1
+        assert e == res.entries[0]
+
+        # request (partially) out of range
+        ok = False
+        try: res = wallet.get_address_book[4, 2]
+        except: ok = True
+        assert ok
+        ok = False
+        try: res = wallet.get_address_book[0, 2]
+        except: ok = True
+        assert ok
+        ok = False
+        try: res = wallet.get_address_book[2, 0]
+        except: ok = True
+        assert ok
+
+        # delete first
+        res = wallet.delete_address_book(0)
+        res = wallet.get_address_book()
+        assert len(res.entries) == 1
+        assert res.entries[0].index == 0
+        assert res.entries[0].address == e.address
+        assert res.entries[0].payment_id == e.payment_id
+        assert res.entries[0].description == e.description
+
+        # delete (new) first
+        res = wallet.delete_address_book(0)
+        res = wallet.get_address_book()
+        assert not 'entries' in res or (res.entries) == 0
+
+        # add non-addresses
+        errors = 0
+        try: wallet.add_address_book('', description = 'bad')
+        except: errors += 1
+        try: wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm ', description = 'bad')
+        except: errors += 1
+        try: wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDn', description = 'bad')
+        except: errors += 1
+        try: wallet.add_address_book('9ujeXrjzf7bfeK3KZdCqnYaMwZVFuXemPU8Ubw335rj2FN1CdMiWNyFV3ksEfMFvRp9L9qum5UxkP5rN9aLcPxbH1au4WAB', description = 'bad')
+        except: errors += 1
+        try: wallet.add_address_book('donate@example.com', description = 'bad')
+        except: errors += 1
+        assert errors == 5
+        res = wallet.get_address_book()
+        assert not 'entries' in res or len(res.entries) == 0
+
+        # openalias
+        res = wallet.add_address_book('donate@getmonero.org', description = 'dev fund')
+        assert res.index == 0
+        res = wallet.get_address_book()
+        assert len(res.entries) == 1
+        e = res.entries[0]
+        assert e.address == '44AFFq5kSiGBoZ4NMDwYtN18obc8AemS33DBLWs3H7otXft3XjrpDtQGv7SqSsaBYBb98uNbr2VBBEt7f2wfn3RVGQBEP3A'
+        assert e.description == 'dev fund'
+
+        # UTF-8
+        res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', description = u'あまやかす')
+        assert res.index == 1
+        res = wallet.get_address_book([1])
+        assert len(res.entries) == 1
+        assert res.entries[0].description == u'あまやかす'
+        e = res.entries[0]
+
+        # duplicate request
+        res = wallet.get_address_book([1, 1])
+        assert len(res.entries) == 2
+        assert res.entries[0] == e
+        assert res.entries[1] == e
+
+        # payment IDs
+        res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', payment_id = '0' * 64)
+        assert res.index == 2
+        ok = False
+        try: res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', payment_id = 'x' * 64)
+        except: ok = True
+        assert ok
+        ok = False
+        try: res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', payment_id = '0' * 65)
+        except: ok = True
+        assert ok
+        ok = False
+        try: res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', payment_id = '0' * 63)
+        except: ok = True
+        assert ok
+        ok = False
+        try: res = wallet.add_address_book('42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm', payment_id = '0' * 16)
+        except: ok = True
+        assert ok
+
+        # various address types
+        res = wallet.make_integrated_address()
+        integrated_address = res.integrated_address
+        integrated_address_payment_id = res.payment_id
+        ok = False
+        try: res = wallet.add_address_book(integrated_address, payment_id = '0' * 64)
+        except: ok = True
+        assert ok
+        res = wallet.add_address_book(integrated_address)
+        assert res.index == 3
+        res = wallet.add_address_book('87KfgTZ8ER5D3Frefqnrqif11TjVsTPaTcp37kqqKMrdDRUhpJRczeR7KiBmSHF32UJLP3HHhKUDmEQyJrv2mV8yFDCq8eB')
+        assert res.index == 4
+
+        # get them back
+        res = wallet.get_address_book([0])
+        assert len(res.entries) == 1
+        assert res.entries[0].address == '44AFFq5kSiGBoZ4NMDwYtN18obc8AemS33DBLWs3H7otXft3XjrpDtQGv7SqSsaBYBb98uNbr2VBBEt7f2wfn3RVGQBEP3A'
+        assert res.entries[0].description == 'dev fund'
+        res = wallet.get_address_book([1])
+        assert len(res.entries) == 1
+        assert res.entries[0].address == '42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm'
+        assert res.entries[0].description == u'あまやかす'
+        res = wallet.get_address_book([2])
+        assert len(res.entries) == 1
+        assert res.entries[0].address == '42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm'
+        res = wallet.get_address_book([3])
+        assert len(res.entries) == 1
+        if False: # for now, the address book splits integrated addresses
+          assert res.entries[0].address == integrated_address
+        else:
+          assert res.entries[0].address == '42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm'
+          assert res.entries[0].payment_id == integrated_address_payment_id + '0' * 48
+        res = wallet.get_address_book([4])
+        assert len(res.entries) == 1
+        assert res.entries[0].address == '87KfgTZ8ER5D3Frefqnrqif11TjVsTPaTcp37kqqKMrdDRUhpJRczeR7KiBmSHF32UJLP3HHhKUDmEQyJrv2mV8yFDCq8eB'
+
+        # empty
+        wallet.delete_address_book(4)
+        wallet.delete_address_book(0)
+        res = wallet.get_address_book([0]) # entries above the deleted one collapse one slot up
+        assert len(res.entries) == 1
+        assert res.entries[0].address == '42ey1afDFnn4886T7196doS9GPMzexD9gXpsZJDwVjeRVdFCSoHnv7KPbBeGpzJBzHRCAs9UxqeoyFQMYbqSWYTfJJQAWDm'
+        assert res.entries[0].description == u'あまやかす'
+        wallet.delete_address_book(2)
+        wallet.delete_address_book(0)
+        wallet.delete_address_book(0)
+        res = wallet.get_address_book()
+        assert not 'entries' in res or len(res.entries) == 0
+
+
+if __name__ == '__main__':
+    AddressBookTest().run_test()
diff --git a/tests/functional_tests/functional_tests_rpc.py b/tests/functional_tests/functional_tests_rpc.py
index ebbb793ff..952135206 100755
--- a/tests/functional_tests/functional_tests_rpc.py
+++ b/tests/functional_tests/functional_tests_rpc.py
@@ -10,7 +10,7 @@ import string
 import os
 
 USAGE = 'usage: functional_tests_rpc.py <python> <srcdir> <builddir> [<tests-to-run> | all]'
-DEFAULT_TESTS = ['bans', 'daemon_info', 'blockchain', 'wallet', 'integrated_address', 'mining', 'transfer', 'txpool', 'multisig', 'cold_signing', 'sign_message', 'proofs', 'get_output_distribution']
+DEFAULT_TESTS = ['bans', 'daemon_info', 'blockchain', 'wallet', 'integrated_address', 'mining', 'transfer', 'txpool', 'multisig', 'cold_signing', 'sign_message', 'proofs', 'get_output_distribution', 'address_book']
 try:
   python = sys.argv[1]
   srcdir = sys.argv[2]
diff --git a/utils/python-rpc/framework/wallet.py b/utils/python-rpc/framework/wallet.py
index 7aabb5f29..c67fefc87 100644
--- a/utils/python-rpc/framework/wallet.py
+++ b/utils/python-rpc/framework/wallet.py
@@ -890,6 +890,41 @@ class Wallet(object):
         }
         return self.rpc.send_json_rpc_request(set_account_tag_description)
 
+    def add_address_book(self, address = '', payment_id = '', description = ''):
+        add_address_book = {
+            'method': 'add_address_book',
+            'jsonrpc': '2.0',
+            'params': {
+                'address': address,
+                'payment_id': payment_id,
+                'description': description,
+            },
+            'id': '0'
+        }
+        return self.rpc.send_json_rpc_request(add_address_book)
+
+    def get_address_book(self, entries = []):
+        get_address_book = {
+            'method': 'get_address_book',
+            'jsonrpc': '2.0',
+            'params': {
+                'entries': entries,
+            },
+            'id': '0'
+        }
+        return self.rpc.send_json_rpc_request(get_address_book)
+
+    def delete_address_book(self, index):
+        delete_address_book = {
+            'method': 'delete_address_book',
+            'jsonrpc': '2.0',
+            'params': {
+                'index': index,
+            },
+            'id': '0'
+        }
+        return self.rpc.send_json_rpc_request(delete_address_book)
+
     def start_mining(self, threads_count, do_background_mining = False, ignore_battery = False):
         start_mining = {
             'method': 'start_mining',