test: Fix P2PK script test #34380

1. 
   billymcbip commented at 12:35 pm on January 22, 2026: contributor

   I found another script_tests case that isn’t behaving the way it was meant to. It’s a P2PK spend where we add an OP_NOP8 to the scriptSig to make it non-push-only. The test should check that scriptSig.IsPushOnly() is only enforced in P2SH mode when the scriptPubKey actually matches the P2SH pattern. To test this, we need to turn on the P2SH flag.
2. test: Fix P2PK script test c9ce1c7c4a
3. DrahtBot added the label Tests on Jan 22, 2026
4. 
   DrahtBot commented at 12:35 pm on January 22, 2026: contributor

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Code Coverage & Benchmarks

   For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34380.

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   ACK	sipa, darosior

   If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

5. billymcbip referenced this in commit 269281b7cc on Jan 22, 2026
6. 
   sipa commented at 2:23 pm on January 22, 2026: member

   ACK c9ce1c7c4a12b54ada7d48f100ec3a141ae99f86
7. darosior approved
8. 
   darosior commented at 2:44 pm on January 22, 2026: member

   utACK c9ce1c7c4a12b54ada7d48f100ec3a141ae99f86

   Good catch!

9. fanquake merged this on Jan 23, 2026
10. fanquake closed this on Jan 23, 2026

    ---

11. billymcbip deleted the branch on Jan 23, 2026

Contributors
billymcbip DrahtBot sipa darosior

Labels
Tests