From 685caf7c3aee6c696b03bf9c5bb6b25faf90c7f4 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 14:40:12 -0800
Subject: [PATCH 1/7] Output inscription ID from `ord wallet inscribe`

---
 src/inscription_id.rs             |   9 ++
 src/main.rs                       |   2 +-
 src/subcommand/wallet/inscribe.rs |  18 +++-
 tests/command_builder.rs          |   6 ++
 tests/lib.rs                      |  33 +++----
 tests/server.rs                   | 153 ++++++++++++++----------------
 tests/wallet/inscribe.rs          | 127 +++++++------------------
 tests/wallet/inscriptions.rs      |  39 ++++----
 tests/wallet/send.rs              |  77 +++++----------
 9 files changed, 188 insertions(+), 276 deletions(-)

diff --git a/src/inscription_id.rs b/src/inscription_id.rs
index ca9dc263e6..a970ffe770 100644
--- a/src/inscription_id.rs
+++ b/src/inscription_id.rs
@@ -15,6 +15,15 @@ impl<'de> Deserialize<'de> for InscriptionId {
   }
 }
 
+impl Serialize for InscriptionId {
+  fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+  where
+    S: Serializer,
+  {
+    serializer.collect_str(self)
+  }
+}
+
 impl Display for InscriptionId {
   fn fmt(&self, f: &mut Formatter) -> fmt::Result {
     write!(f, "{}i{}", self.txid, self.index)
diff --git a/src/main.rs b/src/main.rs
index 016031854c..ab8f2b3df3 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -49,7 +49,7 @@ use {
   html_escaper::{Escape, Trusted},
   lazy_static::lazy_static,
   regex::Regex,
-  serde::{Deserialize, Deserializer, Serialize},
+  serde::{Deserialize, Deserializer, Serialize, Serializer},
   std::{
     cmp,
     collections::{BTreeMap, HashSet, VecDeque},
diff --git a/src/subcommand/wallet/inscribe.rs b/src/subcommand/wallet/inscribe.rs
index fb01fa97f7..7c243e409c 100644
--- a/src/subcommand/wallet/inscribe.rs
+++ b/src/subcommand/wallet/inscribe.rs
@@ -16,6 +16,13 @@ use {
   std::collections::BTreeSet,
 };
 
+#[derive(Serialize)]
+struct Output {
+  commit: Txid,
+  inscription: InscriptionId,
+  reveal: Txid,
+}
+
 #[derive(Debug, Parser)]
 pub(crate) struct Inscribe {
   #[clap(long, help = "Inscribe <SATPOINT>")]
@@ -77,8 +84,15 @@ impl Inscribe {
       .send_raw_transaction(&reveal_tx)
       .context("Failed to send reveal transaction")?;
 
-    println!("commit\t{commit_txid}");
-    println!("reveal\t{reveal_txid}");
+    serde_json::to_writer_pretty(
+      io::stdout(),
+      &Output {
+        commit: commit_txid,
+        reveal: reveal_txid,
+        inscription: reveal_txid.into(),
+      },
+    )?;
+
     Ok(())
   }
 
diff --git a/tests/command_builder.rs b/tests/command_builder.rs
index 7c16286986..0c4c5c58d4 100644
--- a/tests/command_builder.rs
+++ b/tests/command_builder.rs
@@ -134,4 +134,10 @@ impl CommandBuilder {
 
     stdout.into()
   }
+
+  pub(crate) fn output<T: DeserializeOwned>(self) -> T {
+    let stdout = self.stdout_regex(".*").run();
+    serde_json::from_str(&stdout)
+      .unwrap_or_else(|err| panic!("Failed to deserialize JSON: {err}\n{stdout}"))
+  }
 }
diff --git a/tests/lib.rs b/tests/lib.rs
index 4516b772fd..8baffd4bfb 100644
--- a/tests/lib.rs
+++ b/tests/lib.rs
@@ -7,6 +7,7 @@ use {
   pretty_assertions::assert_eq as pretty_assert_eq,
   regex::Regex,
   reqwest::{StatusCode, Url},
+  serde::{de::DeserializeOwned, Deserialize},
   std::{
     fs,
     net::TcpListener,
@@ -35,35 +36,23 @@ macro_rules! assert_regex_match {
   };
 }
 
-fn reveal_txid_from_inscribe_stdout(stdout: &str) -> Txid {
-  stdout
-    .lines()
-    .nth(1)
-    .unwrap()
-    .split('\t')
-    .nth(1)
-    .unwrap()
-    .parse()
-    .unwrap()
+#[derive(Deserialize)]
+struct Inscribe {
+  reveal: Txid,
+  inscription: String,
 }
 
-fn create_inscription(rpc_server: &test_bitcoincore_rpc::Handle, filename: &str) -> String {
+fn inscribe(rpc_server: &test_bitcoincore_rpc::Handle) -> Inscribe {
   rpc_server.mine_blocks(1);
 
-  let stdout = CommandBuilder::new(format!(
-    "--chain {} wallet inscribe {filename}",
-    rpc_server.network()
-  ))
-  .write(filename, "HELLOWORLD")
-  .rpc_server(rpc_server)
-  .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-  .run();
-
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
+  let output = CommandBuilder::new("wallet inscribe foo.txt")
+    .write("foo.txt", "FOO")
+    .rpc_server(rpc_server)
+    .output();
 
   rpc_server.mine_blocks(1);
 
-  format!("{reveal_txid}i0")
+  output
 }
 
 fn create_wallet(rpc_server: &test_bitcoincore_rpc::Handle) {
diff --git a/tests/server.rs b/tests/server.rs
index d71bedcb42..67bc5c4733 100644
--- a/tests/server.rs
+++ b/tests/server.rs
@@ -38,37 +38,29 @@ fn run() {
 fn inscription_page() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  rpc_server.mine_blocks(1);
-
-  let stdout = CommandBuilder::new("wallet inscribe hello.txt")
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
 
-  let reveal_tx = reveal_txid_from_inscribe_stdout(&stdout);
-
-  let inscription_id = format!("{reveal_tx}i0");
-
-  rpc_server.mine_blocks(1);
+  let Inscribe {
+    inscription,
+    reveal,
+  } = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
-    format!("/inscription/{inscription_id}"),
+    format!("/inscription/{inscription}"),
     format!(
-      ".*<meta property=og:image content='/content/{inscription_id}'>.*
+      ".*<meta property=og:image content='/content/{inscription}'>.*
 <h1>Inscription 0</h1>
-.*<a href=/preview/{inscription_id}><iframe .* src=/preview/{inscription_id}></iframe></a>.*
+.*<a href=/preview/{inscription}><iframe .* src=/preview/{inscription}></iframe></a>.*
 <dl>
   <dt>id</dt>
-  <dd class=monospace>{inscription_id}</dd>
+  <dd class=monospace>{inscription}</dd>
   <dt>address</dt>
   <dd class=monospace>bc1.*</dd>
   <dt>output value</dt>
-  <dd>9860</dd>
+  <dd>9862</dd>
   <dt>content</dt>
-  <dd><a href=/content/{inscription_id}>link</a></dd>
+  <dd><a href=/content/{inscription}>link</a></dd>
   <dt>content size</dt>
-  <dd>10 bytes</dd>
+  <dd>3 bytes</dd>
   <dt>content type</dt>
   <dd>text/plain;charset=utf-8</dd>
   <dt>timestamp</dt>
@@ -76,11 +68,11 @@ fn inscription_page() {
   <dt>genesis height</dt>
   <dd>2</dd>
   <dt>genesis transaction</dt>
-  <dd><a class=monospace href=/tx/{reveal_tx}>{reveal_tx}</a></dd>
+  <dd><a class=monospace href=/tx/{reveal}>{reveal}</a></dd>
   <dt>location</dt>
-  <dd class=monospace>{reveal_tx}:0:0</dd>
+  <dd class=monospace>{reveal}:0:0</dd>
   <dt>output</dt>
-  <dd><a class=monospace href=/output/{reveal_tx}:0>{reveal_tx}:0</a></dd>
+  <dd><a class=monospace href=/output/{reveal}:0>{reveal}:0</a></dd>
   <dt>offset</dt>
   <dd>0</dd>
 </dl>.*",
@@ -92,21 +84,14 @@ fn inscription_page() {
 fn inscription_appears_on_reveal_transaction_page() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  let txid = rpc_server.mine_blocks(1)[0].txdata[0].txid();
-
-  let stdout = CommandBuilder::new(format!("wallet inscribe --satpoint {txid}:0:0 hello.txt"))
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
 
-  let reveal_tx = reveal_txid_from_inscribe_stdout(&stdout);
+  let Inscribe { reveal, .. } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
-    format!("/tx/{reveal_tx}"),
-    format!(".*<h1>Transaction .*</h1>.*<a href=/inscription/{reveal_tx}.*"),
+    format!("/tx/{reveal}"),
+    format!(".*<h1>Transaction .*</h1>.*<a href=/inscription/{reveal}.*"),
   );
 }
 
@@ -114,21 +99,18 @@ fn inscription_appears_on_reveal_transaction_page() {
 fn inscription_appears_on_output_page() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  rpc_server.mine_blocks(1);
 
-  let stdout = CommandBuilder::new("wallet inscribe hello.txt")
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  let reveal_tx = reveal_txid_from_inscribe_stdout(&stdout);
+  let Inscribe {
+    reveal,
+    inscription,
+    ..
+  } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
-    format!("/output/{reveal_tx}:0"),
-    format!(".*<h1>Output <span class=monospace>{reveal_tx}:0</span></h1>.*<a href=/inscription/{reveal_tx}.*"),
+    format!("/output/{reveal}:0"),
+    format!(".*<h1>Output <span class=monospace>{reveal}:0</span></h1>.*<a href=/inscription/{inscription}.*"),
   );
 }
 
@@ -136,30 +118,25 @@ fn inscription_appears_on_output_page() {
 fn inscription_page_after_send() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  rpc_server.mine_blocks(1);
-
-  let stdout = CommandBuilder::new("wallet inscribe hello.txt")
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
 
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
-
-  let inscription_id = format!("{reveal_txid}i0");
+  let Inscribe {
+    reveal,
+    inscription,
+    ..
+  } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   let ord_server = TestServer::spawn_with_args(&rpc_server, &[]);
   ord_server.assert_response_regex(
-    format!("/inscription/{inscription_id}"),
+    format!("/inscription/{inscription}"),
     format!(
-      r".*<h1>Inscription 0</h1>.*<dt>location</dt>\s*<dd class=monospace>{reveal_txid}:0:0</dd>.*",
+      r".*<h1>Inscription 0</h1>.*<dt>location</dt>\s*<dd class=monospace>{reveal}:0:0</dd>.*",
     ),
   );
 
   let txid = CommandBuilder::new(format!(
-    "wallet send bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv {inscription_id}"
+    "wallet send bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv {inscription}"
   ))
   .rpc_server(&rpc_server)
   .stdout_regex(".*")
@@ -167,13 +144,13 @@ fn inscription_page_after_send() {
 
   rpc_server.mine_blocks(1);
 
-  let send_txid = txid.trim();
+  let send = txid.trim();
 
   let ord_server = TestServer::spawn_with_args(&rpc_server, &[]);
   ord_server.assert_response_regex(
-    format!("/inscription/{inscription_id}"),
+    format!("/inscription/{inscription}"),
     format!(
-      r".*<h1>Inscription 0</h1>.*<dt>address</dt>\s*<dd class=monospace>bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv</dd>.*<dt>location</dt>\s*<dd class=monospace>{send_txid}:0:0</dd>.*",
+      r".*<h1>Inscription 0</h1>.*<dt>address</dt>\s*<dd class=monospace>bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv</dd>.*<dt>location</dt>\s*<dd class=monospace>{send}:0:0</dd>.*",
     ),
   )
 }
@@ -185,12 +162,12 @@ fn inscription_content() {
 
   rpc_server.mine_blocks(1);
 
-  let inscription_id = create_inscription(&rpc_server, "foo.txt");
+  let output = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
-  let response =
-    TestServer::spawn_with_args(&rpc_server, &[]).request(&format!("/content/{inscription_id}"));
+  let response = TestServer::spawn_with_args(&rpc_server, &[])
+    .request(&format!("/content/{}", output.inscription));
 
   assert_eq!(response.status(), StatusCode::OK);
   assert_eq!(
@@ -201,7 +178,7 @@ fn inscription_content() {
     response.headers().get("content-security-policy").unwrap(),
     "default-src 'unsafe-eval' 'unsafe-inline'"
   );
-  assert_eq!(response.bytes().unwrap(), "HELLOWORLD");
+  assert_eq!(response.bytes().unwrap(), "FOO");
 }
 
 #[test]
@@ -209,15 +186,16 @@ fn home_page_includes_latest_inscriptions() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let inscription_id = create_inscription(&rpc_server, "foo.png");
+  let output = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
     "/",
     format!(
       ".*<h2>Latest Inscriptions</h2>
 <div class=thumbnails>
-  <a href=/inscription/{inscription_id}><iframe .*></a>
-</div>.*"
+  <a href=/inscription/{}><iframe .*></a>
+</div>.*",
+      output.inscription
     ),
   );
 }
@@ -229,9 +207,15 @@ fn home_page_inscriptions_are_sorted() {
 
   let mut inscriptions = String::new();
 
-  for i in 0..8 {
-    let id = create_inscription(&rpc_server, &format!("{i}.png"));
-    inscriptions.insert_str(0, &format!("\n  <a href=/inscription/{id}><iframe .*></a>"));
+  for _ in 0..8 {
+    let output = inscribe(&rpc_server);
+    inscriptions.insert_str(
+      0,
+      &format!(
+        "\n  <a href=/inscription/{}><iframe .*></a>",
+        output.inscription
+      ),
+    );
   }
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
@@ -249,16 +233,17 @@ fn inscriptions_page() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let inscription_id = create_inscription(&rpc_server, "foo.png");
+  let output = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
     "/inscriptions",
     format!(
       ".*<h1>Inscriptions</h1>
 <div class=thumbnails>
-  <a href=/inscription/{inscription_id}>.*</a>
+  <a href=/inscription/{}>.*</a>
 </div>
 .*",
+      output.inscription,
     ),
   );
 }
@@ -270,9 +255,14 @@ fn inscriptions_page_is_sorted() {
 
   let mut inscriptions = String::new();
 
-  for i in 0..8 {
-    let id = create_inscription(&rpc_server, &format!("{i}.png"));
-    inscriptions.insert_str(0, &format!(".*<a href=/inscription/{id}>.*"));
+  for _ in 0..8 {
+    inscriptions.insert_str(
+      0,
+      &format!(
+        ".*<a href=/inscription/{}>.*",
+        inscribe(&rpc_server).inscription
+      ),
+    );
   }
 
   TestServer::spawn_with_args(&rpc_server, &[])
@@ -284,19 +274,20 @@ fn inscriptions_page_has_next_and_previous() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let a = create_inscription(&rpc_server, "a.txt");
-  let b = create_inscription(&rpc_server, "b.txt");
-  let c = create_inscription(&rpc_server, "c.txt");
+  let a = inscribe(&rpc_server);
+  let b = inscribe(&rpc_server);
+  let c = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
-    format!("/inscription/{b}"),
+    format!("/inscription/{}", b.inscription),
     format!(
       ".*<h1>Inscription 1</h1>.*
 <div class=inscription>
-<a class=previous href=/inscription/{a}>❮</a>
-<a href=/preview/{b}>.*</a>
-<a class=next href=/inscription/{c}>❯</a>
-</div>.*"
+<a class=previous href=/inscription/{}>❮</a>
+<a href=/preview/{}>.*</a>
+<a class=next href=/inscription/{}>❯</a>
+</div>.*",
+      a.inscription, b.inscription, c.inscription
     ),
   );
 }
diff --git a/tests/wallet/inscribe.rs b/tests/wallet/inscribe.rs
index 1e5eb455ed..f5e0c706e3 100644
--- a/tests/wallet/inscribe.rs
+++ b/tests/wallet/inscribe.rs
@@ -1,7 +1,7 @@
 use super::*;
 
 #[test]
-fn inscribe_creates_inscription_transactions() {
+fn inscribe_creates_inscriptions() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   rpc_server.mine_blocks(1);
 
@@ -9,65 +9,28 @@ fn inscribe_creates_inscription_transactions() {
 
   create_wallet(&rpc_server);
 
-  let stdout = CommandBuilder::new("wallet inscribe hello.txt")
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
-
-  let inscription_id = format!("{reveal_txid}i0");
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   assert_eq!(rpc_server.descriptors().len(), 3);
 
   rpc_server.mine_blocks(1);
 
   let request =
-    TestServer::spawn_with_args(&rpc_server, &[]).request(&format!("/content/{inscription_id}"));
+    TestServer::spawn_with_args(&rpc_server, &[]).request(&format!("/content/{inscription}"));
 
   assert_eq!(request.status(), 200);
   assert_eq!(
     request.headers().get("content-type").unwrap(),
     "text/plain;charset=utf-8"
   );
-  assert_eq!(request.text().unwrap(), "HELLOWORLD");
-}
-
-#[test]
-fn inscribe_with_satpoint_arg_inscribes_specific_satpoint() {
-  let rpc_server = test_bitcoincore_rpc::spawn();
-  rpc_server.mine_blocks(1);
-
-  assert_eq!(rpc_server.descriptors().len(), 0);
-
-  create_wallet(&rpc_server);
-
-  assert_eq!(rpc_server.descriptors().len(), 2);
-
-  let inscription_id = create_inscription(&rpc_server, "foo.txt");
-
-  assert_eq!(rpc_server.descriptors().len(), 3);
-
-  let request =
-    TestServer::spawn_with_args(&rpc_server, &[]).request(&format!("/content/{inscription_id}"));
-
-  assert_eq!(request.status(), 200);
-  assert_eq!(
-    request.headers().get("content-type").unwrap(),
-    "text/plain;charset=utf-8"
-  );
-  assert_eq!(request.text().unwrap(), "HELLOWORLD");
+  assert_eq!(request.text().unwrap(), "FOO");
 }
 
 #[test]
 fn inscribe_fails_if_bitcoin_core_is_too_old() {
   let rpc_server = test_bitcoincore_rpc::builder().version(230000).build();
 
-  rpc_server.mine_blocks(1);
-
   CommandBuilder::new("wallet inscribe hello.txt")
-    .write("hello.txt", "HELLOWORLD")
     .expected_exit_code(1)
     .expected_stderr("error: Bitcoin Core 24.0.0 or newer required, current version is 23.0.0\n")
     .rpc_server(&rpc_server)
@@ -85,8 +48,7 @@ fn inscribe_no_backup() {
   CommandBuilder::new("wallet inscribe hello.txt --no-backup")
     .write("hello.txt", "HELLOWORLD")
     .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
+    .output::<Inscribe>();
 
   assert_eq!(rpc_server.descriptors().len(), 2);
 }
@@ -134,7 +96,7 @@ fn regtest_has_no_content_size_limit() {
   CommandBuilder::new("--chain regtest wallet inscribe degenerate.png")
     .write("degenerate.png", [1; 1025])
     .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
+    .stdout_regex(".*")
     .run();
 }
 
@@ -149,7 +111,7 @@ fn mainnet_has_no_content_size_limit() {
   CommandBuilder::new("wallet inscribe degenerate.png")
     .write("degenerate.png", [1; 1025])
     .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
+    .stdout_regex(".*")
     .run();
 }
 
@@ -157,12 +119,6 @@ fn mainnet_has_no_content_size_limit() {
 fn inscribe_does_not_use_inscribed_sats_as_cardinal_utxos() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  rpc_server.mine_blocks_with_subsidy(1, 800);
-  CommandBuilder::new("wallet inscribe degenerate.png")
-    .write("degenerate.png", [1; 100])
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
 
   rpc_server.mine_blocks_with_subsidy(1, 100);
 
@@ -181,58 +137,44 @@ fn refuse_to_reinscribe_sats() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  rpc_server.mine_blocks_with_subsidy(1, 800);
-  let stdout = CommandBuilder::new("wallet inscribe degenerate.png")
-    .write("degenerate.png", [1; 100])
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
+  rpc_server.mine_blocks(1);
 
-  let first_inscription_id = reveal_txid_from_inscribe_stdout(&stdout);
+  let Inscribe { reveal, .. } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks_with_subsidy(1, 100);
 
-  CommandBuilder::new(format!(
-    "wallet inscribe --satpoint {first_inscription_id}:0:0 hello.txt"
-  ))
-  .write("hello.txt", "HELLOWORLD")
-  .rpc_server(&rpc_server)
-  .expected_exit_code(1)
-  .expected_stderr(format!(
-    "error: sat at {first_inscription_id}:0:0 already inscribed\n"
-  ))
-  .run();
+  CommandBuilder::new(format!("wallet inscribe --satpoint {reveal}:0:0 hello.txt"))
+    .write("hello.txt", "HELLOWORLD")
+    .rpc_server(&rpc_server)
+    .expected_exit_code(1)
+    .expected_stderr(format!("error: sat at {reveal}:0:0 already inscribed\n"))
+    .run();
 }
 
 #[test]
 fn refuse_to_inscribe_already_inscribed_utxo() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  rpc_server.mine_blocks(1);
 
-  let stdout = CommandBuilder::new("wallet inscribe degenerate.png")
-    .write("degenerate.png", [1; 100])
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  rpc_server.mine_blocks(1);
+  let Inscribe {
+    reveal,
+    inscription,
+    ..
+  } = inscribe(&rpc_server);
 
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
-
-  let inscription_utxo = OutPoint {
-    txid: reveal_txid_from_inscribe_stdout(&stdout),
+  let output = OutPoint {
+    txid: reveal,
     vout: 0,
   };
 
   CommandBuilder::new(format!(
-    "wallet inscribe --satpoint {inscription_utxo}:55555 hello.txt"
+    "wallet inscribe --satpoint {output}:55555 hello.txt"
   ))
   .write("hello.txt", "HELLOWORLD")
   .rpc_server(&rpc_server)
   .expected_exit_code(1)
   .expected_stderr(format!(
-    "error: utxo {inscription_utxo} already inscribed with inscription {reveal_txid}i0 on sat {inscription_utxo}:0\n",
+    "error: utxo {output} already inscribed with inscription {inscription} on sat {output}:0\n",
   ))
   .run();
 }
@@ -243,25 +185,21 @@ fn inscribe_with_optional_satpoint_arg() {
   create_wallet(&rpc_server);
   let txid = rpc_server.mine_blocks(1)[0].txdata[0].txid();
 
-  let stdout = CommandBuilder::new(format!("wallet inscribe hello.txt --satpoint {txid}:0:0"))
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
-
-  let inscription_id = format!("{reveal_txid}i0");
+  let Inscribe { inscription, .. } =
+    CommandBuilder::new(format!("wallet inscribe foo.txt --satpoint {txid}:0:0"))
+      .write("foo.txt", "FOO")
+      .rpc_server(&rpc_server)
+      .output();
 
   rpc_server.mine_blocks(1);
 
   TestServer::spawn_with_args(&rpc_server, &["--index-sats"]).assert_response_regex(
     "/sat/5000000000",
-    format!(".*<a href=/inscription/{inscription_id}>.*"),
+    format!(".*<a href=/inscription/{inscription}>.*"),
   );
 
   TestServer::spawn_with_args(&rpc_server, &[])
-    .assert_response_regex(format!("/content/{inscription_id}",), ".*HELLOWORLD.*");
+    .assert_response_regex(format!("/content/{inscription}",), "FOO");
 }
 
 #[test]
@@ -273,8 +211,7 @@ fn inscribe_with_fee_rate() {
   CommandBuilder::new("--index-sats wallet inscribe degenerate.png --fee-rate 2.0")
     .write("degenerate.png", [1; 520])
     .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
+    .output::<Inscribe>();
 
   let tx = &rpc_server.mempool()[0];
   let mut fee = 0;
diff --git a/tests/wallet/inscriptions.rs b/tests/wallet/inscriptions.rs
index b81503b1fc..2f934f8558 100644
--- a/tests/wallet/inscriptions.rs
+++ b/tests/wallet/inscriptions.rs
@@ -6,20 +6,15 @@ fn inscriptions() {
   create_wallet(&rpc_server);
   rpc_server.mine_blocks(1);
 
-  let reveal_txid = reveal_txid_from_inscribe_stdout(
-    &CommandBuilder::new("wallet inscribe hello.txt")
-      .write("hello.txt", "HELLOWORLD")
-      .rpc_server(&rpc_server)
-      .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-      .run(),
-  );
-  rpc_server.mine_blocks(1);
-
-  let inscription_id = format!("{reveal_txid}i0");
+  let Inscribe {
+    reveal,
+    inscription,
+    ..
+  } = inscribe(&rpc_server);
 
   CommandBuilder::new("wallet inscriptions")
     .rpc_server(&rpc_server)
-    .expected_stdout(format!("{inscription_id}\t{reveal_txid}:0:0\n"))
+    .expected_stdout(format!("{inscription}\t{reveal}:0:0\n"))
     .run();
 
   let stdout = CommandBuilder::new("wallet receive")
@@ -30,7 +25,7 @@ fn inscriptions() {
 
   let address = stdout.trim();
 
-  let stdout = CommandBuilder::new(format!("wallet send {address} {inscription_id}"))
+  let stdout = CommandBuilder::new(format!("wallet send {address} {inscription}"))
     .rpc_server(&rpc_server)
     .expected_exit_code(0)
     .stdout_regex(".*")
@@ -44,7 +39,7 @@ fn inscriptions() {
 
   CommandBuilder::new("wallet inscriptions")
     .rpc_server(&rpc_server)
-    .expected_stdout(format!("{inscription_id}\t{outpoint}:0\n"))
+    .expected_stdout(format!("{inscription}\t{outpoint}:0\n"))
     .run();
 }
 
@@ -55,20 +50,20 @@ fn inscriptions_includes_locked_utxos() {
 
   rpc_server.mine_blocks(1);
 
-  let txid = reveal_txid_from_inscribe_stdout(
-    &CommandBuilder::new("wallet inscribe hello.txt")
-      .write("hello.txt", "HELLOWORLD")
-      .rpc_server(&rpc_server)
-      .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-      .run(),
-  );
+  let Inscribe {
+    inscription,
+    reveal,
+  } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
-  rpc_server.lock(OutPoint { txid, vout: 0 });
+  rpc_server.lock(OutPoint {
+    txid: reveal,
+    vout: 0,
+  });
 
   CommandBuilder::new("wallet inscriptions")
     .rpc_server(&rpc_server)
-    .expected_stdout(format!("{txid}i0\t{txid}:0:0\n"))
+    .expected_stdout(format!("{inscription}\t{reveal}:0:0\n"))
     .run();
 }
diff --git a/tests/wallet/send.rs b/tests/wallet/send.rs
index 7878e4d741..8b1646541d 100644
--- a/tests/wallet/send.rs
+++ b/tests/wallet/send.rs
@@ -6,12 +6,13 @@ fn inscriptions_can_be_sent() {
   create_wallet(&rpc_server);
   rpc_server.mine_blocks(1);
 
-  let inscription_id = create_inscription(&rpc_server, "foo.txt");
+  let output = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   let stdout = CommandBuilder::new(format!(
-    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {inscription_id}"
+    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {}",
+    output.inscription
   ))
   .rpc_server(&rpc_server)
   .stdout_regex(r".*")
@@ -26,11 +27,11 @@ fn inscriptions_can_be_sent() {
 
   let ord_server = TestServer::spawn_with_args(&rpc_server, &[]);
   ord_server.assert_response_regex(
-    format!("/inscription/{inscription_id}"),
+    format!("/inscription/{}", output.inscription),
     format!(
       ".*<h1>Inscription 0</h1>.*<dl>.*
   <dt>content size</dt>
-  <dd>10 bytes</dd>
+  <dd>3 bytes</dd>
   <dt>content type</dt>
   <dd>text/plain;charset=utf-8</dd>
   .*
@@ -65,12 +66,13 @@ fn send_inscribed_sat() {
   create_wallet(&rpc_server);
   rpc_server.mine_blocks(1);
 
-  let inscription_id = create_inscription(&rpc_server, "foo.txt");
+  let output = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   let stdout = CommandBuilder::new(format!(
-    "wallet send bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv  {inscription_id}"
+    "wallet send bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv  {}",
+    output.inscription
   ))
   .rpc_server(&rpc_server)
   .stdout_regex("[[:xdigit:]]{64}\n")
@@ -82,7 +84,7 @@ fn send_inscribed_sat() {
 
   let ord_server = TestServer::spawn_with_args(&rpc_server, &[]);
   ord_server.assert_response_regex(
-    format!("/inscription/{inscription_id}"),
+    format!("/inscription/{}", output.inscription),
     format!(
       ".*<h1>Inscription 0</h1>.*<dt>location</dt>.*<dd class=monospace>{send_txid}:0:0</dd>.*",
     ),
@@ -166,8 +168,7 @@ fn send_does_not_use_inscribed_sats_as_cardinal_utxos() {
   ))
   .write("degenerate.png", [1; 100])
   .rpc_server(&rpc_server)
-  .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-  .run();
+  .output::<Inscribe>();
 
   let txid = rpc_server.mine_blocks_with_subsidy(1, 100)[0].txdata[0].txid();
 
@@ -185,31 +186,25 @@ fn do_not_accidentally_send_an_inscription() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let txid = rpc_server.mine_blocks(1)[0].txdata[0].txid();
-  let stdout = CommandBuilder::new(format!(
-    "wallet inscribe --satpoint {txid}:0:0 degenerate.png"
-  ))
-  .write("degenerate.png", [1; 100])
-  .rpc_server(&rpc_server)
-  .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-  .run();
-
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
+  let Inscribe {
+    reveal,
+    inscription,
+  } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
-  let inscription_utxo = OutPoint {
-    txid: reveal_txid_from_inscribe_stdout(&stdout),
+  let output = OutPoint {
+    txid: reveal,
     vout: 0,
   };
 
   CommandBuilder::new(format!(
-    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4  {inscription_utxo}:55"
+    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {output}:55"
   ))
   .rpc_server(&rpc_server)
   .expected_exit_code(1)
   .expected_stderr(format!(
-    "error: cannot send {inscription_utxo}:55 without also sending inscription {reveal_txid}i0 at {inscription_utxo}:0\n"
+    "error: cannot send {output}:55 without also sending inscription {inscription} at {output}:0\n"
   ))
   .run();
 }
@@ -218,22 +213,14 @@ fn do_not_accidentally_send_an_inscription() {
 fn inscriptions_cannot_be_sent_by_satpoint() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
-  let txid = rpc_server.mine_blocks(1)[0].txdata[0].txid();
 
-  let stdout = CommandBuilder::new(format!("wallet inscribe --satpoint {txid}:0:0 hello.txt"))
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
+  let Inscribe { reveal, .. } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   CommandBuilder::new(format!(
-    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4   {reveal_txid}:0:0"
+    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {reveal}:0:0"
   ))
-  .write("hello.txt", "HELLOWORLD")
   .rpc_server(&rpc_server)
   .expected_stderr("error: inscriptions must be sent by inscription ID\n")
   .expected_exit_code(1)
@@ -271,15 +258,7 @@ fn send_btc_locks_inscriptions() {
 
   rpc_server.mine_blocks(1);
 
-  let stdout = CommandBuilder::new("wallet inscribe hello.txt")
-    .write("hello.txt", "HELLOWORLD")
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  let inscription_id = reveal_txid_from_inscribe_stdout(&stdout);
-
-  rpc_server.mine_blocks(1);
+  let Inscribe { reveal, .. } = inscribe(&rpc_server);
 
   CommandBuilder::new("wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 1btc")
     .rpc_server(&rpc_server)
@@ -294,7 +273,7 @@ fn send_btc_locks_inscriptions() {
         .parse()
         .unwrap(),
       locked: vec![OutPoint {
-        txid: inscription_id,
+        txid: reveal,
         vout: 0,
       }]
     }]
@@ -323,18 +302,10 @@ fn wallet_send_with_fee_rate() {
   create_wallet(&rpc_server);
   rpc_server.mine_blocks(1);
 
-  let stdout = CommandBuilder::new("--index-sats wallet inscribe degenerate.png")
-    .write("degenerate.png", [1; 520])
-    .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
-
-  rpc_server.mine_blocks(1);
-
-  let reveal_txid = reveal_txid_from_inscribe_stdout(&stdout);
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   CommandBuilder::new(format!(
-    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {reveal_txid}i0 --fee-rate 2.0"
+    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {inscription} --fee-rate 2.0"
   ))
   .rpc_server(&rpc_server)
   .stdout_regex("[[:xdigit:]]{64}\n")

From ca85fa26cb0af40f9b755315f17f8c104056910e Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 15:00:43 -0800
Subject: [PATCH 2/7] tweak

---
 src/subcommand/wallet/inscribe.rs | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/subcommand/wallet/inscribe.rs b/src/subcommand/wallet/inscribe.rs
index 7c243e409c..4c33f0ee53 100644
--- a/src/subcommand/wallet/inscribe.rs
+++ b/src/subcommand/wallet/inscribe.rs
@@ -76,20 +76,20 @@ impl Inscribe {
       .sign_raw_transaction_with_wallet(&unsigned_commit_tx, None, None)?
       .hex;
 
-    let commit_txid = client
+    let commit = client
       .send_raw_transaction(&signed_raw_commit_tx)
       .context("Failed to send commit transaction")?;
 
-    let reveal_txid = client
+    let reveal = client
       .send_raw_transaction(&reveal_tx)
       .context("Failed to send reveal transaction")?;
 
     serde_json::to_writer_pretty(
       io::stdout(),
       &Output {
-        commit: commit_txid,
-        reveal: reveal_txid,
-        inscription: reveal_txid.into(),
+        commit,
+        reveal,
+        inscription: reveal.into(),
       },
     )?;
 

From 7535c9f5a0fea01b8655d224653a105657196d05 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 15:06:26 -0800
Subject: [PATCH 3/7] tweak

---
 tests/lib.rs                 | 4 +++-
 tests/server.rs              | 1 +
 tests/wallet/inscribe.rs     | 2 --
 tests/wallet/inscriptions.rs | 1 +
 tests/wallet/send.rs         | 1 +
 5 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/tests/lib.rs b/tests/lib.rs
index 8baffd4bfb..ae6dcac460 100644
--- a/tests/lib.rs
+++ b/tests/lib.rs
@@ -38,8 +38,10 @@ macro_rules! assert_regex_match {
 
 #[derive(Deserialize)]
 struct Inscribe {
-  reveal: Txid,
+  #[allow(dead_code)]
+  commit: Txid,
   inscription: String,
+  reveal: Txid,
 }
 
 fn inscribe(rpc_server: &test_bitcoincore_rpc::Handle) -> Inscribe {
diff --git a/tests/server.rs b/tests/server.rs
index 67bc5c4733..2cc7871879 100644
--- a/tests/server.rs
+++ b/tests/server.rs
@@ -42,6 +42,7 @@ fn inscription_page() {
   let Inscribe {
     inscription,
     reveal,
+    ..
   } = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
diff --git a/tests/wallet/inscribe.rs b/tests/wallet/inscribe.rs
index f5e0c706e3..0b38231063 100644
--- a/tests/wallet/inscribe.rs
+++ b/tests/wallet/inscribe.rs
@@ -13,8 +13,6 @@ fn inscribe_creates_inscriptions() {
 
   assert_eq!(rpc_server.descriptors().len(), 3);
 
-  rpc_server.mine_blocks(1);
-
   let request =
     TestServer::spawn_with_args(&rpc_server, &[]).request(&format!("/content/{inscription}"));
 
diff --git a/tests/wallet/inscriptions.rs b/tests/wallet/inscriptions.rs
index 2f934f8558..3874669e3b 100644
--- a/tests/wallet/inscriptions.rs
+++ b/tests/wallet/inscriptions.rs
@@ -53,6 +53,7 @@ fn inscriptions_includes_locked_utxos() {
   let Inscribe {
     inscription,
     reveal,
+    ..
   } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
diff --git a/tests/wallet/send.rs b/tests/wallet/send.rs
index 8b1646541d..4a51920888 100644
--- a/tests/wallet/send.rs
+++ b/tests/wallet/send.rs
@@ -189,6 +189,7 @@ fn do_not_accidentally_send_an_inscription() {
   let Inscribe {
     reveal,
     inscription,
+    ..
   } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);

From c8037f16751d298032068ba2d1267d67c8626940 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 15:10:27 -0800
Subject: [PATCH 4/7] tweak

---
 tests/server.rs | 18 +++++++-----------
 1 file changed, 7 insertions(+), 11 deletions(-)

diff --git a/tests/server.rs b/tests/server.rs
index 2cc7871879..1fed9651dd 100644
--- a/tests/server.rs
+++ b/tests/server.rs
@@ -163,12 +163,12 @@ fn inscription_content() {
 
   rpc_server.mine_blocks(1);
 
-  let output = inscribe(&rpc_server);
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
-  let response = TestServer::spawn_with_args(&rpc_server, &[])
-    .request(&format!("/content/{}", output.inscription));
+  let response =
+    TestServer::spawn_with_args(&rpc_server, &[]).request(&format!("/content/{inscription}"));
 
   assert_eq!(response.status(), StatusCode::OK);
   assert_eq!(
@@ -187,16 +187,15 @@ fn home_page_includes_latest_inscriptions() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let output = inscribe(&rpc_server);
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
     "/",
     format!(
       ".*<h2>Latest Inscriptions</h2>
 <div class=thumbnails>
-  <a href=/inscription/{}><iframe .*></a>
+  <a href=/inscription/{inscription}><iframe .*></a>
 </div>.*",
-      output.inscription
     ),
   );
 }
@@ -209,13 +208,10 @@ fn home_page_inscriptions_are_sorted() {
   let mut inscriptions = String::new();
 
   for _ in 0..8 {
-    let output = inscribe(&rpc_server);
+    let Inscribe { inscription, .. } = inscribe(&rpc_server);
     inscriptions.insert_str(
       0,
-      &format!(
-        "\n  <a href=/inscription/{}><iframe .*></a>",
-        output.inscription
-      ),
+      &format!("\n  <a href=/inscription/{inscription}><iframe .*></a>"),
     );
   }
 

From 8024e606858db51a1be02cb6099d1b3372462e86 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 15:13:19 -0800
Subject: [PATCH 5/7] tweak

---
 tests/server.rs      | 29 +++++++++++------------------
 tests/wallet/send.rs |  7 +++----
 2 files changed, 14 insertions(+), 22 deletions(-)

diff --git a/tests/server.rs b/tests/server.rs
index 1fed9651dd..160f364f3a 100644
--- a/tests/server.rs
+++ b/tests/server.rs
@@ -230,17 +230,16 @@ fn inscriptions_page() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let output = inscribe(&rpc_server);
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
     "/inscriptions",
     format!(
       ".*<h1>Inscriptions</h1>
 <div class=thumbnails>
-  <a href=/inscription/{}>.*</a>
+  <a href=/inscription/{inscription}>.*</a>
 </div>
 .*",
-      output.inscription,
     ),
   );
 }
@@ -253,13 +252,8 @@ fn inscriptions_page_is_sorted() {
   let mut inscriptions = String::new();
 
   for _ in 0..8 {
-    inscriptions.insert_str(
-      0,
-      &format!(
-        ".*<a href=/inscription/{}>.*",
-        inscribe(&rpc_server).inscription
-      ),
-    );
+    let Inscribe { inscription, .. } = inscribe(&rpc_server);
+    inscriptions.insert_str(0, &format!(".*<a href=/inscription/{inscription}>.*"));
   }
 
   TestServer::spawn_with_args(&rpc_server, &[])
@@ -271,20 +265,19 @@ fn inscriptions_page_has_next_and_previous() {
   let rpc_server = test_bitcoincore_rpc::spawn();
   create_wallet(&rpc_server);
 
-  let a = inscribe(&rpc_server);
-  let b = inscribe(&rpc_server);
-  let c = inscribe(&rpc_server);
+  let Inscribe { inscription: a, .. } = inscribe(&rpc_server);
+  let Inscribe { inscription: b, .. } = inscribe(&rpc_server);
+  let Inscribe { inscription: c, .. } = inscribe(&rpc_server);
 
   TestServer::spawn_with_args(&rpc_server, &[]).assert_response_regex(
-    format!("/inscription/{}", b.inscription),
+    format!("/inscription/{b}"),
     format!(
       ".*<h1>Inscription 1</h1>.*
 <div class=inscription>
-<a class=previous href=/inscription/{}>❮</a>
-<a href=/preview/{}>.*</a>
-<a class=next href=/inscription/{}>❯</a>
+<a class=previous href=/inscription/{a}>❮</a>
+<a href=/preview/{b}>.*</a>
+<a class=next href=/inscription/{c}>❯</a>
 </div>.*",
-      a.inscription, b.inscription, c.inscription
     ),
   );
 }
diff --git a/tests/wallet/send.rs b/tests/wallet/send.rs
index 4a51920888..5c3afe19f2 100644
--- a/tests/wallet/send.rs
+++ b/tests/wallet/send.rs
@@ -6,13 +6,12 @@ fn inscriptions_can_be_sent() {
   create_wallet(&rpc_server);
   rpc_server.mine_blocks(1);
 
-  let output = inscribe(&rpc_server);
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   let stdout = CommandBuilder::new(format!(
-    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {}",
-    output.inscription
+    "wallet send bc1qw508d6qejxtdg4y5r3zarvary0c5xw7kv8f3t4 {inscription}",
   ))
   .rpc_server(&rpc_server)
   .stdout_regex(r".*")
@@ -27,7 +26,7 @@ fn inscriptions_can_be_sent() {
 
   let ord_server = TestServer::spawn_with_args(&rpc_server, &[]);
   ord_server.assert_response_regex(
-    format!("/inscription/{}", output.inscription),
+    format!("/inscription/{inscription}"),
     format!(
       ".*<h1>Inscription 0</h1>.*<dl>.*
   <dt>content size</dt>

From 58c67a25e558ccc5e5d209d00b03f76894d59ac3 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 15:13:53 -0800
Subject: [PATCH 6/7] tweak

---
 tests/wallet/send.rs | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tests/wallet/send.rs b/tests/wallet/send.rs
index 5c3afe19f2..d71f1e9533 100644
--- a/tests/wallet/send.rs
+++ b/tests/wallet/send.rs
@@ -65,13 +65,12 @@ fn send_inscribed_sat() {
   create_wallet(&rpc_server);
   rpc_server.mine_blocks(1);
 
-  let output = inscribe(&rpc_server);
+  let Inscribe { inscription, .. } = inscribe(&rpc_server);
 
   rpc_server.mine_blocks(1);
 
   let stdout = CommandBuilder::new(format!(
-    "wallet send bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv  {}",
-    output.inscription
+    "wallet send bc1qcqgs2pps4u4yedfyl5pysdjjncs8et5utseepv {inscription}",
   ))
   .rpc_server(&rpc_server)
   .stdout_regex("[[:xdigit:]]{64}\n")
@@ -83,7 +82,7 @@ fn send_inscribed_sat() {
 
   let ord_server = TestServer::spawn_with_args(&rpc_server, &[]);
   ord_server.assert_response_regex(
-    format!("/inscription/{}", output.inscription),
+    format!("/inscription/{inscription}"),
     format!(
       ".*<h1>Inscription 0</h1>.*<dt>location</dt>.*<dd class=monospace>{send_txid}:0:0</dd>.*",
     ),

From 821ce85b4aba8f12848ee48bd3fa5a64fb48cd14 Mon Sep 17 00:00:00 2001
From: Casey Rodarmor <casey@rodarmor.com>
Date: Thu, 12 Jan 2023 15:21:46 -0800
Subject: [PATCH 7/7] tweak

---
 tests/wallet/inscribe.rs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/wallet/inscribe.rs b/tests/wallet/inscribe.rs
index 987f412214..550913c39e 100644
--- a/tests/wallet/inscribe.rs
+++ b/tests/wallet/inscribe.rs
@@ -241,6 +241,5 @@ fn inscribe_with_wallet_named_foo() {
   CommandBuilder::new("--wallet foo wallet inscribe degenerate.png")
     .write("degenerate.png", [1; 520])
     .rpc_server(&rpc_server)
-    .stdout_regex("commit\t[[:xdigit:]]{64}\nreveal\t[[:xdigit:]]{64}\n")
-    .run();
+    .output::<Inscribe>();
 }