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.

Sign up for GitHub

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

accounts/keystore: faster tests #25827

Merged
merged 6 commits into from
Oct 12, 2022
Merged

accounts/keystore: faster tests #25827

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
9b6c643
accounts/keystore: speed up tests
holiman Sep 20, 2022
edce3bb
accounts/keystore: fixes to tests
holiman Sep 21, 2022
d39bf3b
accounts/keystore: add windows-support
holiman Sep 21, 2022
6e8d7c2
accounts/keystore: fix squashme
holiman Sep 21, 2022
d968f20
accounts/keystore: add keystore accessor to updating
holiman Oct 6, 2022
cf902af
accounts/keystore: add acessors to account cache, to improve tests
holiman Oct 6, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
accounts/keystore: fix squashme
  • Loading branch information
@holiman
holiman committed Oct 6, 2022
commit 6e8d7c20482993060d76d95bdef0fb9f99278b37
3 changes: 2 additions & 1 deletion accounts/keystore/account_cache_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,8 @@ var (

// waitWatcherStarts waits up to 1s for the keystore watcher to start.
func waitWatcherStart(ks *KeyStore) bool {
if ks.cache.watcher.enabled() {
// On systems where file watch is not supported, just return "ok".
if !ks.cache.watcher.enabled() {
return true
}
// The watcher should start, and then exit.
Expand Down