Create Testengine executable #365
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shargon
reviewed
Sep 21, 2020
erikzhang
reviewed
Sep 22, 2020
…ne in Neo.Compiler.MSIL.UnitTests dependencies
shargon
reviewed
Oct 17, 2020
| @@ -8,7 +8,7 @@ | |||
| [assembly: InternalsVisibleTo("Neo.SmartContract.Framework.UnitTests")] | |||
| namespace Neo.Compiler.MSIL.UnitTests.Utils | |||
src/TestEngine/Program.cs
Outdated
| } | ||
| else | ||
| { | ||
| result = BuildJsonException("One or more arguments are missing"); |
src/TestEngine/Helper.cs
Outdated
| var json = new JObject(); | ||
|
|
||
| json["vm_state"] = testEngine.State.ToString(); | ||
| json["gas_consumed"] = (new BigDecimal(testEngine.GasConsumed, NativeContract.GAS.Decimals)).ToString(); |
There was a problem hiding this comment.
We agreed to use gasconsumed format in json properties
src/TestEngine/Helper.cs
Outdated
| return json; | ||
| } | ||
|
|
||
| public static JObject ToJson(this DataCache<StorageKey, StorageItem> storage) |
There was a problem hiding this comment.
We will need to initialize the storage, for fake balances for example.
src/TestEngine/Program.cs
Outdated
|
|
||
| if (!success) | ||
| { | ||
| // if something went wrong while reading the json, inserts null in this argument position |
There was a problem hiding this comment.
show log or break the execution, it could produce test mistakes.
2b50f6e to
ba806a1
Compare
Add json test support to the TestEngine
Include storage values in the TestEngine arguments
Include other smart contracts in the TestEngine arguments
Increase block height in the TestEngine
0d423f9 to
0c1303e
Compare
Make the TestEngine a global dotnet tool (similar to neoxp)
|
FYI, NGD Enterprise is working on a similar project that is integrated into the blockchain toolkit: https://github.com/ngdenterprise/neo-test/tree/devhawk/test-runner |
devhawk
suggested changes
Jul 8, 2022
There was a problem hiding this comment.
Why are we continuing to make updates to this PR instead of https://github.com/ngdenterprise/neo-test/tree/main/src/runner
fdc6141 to
81c6782
Compare
81c6782 to
c2ef0ae
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Created an executable for the TestEngine to allow other Neo compilers, even those that aren't written using C#, to run their tests with the same executing engine as the C# compiler.