From e03e890a755c717287d62af8484966cbd919a1a6 Mon Sep 17 00:00:00 2001
From: Uxio Fuentefria <uxio@gnosis.pm>
Date: Fri, 13 Mar 2020 19:08:48 +0100
Subject: [PATCH] Write test for command resending txs

---
 .../relay/tests/test_resend_txs.py            | 97 +++++++++++++++++--
 1 file changed, 88 insertions(+), 9 deletions(-)

diff --git a/safe_relay_service/relay/tests/test_resend_txs.py b/safe_relay_service/relay/tests/test_resend_txs.py
index a37c3606..2b98e3fb 100644
--- a/safe_relay_service/relay/tests/test_resend_txs.py
+++ b/safe_relay_service/relay/tests/test_resend_txs.py
@@ -4,18 +4,97 @@
 from django.test import TestCase
 from django.utils import timezone
 
-from gnosis.eth.tests.ethereum_test_case import EthereumTestCaseMixin
+from eth_account import Account
+from hexbytes import HexBytes
+
+from gnosis.eth.constants import NULL_ADDRESS
+from gnosis.safe import Safe
 
 from ..management.commands import resend_txs
-from .factories import SafeMultisigTxFactory
+from ..models import SafeMultisigTx
+from .relay_test_case import RelayTestCaseMixin
 
 
-class TestResendTxsCommand(EthereumTestCaseMixin, TestCase):
+class TestResendTxsCommand(RelayTestCaseMixin, TestCase):
     def test_resend_txs(self):
-        multisig_tx = SafeMultisigTxFactory(ethereum_tx__block=None)
+        # Nothing happens
         call_command(resend_txs.Command())
-        multisig_tx.created = timezone.now() - timedelta(days=1)
-        multisig_tx.save()
-        with self.assertRaises(ValueError):
-            call_command(resend_txs.Command(), gas_price=multisig_tx.ethereum_tx.gas_price + 1)
-        #TODO More testing
+
+        w3 = self.w3
+        # The balance we will send to the safe
+        safe_balance = w3.toWei(0.02, 'ether')
+
+        # Create Safe
+        accounts = [self.create_account(), self.create_account()]
+
+        # Signatures must be sorted!
+        accounts.sort(key=lambda account: account.address.lower())
+
+        safe_creation = self.deploy_test_safe(owners=[x.address for x in accounts],
+                                              threshold=len(accounts),
+                                              initial_funding_wei=safe_balance)
+        my_safe_address = safe_creation.safe_address
+
+        to = Account().create().address
+        value = safe_balance // 4
+        data = HexBytes('')
+        operation = 0
+        safe_tx_gas = 100000
+        data_gas = 300000
+        gas_price = self.transaction_service._get_minimum_gas_price()
+        gas_token = NULL_ADDRESS
+        refund_receiver = NULL_ADDRESS
+        safe = Safe(my_safe_address, self.ethereum_client)
+        nonce = safe.retrieve_nonce()
+        safe_multisig_tx_hash = safe.build_multisig_tx(to,
+                                                       value,
+                                                       data,
+                                                       operation,
+                                                       safe_tx_gas,
+                                                       data_gas,
+                                                       gas_price,
+                                                       gas_token,
+                                                       refund_receiver,
+                                                       safe_nonce=nonce).safe_tx_hash
+
+        signatures = [account.signHash(safe_multisig_tx_hash) for account in accounts]
+        sender = self.transaction_service.tx_sender_account.address
+
+        # Ganache snapshot
+        snapshot_id = w3.testing.snapshot()
+        safe_multisig_tx = self.transaction_service.create_multisig_tx(
+            my_safe_address,
+            to,
+            value,
+            data,
+            operation,
+            safe_tx_gas,
+            data_gas,
+            gas_price,
+            gas_token,
+            refund_receiver,
+            nonce,
+            signatures,
+        )
+
+        tx_receipt = w3.eth.waitForTransactionReceipt(safe_multisig_tx.ethereum_tx.tx_hash)
+        self.assertTrue(tx_receipt['status'])
+        self.assertEqual(w3.toChecksumAddress(tx_receipt['from']), sender)
+        self.assertEqual(w3.toChecksumAddress(tx_receipt['to']), my_safe_address)
+        self.assertEqual(w3.eth.getBalance(to),  value)
+
+        w3.testing.revert(snapshot_id)  # Revert to snapshot in ganache
+        self.assertEqual(w3.eth.getBalance(to),  0)
+
+        old_multisig_tx: SafeMultisigTx = SafeMultisigTx.objects.all().first()
+        old_multisig_tx.created = timezone.now() - timedelta(days=1)
+        old_multisig_tx.save()
+        new_gas_price = old_multisig_tx.ethereum_tx.gas_price + 1
+
+        call_command(resend_txs.Command(), gas_price=new_gas_price)
+        multisig_tx: SafeMultisigTx = SafeMultisigTx.objects.all().first()
+        self.assertNotEqual(multisig_tx.ethereum_tx_id, old_multisig_tx.ethereum_tx_id)
+        self.assertEqual(multisig_tx.ethereum_tx.gas_price, new_gas_price)
+        self.assertEqual(w3.eth.getBalance(to),  value)  # Tx is executed again
+        self.assertEqual(multisig_tx.get_safe_tx().__dict__,
+                         old_multisig_tx.get_safe_tx().__dict__)