message: test: add missing logging to wallet_groups.py
diff: @@ -29,6 +29,7 @@ class WalletGroupTest(BitcoinTestFramework):
  self.skip_if_no_wallet()
  def run_test(self):
  + self.log.info("Setting up")
  # Mine some coins
  self.nodes[0].generate(110)
  @@ -48,6 +49,7 @@ class WalletGroupTest(BitcoinTestFramework):
  # - node[1] should pick one 0.5 UTXO and leave the rest
  # - node[2] should pick one (1.0 + 0.5) UTXO group corresponding to a
  # given address, and leave the rest
  + self.log.info("Test sending transactions picks one UTXO group and leaves the rest")
  txid1 = self.nodes[1].sendtoaddress(self.nodes[0].getnewaddress(), 0.2)
  tx1 = self.nodes[1].getrawtransaction(txid1, True)
  # txid1 should have 1 input and 2 outputs
  @@ -70,7 +72,7 @@ class WalletGroupTest(BitcoinTestFramework):
  assert_approx(v[0], vexp=0.2, vspan=0.0001)
  assert_approx(v[1], vexp=1.3, vspan=0.0001)
  - # Test 'avoid partial if warranted, even if disabled'
  + self.log.info("Test avoiding partial spends if warranted, even if avoidpartialspends is disabled")
  self.sync_all()
  self.nodes[0].generate(1)
  # Nodes 1-2 now have confirmed UTXOs (letters denote destinations):
  @@ -104,7 +106,7 @@ class WalletGroupTest(BitcoinTestFramework):
  assert_equal(input_addrs[0], input_addrs[1])
  # Node 2 enforces avoidpartialspends so needs no checking here
  - # Test wallet option maxapsfee with Node 3
  + self.log.info("Test wallet option maxapsfee")
  addr_aps = self.nodes[3].getnewaddress()
  self.nodes[0].sendtoaddress(addr_aps, 1.0)
  self.nodes[0].sendtoaddress(addr_aps, 1.0)
  @@ -131,6 +133,7 @@ class WalletGroupTest(BitcoinTestFramework):
  # Test wallet option maxapsfee with node 4, which sets maxapsfee
  # 1 sat higher, crossing the threshold from non-grouped to grouped.
  + self.log.info("Test wallet option maxapsfee threshold from non-grouped to grouped")
  addr_aps3 = self.nodes[4].getnewaddress()
  [self.nodes[0].sendtoaddress(addr_aps3, 1.0) for _ in range(5)]
  self.nodes[0].generate(1)
  @@ -147,8 +150,7 @@ class WalletGroupTest(BitcoinTestFramework):
  self.sync_all()
  self.nodes[0].generate(1)
  - # Fill node2's wallet with 10000 outputs corresponding to the same
  - # scriptPubKey
  + self.log.info("Fill a wallet with 10,000 outputs corresponding to the same scriptPubKey")
  for _ in range(5):
  raw_tx = self.nodes[0].createrawtransaction([{"txid":"0"*64, "vout":0}], [{addr2[0]: 0.05}])
  tx = FromHex(CTransaction(), raw_tx)
  @@ -164,6 +166,7 @@ class WalletGroupTest(BitcoinTestFramework):
  # Check that we can create a transaction that only requires ~100 of our
  # utxos, without pulling in all outputs and creating a transaction that
  # is way too big.
  + self.log.info("Test creating txn that only requires ~100 of our UTXOs without pulling in all outputs")
  assert self.nodes[2].sendtoaddress(address=addr2[0], amount=5)
  