-
Type: Task
-
Resolution: Won't Fix
-
Priority: Major - P3
-
None
-
Affects Version/s: None
-
Component/s: Test Coverage
-
Labels:None
Public methods in DB appear unused, which implies they need test coverage:
- resetOptions
And the deprecated methods:
- slaveOk
- addUser
While I could be sold on the unused deprecated methods (although I'm not sure that just because they're deprecated means we can't make sure they work the way we expect, at least until a deletion date is decided) the reset method definitely needs testing. A test would ensure we don't accidentally introduce regression bugs.