Update the test for cache functionality
test: update the test for cache functionality
Update the test for cache functionalitytestupdate test cache functionality
Committed by jmcdo29
commit message
test: update the test for cache functionality
In cache-manager@^4, the `ttl` option was a number that represented the
number of seconds. In cache-manager@^5 the `ttl` is now a number that
represents the number of milliseconds. This difference caused our tests
to be flaky. I've also updated the default in the case of v5 to be 5000
instead of just 5 so that we get the same functionality for the default
value.Editor assessment
The subject — update the test for cache functionality — is generic and would normally sink the message, but the body redeems it thoroughly: it explains that cache-manager v4 measured ttl in seconds while v5 measures milliseconds, that this mismatch caused flaky tests, and that the default moved to 5000 to preserve behavior. That is a real lesson about an upstream API shift, not restatement. The test type is correct, and with no single module involved, omitting a scope is fair. No footers. Hygiene is clean, with wrapped lines and no pasted content.