review: Make method names consistent

- Improve docs for `test_utils::insert_tx`

Author: ValuedMammal

wallet/src/test_utils.rs

@@ -312,7 +312,9 @@ pub fn insert_checkpoint(wallet: &mut Wallet, block: BlockId) {
         .unwrap();
 }
 
-/// Insert a transaction at the current system time.
+/// Inserts a transaction into the local view, assuming it is currently present in the mempool.
+///
+/// This can be used, for example, to track a transaction immediately after it is broadcast.
 pub fn insert_tx(wallet: &mut Wallet, tx: Transaction) {
     let txid = tx.compute_txid();
     let seen_at = std::time::UNIX_EPOCH.elapsed().unwrap().as_secs();

wallet/src/wallet/mod.rs

@@ -2373,7 +2373,10 @@ impl Wallet {
     ///
     /// The `start_time` is used to record the time that a mempool transaction was last seen
     /// (or evicted). See [`Wallet::start_sync_with_revealed_spks`] for more.
-    pub fn start_sync_at(&self, start_time: u64) -> SyncRequestBuilder<(KeychainKind, u32)> {
+    pub fn start_sync_with_revealed_spks_at(
+        &self,
+        start_time: u64,
+    ) -> SyncRequestBuilder<(KeychainKind, u32)> {
         use bdk_chain::keychain_txout::SyncRequestBuilderExt;
         SyncRequest::builder_at(start_time)
             .chain_tip(self.chain.tip())