Skip to content

Regenerate test lcm for cap 71#5293

Open
sisuresh wants to merge 3 commits into
stellar:masterfrom
sisuresh:cap71-lcm
Open

Regenerate test lcm for cap 71#5293
sisuresh wants to merge 3 commits into
stellar:masterfrom
sisuresh:cap71-lcm

Conversation

@sisuresh
Copy link
Copy Markdown
Contributor

Description

Regenerate test lcm for cap 71 and add copilot instructions

Checklist

  • Reviewed the contributing document
  • Rebased on top of master (no merge commits)
  • Ran clang-format v8.0.0 (via make format or the Visual Studio extension)
  • Compiles
  • Ran all tests
  • If change impacts performance, include supporting evidence per the performance document

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates captured LedgerCloseMeta (LCM) test fixtures for CAP-71 by registering additional InvokeHostFunctionTests LCM captures, and adds contributor guidance for regenerating test-lcm/ data when protocol changes affect tests.

Changes:

  • Add new hash→test-name entries to test-lcm/InvokeHostFunctionTests/index.json to reflect regenerated/added captured LCM outputs.
  • Document the required workflow for regenerating and committing test-lcm/ golden data after protocol (including vnext) changes.

Reviewed changes

Copilot reviewed 2 out of 102 changed files in this pull request and generated 1 comment.

File Description
test-lcm/InvokeHostFunctionTests/index.json Adds new LCM capture index entries for additional InvokeHostFunctionTests cases.
.github/copilot-instructions.md Adds instructions for regenerating captured LedgerCloseMeta golden data (including vnext configuration notes).

Comment thread .github/copilot-instructions.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants