Update testmempoolaccept RPC

Author: jlest01

integration_test/src/main.rs

@@ -762,12 +762,12 @@ fn test_test_mempool_accept(cl: &Client) {
     let tx =
         cl.create_raw_transaction(&[input.clone()], &output, Some(500_000), Some(false)).unwrap();
     let res = cl.test_mempool_accept(&[&tx]).unwrap();
-    assert!(!res[0].allowed);
+    assert!(res[0].allowed.is_some() && !res[0].allowed.unwrap());
     assert!(res[0].reject_reason.is_some());
     let signed =
         cl.sign_raw_transaction_with_wallet(&tx, None, None).unwrap().transaction().unwrap();
     let res = cl.test_mempool_accept(&[&signed]).unwrap();
-    assert!(res[0].allowed, "not allowed: {:?}", res[0].reject_reason);
+    assert!(res[0].allowed.unwrap(), "not allowed: {:?}", res[0].reject_reason);
 }
 
 fn test_wallet_create_funded_psbt(cl: &Client) {

json/src/lib.rs

@@ -833,23 +833,30 @@ impl SignRawTransactionResult {
 #[derive(Clone, PartialEq, Eq, Debug, Deserialize, Serialize)]
 pub struct TestMempoolAcceptResult {
     pub txid: bitcoin::Txid,
-    pub allowed: bool,
-    #[serde(rename = "reject-reason")]
-    pub reject_reason: Option<String>,
+    pub wtxid: bitcoin::Txid,
+    #[serde(rename = "package-error")]
+    pub package_error: Option<String>,
+    pub allowed: Option<bool>,
     /// Virtual transaction size as defined in BIP 141 (only present when 'allowed' is true)
     /// Added in Bitcoin Core v0.21
     pub vsize: Option<u64>,
     /// Transaction fees (only present if 'allowed' is true)
     /// Added in Bitcoin Core v0.21
     pub fees: Option<TestMempoolAcceptResultFees>,
+    #[serde(rename = "reject-reason")]
+    pub reject_reason: Option<String>,
 }
 
 #[derive(Clone, PartialEq, Eq, Debug, Deserialize, Serialize)]
 pub struct TestMempoolAcceptResultFees {
     /// Transaction fee in BTC
     #[serde(with = "bitcoin::amount::serde::as_btc")]
     pub base: Amount,
-    // unlike GetMempoolEntryResultFees, this only has the `base` fee
+    /// The effective feerate per KvB
+    #[serde(rename = "effective-feerate", with = "bitcoin::amount::serde::as_btc")]
+    pub effective_feerate: Amount,
+    #[serde(rename = "effective-includes")]
+    pub effective_includes: Vec<String>,
 }
 
 #[derive(Copy, Clone, PartialEq, Eq, Debug, Deserialize, Serialize)]