Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Add function to simplify tests and test recent txs #921

Merged
merged 2 commits into from
Aug 16, 2023

Conversation

juanbono
Copy link
Collaborator

TITLE

Description

This PR adds the test_tx function that encapsulates all the logic to run and get the TransactionExecutionInfo of a transaction using the RPC State Reader.

It also adds several new tests.

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.

@juanbono juanbono marked this pull request as ready for review August 15, 2023 15:13
Copy link
Contributor

@xqft xqft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@xqft xqft enabled auto-merge August 15, 2023 15:53
@xqft xqft added this pull request to the merge queue Aug 16, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #921 (eb70623) into main (bc9d4f2) will increase coverage by 0.01%.
The diff coverage is 96.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #921      +/-   ##
==========================================
+ Coverage   90.80%   90.82%   +0.01%     
==========================================
  Files          54       54              
  Lines       13556    13591      +35     
==========================================
+ Hits        12310    12344      +34     
- Misses       1246     1247       +1     
Files Changed Coverage Δ
src/transaction/invoke_function.rs 99.48% <94.11%> (-0.10%) ⬇️
src/lib.rs 98.75% <100.00%> (+0.01%) ⬆️
src/transaction/mod.rs 93.44% <100.00%> (+0.22%) ⬆️

Merged via the queue into main with commit 96148e7 Aug 16, 2023
@juanbono juanbono deleted the add_tests_improve_testing branch August 25, 2023 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants