-
Notifications
You must be signed in to change notification settings - Fork 14
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
Replace sleep
calls in tests with a direct cache refresh function
#1648
Conversation
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.
Unrelated: could we rename create_report_with_id()
to create_report_with_key()
? There are enough IDs in a report that I think its current name is unclear.
aggregator/src/cache.rs
Outdated
@@ -125,9 +128,25 @@ impl GlobalHpkeKeypairCache { | |||
let keypairs = self.keypairs.lock().unwrap(); | |||
keypairs.get(id).cloned() | |||
} | |||
|
|||
#[cfg(feature = "test-util")] | |||
pub async fn refresh(&self) -> Result<(), Error> { |
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.
I think we should extract the common body of this and the async block in new()
to a new private method.
#1649 for renaming function. |
Make rustdoc happy Errant file change Cleanups
bafcf8c
to
de52002
Compare
Co-authored-by: David Cook <dcook@divviup.org>
Follow up to #1640 (comment).