Improve the new DefaultProcessorTest

This patch changes the data provider to send the GUIDs of the items we expect to be imported instead of only the count of expected items. The test then adds another assertion to check that items with those GUIDs were created. The existing assertion continues to ensure that the total number of items matches the number expected.

I also moved the processor instantiation to the setUp() method. I don't remember why I decided not to do this when I initially wrote the test. For some reason I thought it was a bad idea. But doing this will reduce code duplication when other test functions are eventually added.

Nice cleanup

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