Test coverage

I no longer see MemcacheBackendUnitTest in 8.x version. Did we lose test coverage over the years?

MemcacheBackendUnitTest was moved/renamed to the MemcacheBackendTest in 2018 by Damian.

https://git.drupalcode.org/project/memcache/-/commit/f30543ba2d991e5fd06...

There isn't any issue associated with it, but I'm guessing it had to do with the bootstrapping of memcache. Interestingly its still referencing GenericCacheBackendUnitTestBase -- so I'm not sure why it was moved out of a unit test.

The existing failures have come up in a couple of issues, 🐛 Transaction support for cache (tags) invalidation Needs review and 📌 Add optional DDEV dev environment Needs review

Debugging I found that a lot of times where the get is called the cache item created and last bin deleted timestamps were the same - so the data is returned from memcache but when timeIsGreaterThanBinDeletionTime is called in the memcache backed, the timestamps are equal so it returns false and doesn't return anything from the cache backend service.

I tried in https://git.drupalcode.org/project/memcache/-/merge_requests/22/diffs?co... to reduce the tolerance to 0 and that greatly reduced the number of test failures, but not all of them - so I suspect there are additional issues, or the tolerance of 0 is still somehow flakey.

If think it makes sense to tackle the existing failures in this issue first so just cross posting here.

Thanks ericgsmith for the MR! I've been playing around with the test failures as well to no avail. Going to merge in your MR for now but keep this issue open and NW due to the remaining failures...

japerry → credited Wim Leers → .

Tests are passing and reliably both locally and on gitlabci! Thanks Wim for the rubber ducking with me!

Automatically closed - issue fixed for 2 weeks with no activity.