-
Notifications
You must be signed in to change notification settings - Fork 210
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: use larger amounts in tests #3872
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #3872 +/- ##
=======================================
Coverage 57.17% 57.18%
=======================================
Files 193 193
Lines 42578 42578
=======================================
+ Hits 24346 24348 +2
+ Misses 18232 18230 -2 ☔ View full report in Codecov by Sentry. |
@@ -87,7 +87,7 @@ pub async fn latency_tests(dev_fed: DevFed) -> Result<()> { | |||
let iterations = 30; | |||
let mut reissues = Vec::with_capacity(iterations); | |||
for _ in 0..iterations { | |||
let notes = cmd!(fed, "spend", "50000").out_json().await?["notes"] | |||
let notes = cmd!(fed, "spend", "1000000").out_json().await?["notes"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still isn't a whole lot 😆 (1k sat)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has so many zeros already... :D
Maybe we need ksats(100).to_string()
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marginally better, but we should have a "reissue 10 BTC when the wallet had only 10 BTC in total" (spending everything is the worst case, up to 4 notes per denomination) test case to give us more confidence.
@@ -373,8 +373,8 @@ async fn cli_tests(dev_fed: DevFed) -> Result<()> { | |||
info!("Testing Client"); | |||
// ## reissue e-cash | |||
info!("Testing reissuing e-cash"); | |||
const CLIENT_START_AMOUNT: u64 = 420000; | |||
const CLIENT_SPEND_AMOUNT: u64 = 42; | |||
const CLIENT_START_AMOUNT: u64 = 5_000_000_000; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only change that makes the test test any amount of significance.
Closes #3866