Skip to content

Commit

Permalink
fix(solana): fix broken smoke test
Browse files Browse the repository at this point in the history
The smoke test was expecting the mcm.so file due to the config in `blockchain_a.solana_programs` , since we dont provide any program to deploy in the smoke test, we should not define that config.

I have also updated the default port for solana to be 8999 as that is the default port for solana test validator and it is what users would expect.
  • Loading branch information
graham-chainlink committed Jan 10, 2025
1 parent cdc37af commit 63751d3
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
2 changes: 1 addition & 1 deletion framework/components/blockchain/solana.go
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ func defaultSolana(in *Input) {
in.Image = "solanalabs/solana:v1.18.26"
}
if in.Port == "" {
in.Port = "8545"
in.Port = "8999"
}
}

Expand Down
4 changes: 0 additions & 4 deletions framework/examples/myproject/smoke_solana.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,3 @@
type = "solana"
public_key = "9n1pyVGGo6V4mpiSDMVay5As9NurEkY283wwRk1Kto2C"
contracts_dir = "."
image = "solanalabs/solana:v1.18.26"

[blockchain_a.solana_programs]
mcm = "6UmMZr5MEqiKWD5jqTJd1WCR5kT8oZuFYBLJFi1o6GQX"

0 comments on commit 63751d3

Please sign in to comment.