It just seemed like one of the cases that can occur. I guess the intention is  to ensure that it works in this case by calling process_events for first set only.

Mistyped from my iPhone

On Nov 1, 2013, at 6:28 PM, "cyli" <notifications@github.com<mailto:notifications@github.com>> wrote:


In otter/test/test_scheduler.py:

>
> -        d = self.scheduler_service.check_for_events(100)
> +    def test_events_batch_error(self):
> +        """
> +        When error occurs after first batch of events are processed


This docstring doesn't say what this is supposed to test. It just seems to test the case that if an error occurs during fetching, process will not be called? In which case is it important that this occurs during the second fetch, as opposed to the first?

—
Reply to this email directly or view it on GitHub<https://github.com/rackerlabs/otter/pull/366/files#r7386153>.