search

mahmoud / boltons

🔩 Like builtins, but boltons. 250+ constructs, recipes, and snippets which extend (and rely on nothing but) the Python standard library. Nothing like Michael Bolton.
https://boltons.readthedocs.org
Other
6.51k stars 353 forks source link

Test and fix for #348 #349

Closed mahmoud closed 11 months ago

mahmoud commented 11 months ago

The test repro'd #348, plus I checked .values() for good measure. It wasn't really a supported use case before, but ideally it should be; cacheutil are dict subtypes (for better or worse).

Before merging, I should probably add LRI tests for this, just to be safe. Also need to think a little harder about whether there are more invariant tests that need to be added for the linked list.

mahmoud commented 11 months ago

Hey, thanks for your patience! @markrwilliams, one of the original authors also took a look and gave it a :+1: offline. Just a good old fashioned latent bug. As far as changelogs, I usually hammer those out at release. :) Thanks!