Use TypeError instead of ValueError and give a nicer error mesasge

when someone calls Cache.invalidate with the wrong type.
This commit is contained in:
Mark Haines 2015-08-10 14:07:08 +01:00
parent 8c3a62b5c7
commit 559c51debc

View File

@ -116,7 +116,9 @@ class Cache(object):
def invalidate(self, key): def invalidate(self, key):
self.check_thread() self.check_thread()
if not isinstance(key, tuple): if not isinstance(key, tuple):
raise ValueError("keyargs must be a tuple.") raise TypeError(
"The cache key must be a tuple not %r" % (type(key),)
)
# Increment the sequence number so that any SELECT statements that # Increment the sequence number so that any SELECT statements that
# raced with the INSERT don't update the cache (SYN-369) # raced with the INSERT don't update the cache (SYN-369)