Disable cache expiration if timeout is -1 instead of None

[lpsinger]

There does not appear to be a way to set the value of cache_timeout to None in a .ini file because there is no value that will be decoded to None and will also validate as an integer.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.22%. Comparing base (fc6a0dd) to head (65efd63).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3579   +/-   ##
=======================================
  Coverage   73.22%   73.22%           
=======================================
  Files         226      226           
  Lines       21000    21000           
=======================================
  Hits        15378    15378           
  Misses       5622     5622           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

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

[bsipocz]

Thank you @lpsinger, this makes all sense. It also makes me wonder if we should have some actual testing of the config cfg file rather than setting the values directly in the tests.

[lpsinger]

Thank you @lpsinger, this makes all sense. It also makes me wonder if we should have some actual testing of the config cfg file rather than setting the values directly in the tests.

That's a good idea. I suggest that should be a separate PR.

[bsipocz]

That's a good idea. I suggest that should be a separate PR.

Certainly! This still needs a changelog entry before merge, but I can do it now.