feat: Add tenure-height to the /v2/info endpoint

[aldur]

Description

#5304 looked simple enough for me to give it a shot.

I have used this one for the definition of tenure-height.

Applicable issues

• Closes Add tenure-height to the /v2/info endpoint #5304.

Additional info (benefits, drawbacks, caveats)

NA

Checklist

• Test coverage for new or modified code paths
• Changelog is updated
• Required documentation changes (e.g., docs/rpc/openapi.yaml and rpc-endpoints.md for v2 endpoints, event-dispatcher.md for new events)
• New clarity functions have corresponding PR in clarity-benchmarking repo
• New integration test(s) added to bitcoin-tests.yml

[aldur]

Moved to draft as I investigate test failures.

[obycode]

Don't worry about these failing test cases. There is another PR open to fix the mock-mining test.

[aldur]

Thanks @obycode! If it's OK not to worry about mutants either (AFAICT they were missing before), this is ready for review.

[obycode]

Thanks @obycode! If it's OK not to worry about mutants either (AFAICT they were missing before), this is ready for review.

Yup, we are not enforcing the mutants checks yet and can expect failures on that.

[aldur]

Thanks @kantai! I rebased on latest develop, spread your caching suggestion on another file, and applied the other one on the parser.

[aldur]

Thanks @jcnelson, I took the approach you suggested and cached the coinbase_height. I have pushed the work so far, but the caching layer needs tests.

[jferrant]

I was asked to add a test here. Addressing code comments and adding test now.

[jcnelson]

Thanks for carrying this over the finish line @jferrant!

[blockstack-devops]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.