-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
neotest: Add contract signer support #3233
neotest: Add contract signer support #3233
Conversation
Just as an idea --- can we reuse a local client in |
I like this idea, but even with RPC-based network fee calculation in neotest I still think that it might be a good point to implement contract signer so that it would be easier to construct contract-based witnesses. |
No functional changes, just moved a part of emit.Array to a separate exported method. It may be useful for contract-based witness invocation scripts construction and not only. Ref. #3233 (comment). Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
No functional changes, just moved a part of emit.Array to a separate exported method. It may be useful for contract-based witness invocation scripts construction and not only. Ref. #3233 (comment). Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
No functional changes, just moved a part of emit.Array to a separate exported method. It may be useful for contract-based witness invocation scripts construction and not only. Ref. #3233 (comment). Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
No functional changes, just moved a part of emit.Array to a separate exported method. It may be useful for contract-based witness invocation scripts construction and not only. Ref. #3233 (comment). Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
d0dd4dc
to
2f960fc
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #3233 +/- ##
==========================================
- Coverage 85.34% 85.25% -0.09%
==========================================
Files 327 327
Lines 44084 44144 +60
==========================================
+ Hits 37622 37635 +13
- Misses 4983 5020 +37
- Partials 1479 1489 +10 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Dmitrii Stepanov <dima-stepan@yandex.ru>
2f960fc
to
7b53a0c
Compare
Refs nspcc-dev#3245 Refs nspcc-dev#3233 Signed-off-by: Evgenii Stratonikov <fyfyrchik@runbox.com>
Refs nspcc-dev#3245 Refs nspcc-dev#3233 Signed-off-by: Evgenii Stratonikov <fyfyrchik@runbox.com>
Refs nspcc-dev#3245 Refs nspcc-dev#3233 Signed-off-by: Evgenii Stratonikov <fyfyrchik@runbox.com>
Problem
When using another contract for verification in unit tests, the transaction cost is calculated incorrectly.
...
Solution
To calculate the transaction cost, a test neo-vm is started on which the verification is performed.
...