itest: improve comments

Remove inaccurate comments. Rephrase and reword comments for clarity.

Author: ffranr

fn/option.go

@@ -59,7 +59,7 @@ func (o Option[A]) UnwrapOr(a A) A {
 	return a
 }
 
-// UnwrapPtr is used to extract a reference to a value from an option, and we
+// UnwrapToPtr is used to extract a reference to a value from an option, and we
 // supply an empty pointer in the case when the Option is empty.
 func (o Option[A]) UnwrapToPtr() *A {
 	var v *A

itest/psbt_test.go

@@ -1042,8 +1042,8 @@ func testPsbtMultiSend(t *harnessTest) {
 	ctxt, cancel := context.WithTimeout(ctxb, defaultWaitTimeout)
 	defer cancel()
 
-	// Now that we have the asset created, we'll make a new node that'll
-	// serve as the node which'll receive the assets.
+	// With the asset created, we'll set up a new node that will act as the
+	// receiver of the transfer.
 	secondTapd := setupTapdHarness(
 		t.t, t, t.lndHarness.Bob, t.universeServer,
 	)

itest/send_test.go

@@ -573,7 +573,8 @@ func testBasicSendPassiveAsset(t *harnessTest) {
 
 // testReattemptFailedSendHashmailCourier tests that a failed attempt at
 // sending an asset proof will be reattempted by the tapd node. This test
-// targets the hashmail courier.
+// targets the hashmail courier. The proof courier is specified in the test
+// list entry.
 func testReattemptFailedSendHashmailCourier(t *harnessTest) {
 	var (
 		ctxb = context.Background()
@@ -654,11 +655,6 @@ func testReattemptFailedSendHashmailCourier(t *harnessTest) {
 
 	// Simulate a failed attempt at sending the asset proof by stopping
 	// the receiver node.
-	//
-	// The receiving tapd node does not return a proof received confirmation
-	// message via the universe RPC courier. We can simulate a proof
-	// transfer failure by stopping the courier service directly and not the
-	// receiving tapd node.
 	require.NoError(t.t, t.tapd.stop(false))
 
 	// Send asset and then mine to confirm the associated on-chain tx.
@@ -751,11 +747,6 @@ func testReattemptFailedSendUniCourier(t *harnessTest) {
 
 	// Simulate a failed attempt at sending the asset proof by stopping
 	// the proof courier service.
-	//
-	// In following the hashmail proof courier protocol, the receiver node
-	// returns a proof received confirmation message via the courier.
-	// We can simulate a proof transfer failure by stopping the receiving
-	// tapd node. The courier service should still be operational.
 	require.NoError(t.t, t.proofCourier.Stop())
 
 	// Send asset and then mine to confirm the associated on-chain tx.
@@ -765,9 +756,9 @@ func testReattemptFailedSendUniCourier(t *harnessTest) {
 	wg.Wait()
 }
 
-// testReattemptFailedReceiveUniCourier tests that a failed attempt at
-// receiving an asset proof will be reattempted by the receiving tapd node. This
-// test targets the universe proof courier.
+// testReattemptFailedReceiveUniCourier ensures that a failed attempt to receive
+// an asset proof is retried by the receiving Tapd node.  This test focuses on
+// the universe proof courier.
 func testReattemptFailedReceiveUniCourier(t *harnessTest) {
 	ctxb := context.Background()
 

tapdb/multiverse.go

@@ -841,8 +841,8 @@ func (b *MultiverseStore) RootNodes(ctx context.Context,
 }
 
 // FetchProofLeaf returns a proof leaf for the target key. If the key doesn't
-// have a script key specified, then all the proof leafs for the minting
-// outpoint will be returned. If neither are specified, then all inserted proof
+// have a script key specified, then all the proof leafs for the outpoint will
+// be returned. If neither are specified, then all inserted proof
 // leafs will be returned.
 func (b *MultiverseStore) FetchProofLeaf(ctx context.Context,
 	id universe.Identifier,