AccountsDb::load_account_with takes bool instead of fn

[apfitzge]

Problem

• should_put_in_read_cache_fn is never a function
  • it is also ALWAYS false outside of our tests, so it's possible we remove the parameter entirely, and just adjust our tests appropriately

Summary of Changes

• should_put_in_read_cache_fn -> should_put_in_read_cache as simple boolean

[brooksprumo]

Seems fine to me. Probably worthwhile to find if/when we even inspected the account in the fn version.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.8%. Comparing base (6fbbaf6) to head (92734dd).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master    #7294     +/-   ##
=========================================
- Coverage    82.8%    82.8%   -0.1%     
=========================================
  Files         801      801             
  Lines      363392   363393      +1     
=========================================
- Hits       300936   300927      -9     
- Misses      62456    62466     +10     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.