impl Testable for Option<bool>

BurntSushi / quickcheck

Automated property based testing for Rust (with shrinking).

The Unlicense

2.4k stars 149 forks source link

impl Testable for Option<bool> #234

Closed tobz1000 closed 5 years ago

tobz1000 commented 5 years ago

Fixes #105.

Personally I think this is a worthwhile change for the ergonomics it enables. However, there is probably some ambiguity around the test result when returning None. Please let me know how best to document this if you think it's necessary.

BurntSushi commented 5 years ago

I don't think my previous opinion has changed here. Sorry.