Added statusBar, getAllInboxMessages, getAllInboxMessageIds, getInboxMessageById, getInboxMessagesByReceiver, trashMessage, trashInboxMessage & addSubscription, methods testcases & updated related functions

src/api.py

@@ -89,9 +89,10 @@ from addresses import (
 )
 from bmconfigparser import BMConfigParser
 from debug import logger
-from helper_sql import SqlBulkExecute, sqlExecute, sqlQuery, sqlStoredProcedure
+from helper_sql import SqlBulkExecute, sqlExecute, sqlQuery, sqlStoredProcedure, sql_ready
 from inventory import Inventory
 from network.threads import StoppableThread
+from six.moves import queue
 from version import softwareVersion
 
 try:  # TODO: write tests for XML vulnerabilities
@@ -230,6 +231,7 @@ class singleAPI(StoppableThread):
 
             def serve_forever(self, poll_interval=None):
                 """Start the RPCServer"""
+                sql_ready.wait()
                 while state.shutdown == 0:
                     self.handle_request()
 
@@ -1422,11 +1424,25 @@ class BMRPCDispatcher(object):
         """Test two numeric params"""
         return a + b
 
+    @command('clearUISignalQueue')
+    def HandleclearUISignalQueue(self):
+        """clear UISignalQueue"""
+        queues.UISignalQueue.queue.clear()
+
     @command('statusBar')
     def HandleStatusBar(self, message):
         """Update GUI statusbar message"""
         queues.UISignalQueue.put(('updateStatusBar', message))
 
+    @command('getStatusBar')
+    def HandleGetStatusBar(self):
+        """Get GUI statusbar message"""
+        try:
+            _, data = queues.UISignalQueue.get(block=False)
+        except queue.Empty:
+            return None
+        return data
+
     @command('deleteAndVacuum')
     def HandleDeleteAndVacuum(self):
         """Cleanup trashes and vacuum messages database"""

src/bitmessagemain.py

@@ -33,6 +33,8 @@ import defaults
 import shared
 import shutdown
 import state
+
+from testmode_init import populate_api_test_data
 from bmconfigparser import BMConfigParser
 from debug import logger  # this should go before any threads
 from helper_startup import (
@@ -295,6 +297,9 @@ class Main(object):
         else:
             config.remove_option('bitmessagesettings', 'dontconnect')
 
+        if state.testmode:
+            populate_api_test_data()
+
         if daemon:
             while state.shutdown == 0:
                 time.sleep(1)

src/testmode_init.py

@@ -0,0 +1,40 @@
+import time
+import uuid
+
+import helper_inbox
+import helper_sql
+
+# from .tests.samples import sample_inbox_msg_ids, sample_inbox_msg_receiver_address
+sample_inbox_msg_receiver_address = 'BM-2cWzSnwjJ7yRP3nLEWUV5LisTZyREWSzUK'
+sample_inbox_msg_ids = ['27e644765a3e4b2e973ee7ccf958ea20', '51fc5531-3989-4d69-bbb5-68d64b756f5b',
+                        '2c975c515f8b414db5eea60ba57ba455', 'bc1f2d8a-681c-4cc0-9a12-6067c7e1ac24']
+
+
+def populate_api_test_data():
+    '''Adding test records in inbox table'''
+    helper_sql.sql_ready.wait()
+
+    test1 = (
+        sample_inbox_msg_ids[0], sample_inbox_msg_receiver_address,
+        sample_inbox_msg_receiver_address, 'Test1 subject', int(time.time()),
+        'Test1 body', 'inbox', 2, 0, uuid.uuid4().bytes
+    )
+    test2 = (
+        sample_inbox_msg_ids[1], sample_inbox_msg_receiver_address,
+        sample_inbox_msg_receiver_address, 'Test2 subject', int(time.time()),
+        'Test2 body', 'inbox', 2, 0, uuid.uuid4().bytes
+    )
+    test3 = (
+        sample_inbox_msg_ids[2], sample_inbox_msg_receiver_address,
+        sample_inbox_msg_receiver_address, 'Test3 subject', int(time.time()),
+        'Test3 body', 'inbox', 2, 0, uuid.uuid4().bytes
+    )
+    test4 = (
+        sample_inbox_msg_ids[3], sample_inbox_msg_receiver_address,
+        sample_inbox_msg_receiver_address, 'Test4 subject', int(time.time()),
+        'Test4 body', 'inbox', 2, 0, uuid.uuid4().bytes
+    )
+    helper_inbox.insert(test1)
+    helper_inbox.insert(test2)
+    helper_inbox.insert(test3)
+    helper_inbox.insert(test4)

src/tests/samples.py

@@ -35,3 +35,10 @@ sample_deterministic_addr3 = 'BM-2DBPTgeSawWYZceFD69AbDT5q4iUWtj1ZN'
 sample_deterministic_addr4 = 'BM-2cWzSnwjJ7yRP3nLEWUV5LisTZyREWSzUK'
 sample_daddr3_512 = 18875720106589866286514488037355423395410802084648916523381
 sample_daddr4_512 = 25152821841976547050350277460563089811513157529113201589004
+
+sample_statusbar_msg = "new status bar message"
+sample_inbox_msg_receiver_address = 'BM-2cWzSnwjJ7yRP3nLEWUV5LisTZyREWSzUK'
+sample_inbox_msg_ids = ['27e644765a3e4b2e973ee7ccf958ea20', '51fc5531-3989-4d69-bbb5-68d64b756f5b',
+                        '2c975c515f8b414db5eea60ba57ba455', 'bc1f2d8a-681c-4cc0-9a12-6067c7e1ac24']
+sample_test_subscription_address = ['BM-2cWQLCBGorT9pUGkYSuGGVr9LzE4mRnQaq', 'BM-GtovgYdgs7qXPkoYaRgrLFuFKz1SFpsw']
+sample_subscription_name = 'test sub'

src/tests/test_api.py

@@ -6,12 +6,15 @@ import base64
 import json
 import time
 
+from binascii import hexlify
 from six.moves import xmlrpc_client  # nosec
 
 import psutil
 
 from .samples import (
-    sample_seed, sample_deterministic_addr3, sample_deterministic_addr4)
+    sample_seed, sample_deterministic_addr3, sample_deterministic_addr4, sample_statusbar_msg,
+    sample_inbox_msg_receiver_address, sample_inbox_msg_ids, sample_test_subscription_address,
+    sample_subscription_name)
 
 from .test_process import TestProcessProto
 
@@ -46,17 +49,9 @@ class TestAPIShutdown(TestAPIProto):
 
 
 # TODO: uncovered API commands
-# getAllInboxMessages
-# getAllInboxMessageIds
-# getInboxMessageById
-# getInboxMessagesByReceiver
-# trashMessage
-# trashInboxMessage
-# addSubscription
 # disseminatePreEncryptedMsg
 # disseminatePubkey
 # getMessageDataByDestinationHash
-# statusBar
 
 
 class TestAPI(TestAPIProto):
@@ -91,6 +86,53 @@ class TestAPI(TestAPIProto):
             'API Error 0020: Invalid method: test'
         )
 
+    def test_statusbar_method(self):
+        """Test statusbar method"""
+        self.api.clearUISignalQueue()
+        self.assertEqual(
+            self.api.statusBar(sample_statusbar_msg),
+            'null'
+        )
+        self.assertEqual(
+            self.api.getStatusBar(),
+            sample_statusbar_msg
+        )
+
+    def test_message_inbox(self):
+        """Test message inbox methods"""
+        self.assertTrue(
+            len(json.loads(
+                self.api.getAllInboxMessages())["inboxMessages"]) == 2
+        )
+        self.assertTrue(
+            len(json.loads(
+                self.api.getAllInboxMessageIds())["inboxMessageIds"]) == 2
+        )
+        self.assertTrue(
+            len(json.loads(
+                self.api.getInboxMessageById(hexlify(sample_inbox_msg_ids[2])))["inboxMessage"]) == 1
+        )
+        self.assertTrue(
+            len(json.loads(
+                self.api.getInboxMessagesByReceiver(sample_inbox_msg_receiver_address))["inboxMessages"]) == 2
+        )
+
+    def test_delete_message(self):
+        """Test delete message methods"""
+        messages_before_delete = len(json.loads(self.api.getAllInboxMessageIds())["inboxMessageIds"])
+        self.assertEqual(
+            self.api.trashMessage(hexlify(sample_inbox_msg_ids[0])),
+            'Trashed message (assuming message existed).'
+        )
+        self.assertEqual(
+            self.api.trashInboxMessage(hexlify(sample_inbox_msg_ids[1])),
+            'Trashed inbox message (assuming message existed).'
+        )
+        self.assertEqual(
+            len(json.loads(self.api.getAllInboxMessageIds())["inboxMessageIds"]),
+            messages_before_delete - 2
+        )
+
     def test_clientstatus_consistency(self):
         """If networkStatus is notConnected networkConnections should be 0"""
         status = json.loads(self.api.clientStatus())
@@ -193,9 +235,28 @@ class TestAPI(TestAPIProto):
 
     def test_subscriptions(self):
         """Testing the API commands related to subscriptions"""
+
+        self.assertEqual(
+            self.api.addSubscription(sample_test_subscription_address[0], sample_subscription_name.encode('base64')),
+            'Added subscription.'
+        )
+
+        added_subscription = {'label': None, 'enabled': False}
+        # check_address
+        for sub in json.loads(self.api.listSubscriptions())['subscriptions']:
+            # special address, added when sqlThread starts
+            if sub['address'] == sample_test_subscription_address[0]:
+                added_subscription = sub
+                break
+
+        self.assertEqual(
+            base64.decodestring(added_subscription['label']) if added_subscription['label'] else None,
+            sample_subscription_name)
+        self.assertTrue(added_subscription['enabled'])
+
         for s in json.loads(self.api.listSubscriptions())['subscriptions']:
             # special address, added when sqlThread starts
-            if s['address'] == 'BM-GtovgYdgs7qXPkoYaRgrLFuFKz1SFpsw':
+            if s['address'] == sample_test_subscription_address[1]:
                 self.assertEqual(
                     base64.decodestring(s['label']),
                     'Bitmessage new releases/announcements')
@@ -206,7 +267,10 @@ class TestAPI(TestAPIProto):
                 'Could not find Bitmessage new releases/announcements'
                 ' in subscriptions')
         self.assertEqual(
-            self.api.deleteSubscription('BM-GtovgYdgs7qXPkoYaRgrLFuFKz1SFpsw'),
+            self.api.deleteSubscription(sample_test_subscription_address[0]),
+            'Deleted subscription if it existed.')
+        self.assertEqual(
+            self.api.deleteSubscription(sample_test_subscription_address[1]),
             'Deleted subscription if it existed.')
         self.assertEqual(
             json.loads(self.api.listSubscriptions())['subscriptions'], [])