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

use intended lamports_per_signature for fee_structure #34845

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

tao-stones
Copy link
Contributor

Problem

FeeStructure used in tests should be initialized with intended lamports_per_signature. Using FeeStructure::default() only works if lamports_per_signature == 0 or 5_000.

Summary of Changes

  • Initialize FeeStructure with intended lamports_per_signature for tests

Fixes #

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8f9d915) 81.8% compared to head (2bbfa3e) 81.7%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #34845     +/-   ##
=========================================
- Coverage    81.8%    81.7%   -0.1%     
=========================================
  Files         825      825             
  Lines      223269   223270      +1     
=========================================
- Hits       182635   182623     -12     
- Misses      40634    40647     +13     

@tao-stones tao-stones requested a review from LucasSte January 18, 2024 23:54
@tao-stones tao-stones merged commit 32a3de0 into solana-labs:master Jan 19, 2024
35 checks passed
@tao-stones tao-stones deleted the harden-accounts-tests branch January 19, 2024 15:18
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