addressed @cyli comments

- removing side_effect comment and changed to return_value since all the tests are passing with
return_value and I dont know the reason for the comment
- cleaning up store at end of test
- checking number of times fetch_batch_of_events called in every test