I updated some of the tests. I noticed the term "messages" is used
throughout test_queues.py and this appears to be referring to tasks, yet
queues.py has no reference to the term "messages", so I replaced "messages"
with "tasks" where applicable in test_queues.py. Was this OK? I very well
could just be missing something entirely, but my assumption was that tasks
used to be referred to as messages at one point and that stuck around in
the tests a little too long.

Also, I feel like test_queues.py has a lot of code duplication that
probably should be DRYed up. Am I wrong in thinking this?


On Thu, May 9, 2013 at 9:11 AM, ericolson-wf <notifications@github.com>wrote:

> In furious/test_stubs/appengine/queues.py:
>
> >
> > -    return {'iterations': iterations, 'tasks_processed': tasks_processed}
> > +    return {'iterations': iterations, 'tasks_processed': total_tasks_processed}
>
> Yes, I didn't update the tests well enough. They would have broken if one
> of them included two non-zero iterations - so we need to make sure at least
> one test does that. Let me know if you want me to do any of this.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/WebFilings/furious/pull/80/files#r4153048>
> .
>