[ Jens Andersen ]
> > +@patch('os._exit')
> > +@patch('os.execvpe')
> > +@patch('os.environ.update')
> > +@patch('resource.setrlimit')
> > +@patch('proccer.agent._close_all_fds')
> > +@patch('os.dup2')
> > +@patch('os.setsid')

I don't know where this started, but I really, really want it to stop.

First off using patch as a decorator is butt-ugly; secondly patching stuff in
__builtins__ or os is generally a good sign that you're testing something the
wrong way; thirdly patching this many functions is also a godd sign that the
test is off.

-- 
Sune Kirkeby  | Some people, when faced with a problem, think, "I know, I'll
              | use binary." Now they have 10 problems.