Add extra tests for intended behaviour with P2PKH-only transactions & with old artifacts

Author: rkalis

packages/cashscript/test/TransactionBuilder.test.ts

@@ -296,18 +296,54 @@ describe('Transaction Builder', () => {
     });
   });
 
-  it('should not fail when spending from only P2PKH inputs', async () => {
+  it('should not fail when validly spending from only P2PKH inputs', async () => {
     const aliceUtxos = (await provider.getUtxos(aliceAddress)).filter(isNonTokenUtxo);
     const sigTemplate = new SignatureTemplate(alicePriv);
 
     expect(aliceUtxos.length).toBeGreaterThan(2);
 
     const change = aliceUtxos[0].satoshis + aliceUtxos[1].satoshis - 1000n;
 
-    await expect(new TransactionBuilder({ provider })
+    const transaction = new TransactionBuilder({ provider })
       .addInput(aliceUtxos[0], sigTemplate.unlockP2PKH())
       .addInput(aliceUtxos[1], sigTemplate.unlockP2PKH())
-      .addOutput({ to: aliceAddress, amount: change })
-      .send()).resolves.not.toThrow();
+      .addOutput({ to: aliceAddress, amount: change });
+
+    await expect(transaction.send()).resolves.not.toThrow();
+  });
+
+  // TODO: Currently, P2PKH inputs are not evaluated at all
+  it.skip('should fail when invalidly spending from only P2PKH inputs', async () => {
+    const aliceUtxos = (await provider.getUtxos(aliceAddress)).filter(isNonTokenUtxo);
+    const incorrectSigTemplate = new SignatureTemplate(bobPriv);
+
+    expect(aliceUtxos.length).toBeGreaterThan(2);
+
+    const change = aliceUtxos[0].satoshis + aliceUtxos[1].satoshis - 1000n;
+
+    const transaction = new TransactionBuilder({ provider })
+      .addInput(aliceUtxos[0], incorrectSigTemplate.unlockP2PKH())
+      .addInput(aliceUtxos[1], incorrectSigTemplate.unlockP2PKH())
+      .addOutput({ to: aliceAddress, amount: change });
+
+    await expect(transaction.send()).rejects.toThrow();
+  });
+
+  // TODO: Currently, P2PKH inputs are not evaluated at all
+  it.skip('should fail when invalidly spending from P2PKH and correctly from contract inputs', async () => {
+    const aliceUtxos = (await provider.getUtxos(aliceAddress)).filter(isNonTokenUtxo);
+    const p2pkhUtxos = (await p2pkhInstance.getUtxos()).filter(isNonTokenUtxo).sort(utxoComparator).reverse();
+    const incorrectSigTemplate = new SignatureTemplate(bobPriv);
+
+    expect(aliceUtxos.length).toBeGreaterThan(2);
+
+    const change = aliceUtxos[0].satoshis + aliceUtxos[1].satoshis - 1000n;
+
+    const transaction = new TransactionBuilder({ provider })
+      .addInput(aliceUtxos[0], incorrectSigTemplate.unlockP2PKH())
+      .addInput(p2pkhUtxos[0], p2pkhInstance.unlock.spend(carolPub, new SignatureTemplate(carolPriv)))
+      .addOutput({ to: aliceAddress, amount: change });
+
+    await expect(transaction.send()).rejects.toThrow();
   });
 });

packages/cashscript/test/debugging-old-artifacts.test.ts

@@ -0,0 +1,57 @@
+import { Contract, MockNetworkProvider, randomUtxo, SignatureTemplate, TransactionBuilder } from '../src/index.js';
+import { alicePkh, alicePriv, alicePub, bobPriv } from './fixture/vars.js';
+
+const artifact = {
+  contractName: 'P2PKH',
+  constructorInputs: [
+    { name: 'pkh', type: 'bytes20' },
+  ],
+  abi: [
+    {
+      name: 'spend',
+      inputs: [
+        { name: 'pk', type: 'pubkey' },
+        { name: 's', type: 'sig' },
+      ],
+    },
+  ],
+  bytecode: 'OP_OVER OP_HASH160 OP_EQUALVERIFY OP_CHECKSIG',
+  source: 'pragma cashscript ^0.7.0;\n\ncontract P2PKH(bytes20 pkh) {\n    // Require pk to match stored pkh and signature to match\n    function spend(pubkey pk, sig s) {\n        require(hash160(pk) == pkh);\n        require(checkSig(s, pk));\n    }\n}\n',
+  compiler: {
+    name: 'cashc',
+    version: '0.7.0',
+  },
+  updatedAt: '2025-08-05T09:04:50.388Z',
+};
+
+describe('Debugging tests - old artifacts', () => {
+  it('should succeed when passing the correct parameters', () => {
+    const provider = new MockNetworkProvider();
+    const contractTestLogs = new Contract(artifact, [alicePkh], { provider });
+    const contractUtxo = randomUtxo();
+    provider.addUtxo(contractTestLogs.address, contractUtxo);
+
+    const transaction = new TransactionBuilder({ provider })
+      .addInput(contractUtxo, contractTestLogs.unlock.spend(alicePub, new SignatureTemplate(alicePriv)))
+      .addOutput({ to: contractTestLogs.address, amount: 10000n });
+
+    console.warn(transaction.bitauthUri());
+
+    expect(() => transaction.debug()).not.toThrow();
+  });
+
+  it('should fail when passing the wrong parameters', () => {
+    const provider = new MockNetworkProvider();
+    const contractTestLogs = new Contract(artifact, [alicePkh], { provider });
+    const contractUtxo = randomUtxo();
+    provider.addUtxo(contractTestLogs.address, contractUtxo);
+
+    const transaction = new TransactionBuilder({ provider })
+      .addInput(contractUtxo, contractTestLogs.unlock.spend(alicePub, new SignatureTemplate(bobPriv)))
+      .addOutput({ to: contractTestLogs.address, amount: 10000n });
+
+    console.warn(transaction.bitauthUri());
+
+    expect(() => transaction.debug()).toThrow();
+  });
+});