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
Merge branch 'auto_377_there_otter_be_an_otterclock' into scheduler_locking