Skip to content
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

Feat: extend ExecutionProfile with logs for tests #910

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

mrLSD
Copy link
Member

@mrLSD mrLSD commented Feb 29, 2024

Description

Extended ExecutionProfile with logs for tests.

Changes related only to engine-tests and do not affect main contract.

Motivation

To profile NEAR contract execution flow it's convenient to debug.

@mrLSD mrLSD self-assigned this Feb 29, 2024
@mrLSD mrLSD added C-enhancement Category: New feature or request A-testing Area: If something has added tests, or changed them. labels Feb 29, 2024
@mrLSD mrLSD requested a review from aleksuss February 29, 2024 08:19
@aleksuss aleksuss added this pull request to the merge queue Feb 29, 2024
Merged via the queue into develop with commit 9d7de27 Feb 29, 2024
24 checks passed
@aleksuss aleksuss deleted the feat/extend-profiler-for-tests branch February 29, 2024 09:11
aleksuss pushed a commit that referenced this pull request Mar 27, 2024
## Description

Extended `ExecutionProfile` with logs for tests. 

Changes related only to `engine-tests` and do not affect main contract.

### Motivation

To profile NEAR contract execution flow it's convenient to debug.
@aleksuss aleksuss mentioned this pull request Mar 27, 2024
aleksuss added a commit that referenced this pull request Mar 27, 2024
### Additions

- Added a new view transaction `ft_balances_of` for getting balances for
multiple accounts by [@karim-en]. ([#905])

### Changes

- The `ft_resolve_transfer` callback doesn't require running the
contract to finish the `ft_transfer_call` correctly
  by [@aleksuss]. ([#906])
- Borsh has been bumped to 1.3 what allows to get rid of additional
feature `borsh-compat` by [@aleksuss]. ([#907])
- The `ExecutionProfile` has been extended with logs for tests by
[@mrLSD]. ([#910])
- The interface of the engine standalone storage has been extended with
a couple of methods for allowing set/get
  arbitrary data outside the crate by [@aleksuss]. ([#913])

### Fixes

- Minor improvements and fixes by [@raventid]. ([#916])

[#905]: #905
[#906]: #906
[#907]: #907
[#910]: #910
[#913]: #913
[#916]: #916

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Karim <karim@aurora.dev>
Co-authored-by: Michael Birch <michael.birch@aurora.dev>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evgeny Ukhanov <evgeny@aurora.dev>
Co-authored-by: Julian Pokrovsky <raventid@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testing Area: If something has added tests, or changed them. C-enhancement Category: New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants