test: Properly set BIP34 height in CreateNewBlock_validity unit test #22277

The coinbase scriptSig in this unit test has several issues:

• The BIP34 height is not the "first item" as required (See https://github.com/bitcoin/bips/blob/master/bip-0034.mediawiki#specification)
• It uses the wrong encoding ( See https://github.com/bitcoin/bitcoin/blob/da69d9965a112c6421fce5649b5a18beb7513526/src/validation.cpp#L3250 )
• It uses the wrong height (off by one)

While BIP34 isn't currently enforced in this unit test, this should be fixed to avoid confusion and to promote self-consistency.

The change obviously requires new proof of work (BLOCKINFO).

Also change the block version from 1 to VERSIONBITS_TOP_BITS, because this test shouldn't care about the block version and bumping it is required for other changes.

<!--e57a25ab6845829454e8d69fc972939a-->

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

<!--174a7506f384e20aa4161008e828411d-->

Conflicts

No conflicts as of last run.

Code review ACK faa670d3862783017f5cd1491f37648e1875f19f

Checked that the block's coinbase scriptSig assignment matches now the BIP34 specification, verified upated proof of work (BLOCKINFO) by running the test. (By the way, being curious, how did you re-create it? I guess putting some grinding code directly into the loop? Could be interesting if ever other contributors want to change the test in a way that needs new PoW)

By the way, being curious, how did you re-create it? I guess putting some grinding code directly into the loop?

Jup, IIRC I just ground in the loop and used std::cout or something similar to create the new table.