[infinispan-dev] Adding tests for new cache mode

[Dan Berindei]

On Tue, May 17, 2016 at 3:26 PM, Tristan Tarrant <ttarrant at redhat.com> wrote:
> On 17/05/2016 14:10, Radim Vansa wrote:
>> Hi,
>>
>> I've decided to start working on Scattered Cache [1][2] POC. I'd like to
>> use most of the tests for distributed mode, but just extending
>> DistXxxTest with ScatteredXxxTest and overriding getCacheMode() seems
>> quite inelegant, though this is a common practice for repl/dist tests. I
>> had similar problem with Simple Cache, but I didn't need as many tests
>> for that.
>>
>> @Parameters are not used as much in our testsuite - is there any reason
> Not using @Parameters is a mistake, IMHO, so if you're willing to
> convert the relevant ones, that would be lovely.
>

+1 to switch to @Parameters as many tests as you want!

Dan