I plan to add simple tests soon...

#1 1565020.patch16.42 KBbleen


bleen’s picture

Status: Active » Needs review
16.42 KB

This patch creates tests for each field type and fixes a bunch of errors found by the tests (mostly around case sensitivity)

bleen’s picture

... the biggest change here BTW is that I removed the DISTINCT() call because it is not case-sensitive. I would rather err on the side of showing duplicates, then removing legitimate suggestions

bleen’s picture

Status: Needs review » Fixed

I committed this patch .. better to have tests that need some more review than to have no tests at all.

Status: Fixed » Closed (fixed)

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