Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Fix handling of invoked ix accounts in program-test #16170

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

jstarry
Copy link
Contributor

@jstarry jstarry commented Mar 27, 2021

Problem

When testing programs in the native environment, processing invoked instructions with duplicate accounts is broken

@codecov
Copy link

codecov bot commented Mar 27, 2021

Codecov Report

Merging #16170 (6f10259) into master (06ac0fe) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

@@            Coverage Diff            @@
##           master   #16170     +/-   ##
=========================================
- Coverage    80.0%    80.0%   -0.1%     
=========================================
  Files         410      410             
  Lines      109109   109110      +1     
=========================================
- Hits        87377    87376      -1     
- Misses      21732    21734      +2     

@jstarry jstarry requested a review from mvines March 28, 2021 12:55
@jstarry jstarry merged commit 27ab415 into solana-labs:master Mar 29, 2021
mergify bot pushed a commit that referenced this pull request Mar 29, 2021
(cherry picked from commit 27ab415)

# Conflicts:
#	program-test/src/lib.rs
mergify bot pushed a commit that referenced this pull request Mar 29, 2021
mergify bot added a commit that referenced this pull request Mar 29, 2021
(cherry picked from commit 27ab415)

Co-authored-by: Justin Starry <justin@solana.com>
jstarry added a commit that referenced this pull request Mar 29, 2021
mergify bot added a commit that referenced this pull request Mar 29, 2021
(cherry picked from commit 27ab415)

Co-authored-by: Justin Starry <justin@solana.com>
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
@jstarry jstarry deleted the program-test-fix branch November 29, 2021 01:09
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.

2 participants