Refactor code to resolve jobs in WAIT_ASYNC into JQManagedJob. Add tests...

....
Yes that makes great sense. Surprised I didn't do that already. It was supposed to follow the same pattern as run(). I believe the worker already does the same pattern you suggest.  Go for it. 

Sent from my iPhone

> On Oct 6, 2014, at 2:50 PM, Loren Norman <notifications@github.com> wrote:
> 
> In src/JQJobs/JQManagedJob.php:
> 
> > +            try {
> > +                $disposition = ErrorManager::wrap(array($mJob->getJob(), 'resolveWaitAsyncJob'), array($data), array($mJob, 'errorHandlerFailJob'));
> > +            } catch (Exception $e) {
> > +                if ($convertExceptionToFailure)
> > +                {
> > +                    $err = $e->getMessage();
> > +                    $disposition = self::STATUS_FAILED;
> > +                }
> > +                else
> > +                {
> > +                    throw $e;
> > +                }
> > +            }
> > +
> > +            // move job to disposition returned by resolveWaitAsyncJob
> > +            switch ($disposition) {
> Can we return the disposition so that the caller can check the outcome? Another trivial implementation thing that I'm uncertain about the design on.
> 
> —
> Reply to this email directly or view it on GitHub.