add tests

Author: shruggr

block/header_test.go

@@ -2,6 +2,7 @@ package block
 
 import (
 	"encoding/hex"
+	"strings"
 	"testing"
 )
 
@@ -110,3 +111,29 @@ func TestHeaderPrevBlockAndMerkleRoot(t *testing.T) {
 		t.Errorf("PrevBlock = %s, want %s", prevBlockStr, expectedPrevBlock)
 	}
 }
+
+func TestHeaderString(t *testing.T) {
+	genesisHex := "0100000000000000000000000000000000000000000000000000000000000000000000003ba3edfd7a7b12b27ac72c3e67768f617fc81bc3888a51323a9fb8aa4b1e5e4a29ab5f49ffff001d1dac2b7c"
+
+	header, err := NewHeaderFromHex(genesisHex)
+	if err != nil {
+		t.Fatalf("NewHeaderFromHex() error = %v", err)
+	}
+
+	str := header.String()
+	if str == "" {
+		t.Error("String() returned empty string")
+	}
+
+	expectedHash := "000000000019d6689c085ae165831e934ff763ae46a2a6c172b3f1b60a8ce26f"
+	if !strings.Contains(str, expectedHash) {
+		t.Errorf("String() should contain hash %s, got %s", expectedHash, str)
+	}
+}
+
+func TestNewHeaderFromHexInvalid(t *testing.T) {
+	_, err := NewHeaderFromHex("invalid hex")
+	if err == nil {
+		t.Error("NewHeaderFromHex() with invalid hex should return error")
+	}
+}

transaction/beef_test.go

@@ -267,6 +267,26 @@ func TestBeefClone(t *testing.T) {
 	original, err := NewBeefFromBytes(beefBytes)
 	require.NoError(t, err)
 
+	// Test cloning with nil fields by adding a BeefTx with minimal data
+	nilFieldsTxID := chainhash.HashH([]byte("test-nil-fields"))
+	original.Transactions[nilFieldsTxID] = &BeefTx{
+		DataFormat:  TxIDOnly,
+		KnownTxID:   nil,
+		InputTxids:  nil,
+		Transaction: nil,
+	}
+
+	// Test cloning with InputTxids populated
+	inputTxidsTxID := chainhash.HashH([]byte("test-input-txids"))
+	knownID := chainhash.HashH([]byte("known-id"))
+	inputID1 := chainhash.HashH([]byte("input-1"))
+	original.Transactions[inputTxidsTxID] = &BeefTx{
+		DataFormat:  TxIDOnly,
+		KnownTxID:   &knownID,
+		InputTxids:  []*chainhash.Hash{&inputID1, nil},
+		Transaction: nil,
+	}
+
 	// Clone the object
 	clone := original.Clone()
 

transaction/merklepath_test.go

@@ -296,3 +296,25 @@ func TestMerklePathCombine(t *testing.T) {
 		}
 	})
 }
+
+func TestMerklePathClone(t *testing.T) {
+	t.Run("clones nil merkle path", func(t *testing.T) {
+		var mp *MerklePath
+		clone := mp.Clone()
+		require.Nil(t, clone)
+	})
+
+	t.Run("clones valid merkle path", func(t *testing.T) {
+		original, err := NewMerklePathFromHex(BRC74Hex)
+		require.NoError(t, err)
+
+		clone := original.Clone()
+		require.NotNil(t, clone)
+		require.Equal(t, original.BlockHeight, clone.BlockHeight)
+		require.Equal(t, len(original.Path), len(clone.Path))
+
+		// Verify modifying clone doesn't affect original
+		clone.BlockHeight = 999999
+		require.NotEqual(t, original.BlockHeight, clone.BlockHeight)
+	})
+}