Skip to content

Commit e3c4bb1

Browse files
committed
Merge bitcoin/bitcoin#32058: test: get rid of redundant TODO tag
d065208 test: get rid of redundant TODO tag (Chandra Pratap) Pull request description: The `FEE` parameter in `test/functional/feature_dbcrash.py::generate_small_transaction()` is not a fee rate, but an absolute fee. Hence, it doesn't make sense to replace it with node relay based fee calculation. Get rid of the TODO comment suggesting otherwise. ACKs for top commit: maflcko: lgtm ACK d065208 Tree-SHA512: f2b7f51ffb23de8e14ca071edd731410176a20750115a65db0ae67714389e03ffe1593ce88368e96d211329bd93c772f665de7c3a59b932681bc5b80db908d9f
2 parents 930b237 + d065208 commit e3c4bb1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/functional/feature_dbcrash.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ def verify_utxo_hash(self):
184184
assert_equal(nodei_utxo_hash, node3_utxo_hash)
185185

186186
def generate_small_transactions(self, node, count, utxo_list):
187-
FEE = 1000 # TODO: replace this with node relay fee based calculation
187+
FEE = 1000
188188
num_transactions = 0
189189
random.shuffle(utxo_list)
190190
while len(utxo_list) >= 2 and num_transactions < count:

0 commit comments

Comments
 (0)