Simulate execute on solana SDK [DPA-1418] #250
Conversation
🦋 Changeset detectedLatest commit: c99c960 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
| return fmt.Errorf("not implemented") | ||
| } | ||
|
|
||
| func (s *Simulator) SimulateOperation( |
There was a problem hiding this comment.
this was kind of what i was trying to avoid in SetRoot - another implementation for simulation, thats why i was experimenting the simulatable framework so we can easily make the existing implementation simulatable.
With the framework, it would be only a few lines of change for you here i believe? And the benefit is we get to test the most of the actual implementation of ExecuteOperation. (same code)
| @@ -153,6 +153,48 @@ func mockSolanaTransaction( | |||
| }).Once() | |||
| } | |||
|
|
|||
| func mockSolanaSimulation( | |||
There was a problem hiding this comment.
haha damn, looks like we overlap again lol
|
super seeded by #277 |
Adds the simulate operation functionality for the solana SDK
Summary
This pull request introduces a new
Simulatorclass for the Solana SDK, along with corresponding tests. The most important changes include the implementation of theSimulatorclass, a new test suite for theSimulator, and a helper function for mocking Solana transactions.New
Simulatorclass implementation:sdk/solana/simulator.go: Added a newSimulatorclass with methods for simulating operations and setting roots. This includes theNewSimulatorconstructor and methodsSimulateSetRootandSimulateOperation.Testing:
sdk/solana/simulator_test.go: Added tests for theSimulatorclass, includingTestNewSimulatorto test the constructor andTestSimulator_SimulateOperationto test theSimulateOperationmethod.Helper function for mocking:
sdk/solana/utils_test.go: Added a new helper functionmockSolanaSimulationto mock Solana transaction simulations in tests.