fix: remove workarounds introduced due to missing bitcoin#20267 (bdb is not compiled)

This partially reverts commit da8e563.
Also it adds missing changed from bitcoin#16404

Author: knst

test/functional/feature_filelock.py

@@ -15,10 +15,6 @@ def set_test_params(self):
         self.setup_clean_chain = True
         self.num_nodes = 2
 
-    # TODO: remove it after 20267 done
-    def skip_test_if_missing_module(self):
-        self.skip_if_no_bdb()
-
     def setup_network(self):
         self.add_nodes(self.num_nodes, extra_args=None)
         self.nodes[0].start()

test/functional/interface_bitcoin_cli.py

@@ -38,8 +38,6 @@ def set_test_params(self):
 
     def skip_test_if_missing_module(self):
         self.skip_if_no_cli()
-        # TODO: drop it when bitcoin#20267 is done
-        self.skip_if_no_bdb()
 
     def run_test(self):
         """Main test logic"""

test/functional/interface_zmq.py

@@ -92,8 +92,6 @@ def test_basic(self):
 
         # Relax so that the subscriber is ready before publishing zmq messages
         sleep(0.2)
-        self.import_deterministic_coinbase_privkeys()
-
 
         num_blocks = 5
         self.log.info("Generate %(n)d blocks (and %(n)d coinbase txes)" % {"n": num_blocks})

test/functional/wallet_descriptor.py

@@ -20,8 +20,6 @@ def set_test_params(self):
 
     def skip_test_if_missing_module(self):
         self.skip_if_no_wallet()
-        # TODO: drop it when bitcoin#20267 is done
-        self.skip_if_no_bdb()
         self.skip_if_no_sqlite()
 
     def run_test(self):