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

Add RPCState::get_transaction_trace #909

Merged
merged 12 commits into from
Aug 10, 2023
Merged

Add RPCState::get_transaction_trace #909

merged 12 commits into from
Aug 10, 2023

Conversation

matias-gonz
Copy link
Contributor

@matias-gonz matias-gonz commented Aug 9, 2023

Closes #903
Closes #904

@matias-gonz matias-gonz self-assigned this Aug 9, 2023
@matias-gonz matias-gonz changed the title Add RPCState::get_transaction_receipt Add RPCState::get_transaction_trace Aug 9, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 10, 2023

Codecov Report

Merging #909 (2235040) into main (f05da40) will decrease coverage by 0.27%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #909      +/-   ##
==========================================
- Coverage   91.11%   90.84%   -0.27%     
==========================================
  Files          54       54              
  Lines       13509    13549      +40     
==========================================
  Hits        12309    12309              
- Misses       1200     1240      +40     
Files Changed Coverage Δ
src/execution/mod.rs 93.64% <0.00%> (-4.01%) ⬇️
src/utils.rs 92.76% <0.00%> (-2.30%) ⬇️

@matias-gonz matias-gonz marked this pull request as ready for review August 10, 2023 19:41

// https://alpha4-2.starknet.io/feeder_gateway/get_transaction_trace?transactionHash=0x019feb888a2d53ffddb7a1750264640afab8e9c23119e648b5259f1b5e7d51bc
#[test]
fn test_get_transaction_receipt() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should be get_transaction_trace right?

@matias-gonz matias-gonz added this pull request to the merge queue Aug 10, 2023
Merged via the queue into main with commit 7abb894 Aug 10, 2023
@juanbono juanbono deleted the get_tx_receipt branch August 25, 2023 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants