aiokafka icon indicating copy to clipboard operation
aiokafka copied to clipboard

Replace `assert`'s in Consumer methods with proper validation

Open tvoinarovskyi opened this issue 8 years ago • 0 comments

Currently Consumer does something like:

assert all(map(lambda k: isinstance(k, TopicPartition), partitions))
  • Replace this with a proper value check, raise sensible errors like TypeError or ValueError.
  • Add respective simple validation tests

tvoinarovskyi avatar Apr 19 '17 17:04 tvoinarovskyi