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

chore: fix broken stake program live tests #25107

Merged
merged 2 commits into from
May 10, 2022

Conversation

jstarry
Copy link
Member

@jstarry jstarry commented May 10, 2022

Problem

  • The stake program live tests rely on getting a new finalized blockhash whenever a duplicate transaction is sent, but it takes awhile for a new validator to start finalizing blocks.
  • Stake tests don't use the min delegation

Summary of Changes

  • Update stake program test to not use a dup transaction
  • Ensure min delegation is used

Fixes #

@jstarry jstarry added the automerge Merge this Pull Request automatically once CI passes label May 10, 2022
@codecov
Copy link

codecov bot commented May 10, 2022

Codecov Report

Merging #25107 (304fcb1) into master (69a0ff9) will decrease coverage by 6.8%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #25107       +/-   ##
===========================================
- Coverage    82.0%    75.1%     -6.9%     
===========================================
  Files         598       38      -560     
  Lines      165882     2244   -163638     
  Branches        0      323      +323     
===========================================
- Hits       136125     1687   -134438     
+ Misses      29757      444    -29313     
- Partials        0      113      +113     

@mergify mergify bot removed the automerge Merge this Pull Request automatically once CI passes label May 10, 2022
@mergify
Copy link
Contributor

mergify bot commented May 10, 2022

automerge label removed due to a CI failure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant