+1 (not really a voter) I think iterator unit tests should use SortedMapIterator, not anything like a full accumulo stack, and I think MAC is far more suitable for integration tests because it actually runs the same code...it's impossible for an outsider to tell in which behaviors mock reflects actual accumulo and in which it does something totally different.
I do think MAC needs some help, but I think the process of excising mock from our own tests will flesh out what we need there better than anything else we could do. On Thu, Nov 14, 2013 at 9:20 PM, <dlmar...@comcast.net> wrote: > +1 > > > > *From:* Keith Turner [mailto:ke...@deenlo.com] > *Sent:* Thursday, November 14, 2013 3:42 PM > *To:* dev@accumulo.apache.org; u...@accumulo.apache.org > *Subject:* [VOTE] Deprecate mock in 1.6.0 > > > > Should we deprecate mock accumulo for 1.6.0? This was considered [1] for > 1.5.0. I started thinking about this because I never added conditional > writer to mock. > > > > [1] : https://issues.apache.org/jira/browse/ACCUMULO-878 >