Skip to content

test: initialize submodules before local suite#1

Open
newmattock wants to merge 1 commit into
BitgesellOfficial:masterfrom
newmattock:init-submodules-in-test-script
Open

test: initialize submodules before local suite#1
newmattock wants to merge 1 commit into
BitgesellOfficial:masterfrom
newmattock:init-submodules-in-test-script

Conversation

@newmattock
Copy link
Copy Markdown

Summary:

  • Initializes required Git submodules before running the local multichain test suite.
  • Documents the submodule setup step in the README test instructions.
  • Uses an EXIT trap so the Docker Compose stack is torn down if tests fail.

Related bounty or issue:

What changed:

  • test.sh now runs git submodule update --init --recursive when executed inside a Git checkout.
  • test.sh now exits on failures and keeps docker-compose down registered as cleanup.
  • README test setup now calls out the required submodule initialization for existing checkouts.

Validation:

  • bash -n test.sh
  • git diff --check

Notes:

  • I reproduced that a plain clone leaves chain/filecoin/filecoin-ffi and chain/solana/solana-ffi uninitialized, which makes the Go module replacement for github.com/renproject/solana-ffi resolve to a missing local go.mod before tests can run.
  • I did not run the full Docker-backed suite here.

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.

1 participant