From 7c5ebbf87e2c716dea3a813e22a21f6fdd2d14e9 Mon Sep 17 00:00:00 2001
From: zebra-lucky <zebra.lucky@gmail.com>
Date: Mon, 28 Oct 2024 23:26:44 +0200
Subject: [PATCH] fix estimate_fee_per_kb

---
 electrum/plugins/joinmarket/jm_base_code.py      |  4 +---
 electrum/plugins/joinmarket/notes.txt            | 16 +++-------------
 .../tests/jmbitcoin/test_tx_signing.py           |  2 +-
 3 files changed, 5 insertions(+), 17 deletions(-)

diff --git a/electrum/plugins/joinmarket/jm_base_code.py b/electrum/plugins/joinmarket/jm_base_code.py
index 9e192f007..77a84563b 100644
--- a/electrum/plugins/joinmarket/jm_base_code.py
+++ b/electrum/plugins/joinmarket/jm_base_code.py
@@ -136,9 +136,7 @@ class JMBaseCodeMixin:
         tx_fees_factor = abs(self.jmconf.tx_fees_factor)
 
         # NOTE: can not get data from estimatesmartfee with electrum protocol,
-        # but value with default config seems 1000 sat/kb
-        mempoolminfee_in_sat = 1000
-        # mempoolminfee_in_sat = self._get_mempool_min_fee()
+        mempoolminfee_in_sat = None
         # in case of error
         if mempoolminfee_in_sat is None:
             mempoolminfee_in_sat = fallback_fee
diff --git a/electrum/plugins/joinmarket/notes.txt b/electrum/plugins/joinmarket/notes.txt
index e80273b78..c57e2a714 100644
--- a/electrum/plugins/joinmarket/notes.txt
+++ b/electrum/plugins/joinmarket/notes.txt
@@ -1,7 +1,8 @@
 -------------------------------------------------------------------------------
 plugin:
+    - покращення покриття тестами
     - потрібно якось документувати
-    - окремий пункт до якого я взагалі ще не дійшов це qml gui
+    - qml gui
     - jm_qt_obwatch: advanced sorting by columns, filter by text
     - irc reconnect interval 5 sec -> change to progressive interval
     - коли yieldgenerator запускається після ob-watcher.py дані по fidelity
@@ -9,17 +10,6 @@ plugin:
     - коли IRC сервер запускається після зупинки треба через деякий час
       відсилати повторний запит по OBWatch -- makers не шлють offers
       після reconnect
-tests:
-    - jm_utils: add_txin_sig: add testing for p2sh-p2wpkh
-    - tests: place back test_walletservice.py, test_wallet.py?,
-      test_tx_creation
-    - tests: look to base code test/jmclient/test_coinjoin.py
-        test/unified/test_e2e_coinjoin.py
-    - ну і так... покриття тестами як завжди неповне...
 -------------------------------------------------------------------------------
-блін, транзакції з низьким fee позалипали без блоків, тепер у makers немає
-liquidity
-
-це до речі можлива DOS атака на makers
-на makers треба перевіряти tx fee по поточним feerate
+транзакції з низьким fee позалипали без блоків, тепер у makers немає liquidity
 -------------------------------------------------------------------------------
diff --git a/electrum/plugins/joinmarket/tests/jmbitcoin/test_tx_signing.py b/electrum/plugins/joinmarket/tests/jmbitcoin/test_tx_signing.py
index d12fe797e..5e115db43 100644
--- a/electrum/plugins/joinmarket/tests/jmbitcoin/test_tx_signing.py
+++ b/electrum/plugins/joinmarket/tests/jmbitcoin/test_tx_signing.py
@@ -55,7 +55,7 @@ from electrum.plugins.joinmarket import jmbitcoin as btc
 # 7ec1cdc658945013e04ff3d6946f15ccc9df52c323f0ac012e7f61
 #
 # Virtual sizes can be calculated from electrum.transaction.Transaction.
-# deserialize(unhexlify(txhex)).get_virtual_size()  FIXME
+# deserialize(unhexlify(txhex)).get_virtual_size()
 #
 # More cases copied from:
 # https://github.com/kristapsk/bitcoin-scripts/