From 97c290556a77abc8e09964c253c191004fbd4e03 Mon Sep 17 00:00:00 2001
From: zebra-lucky <zebra.lucky@gmail.com>
Date: Mon, 28 Oct 2024 23:34:17 +0200
Subject: [PATCH] Revert "fix estimate_fee_per_kb"

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

diff --git a/electrum/plugins/joinmarket/jm_base_code.py b/electrum/plugins/joinmarket/jm_base_code.py
index 77a84563b..9e192f007 100644
--- a/electrum/plugins/joinmarket/jm_base_code.py
+++ b/electrum/plugins/joinmarket/jm_base_code.py
@@ -136,7 +136,9 @@ class JMBaseCodeMixin:
         tx_fees_factor = abs(self.jmconf.tx_fees_factor)
 
         # NOTE: can not get data from estimatesmartfee with electrum protocol,
-        mempoolminfee_in_sat = None
+        # but value with default config seems 1000 sat/kb
+        mempoolminfee_in_sat = 1000
+        # mempoolminfee_in_sat = self._get_mempool_min_fee()
         # 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 c57e2a714..e80273b78 100644
--- a/electrum/plugins/joinmarket/notes.txt
+++ b/electrum/plugins/joinmarket/notes.txt
@@ -1,8 +1,7 @@
 -------------------------------------------------------------------------------
 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
@@ -10,6 +9,17 @@ 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
+блін, транзакції з низьким fee позалипали без блоків, тепер у makers немає
+liquidity
+
+це до речі можлива DOS атака на makers
+на makers треба перевіряти tx fee по поточним feerate
 -------------------------------------------------------------------------------
diff --git a/electrum/plugins/joinmarket/tests/jmbitcoin/test_tx_signing.py b/electrum/plugins/joinmarket/tests/jmbitcoin/test_tx_signing.py
index 5e115db43..d12fe797e 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()
+# deserialize(unhexlify(txhex)).get_virtual_size()  FIXME
 #
 # More cases copied from:
 # https://github.com/kristapsk/bitcoin-scripts/