Improve inline testing example

**Problem**

Previous example depended on the manager having been started via `Run`
or `RunWithContext`. This kicks off goroutines that result in data races
and flaky builds.

**Solution**

Manually add the `mgr.Pool` in inline test example to avoid requiring
a call to `mgr.Run*` and the resulting data races.

README.md

@@ -156,11 +156,28 @@ func Push(mgr worker.Manager, job *faktory.Job) error {
 }
 
 func syntheticPush(mgr worker.Manager, job *faktory.Job) error {
-	if mgr.IsRegistered(job.Type) {
-		return mgr.Dispatch(job)
+	mgr.mut.Lock()
+	defer mgr.mut.Unlock()
+
+	if !mgr.IsRegistered(job.Type) {
+		return fmt.Errorf("failed to execute job type %s inline, job not registered", job.Type)
+	}
+
+	// Manually defining the Pool avoids using mgr.Run/mgr.RunWithContext, which can trigger data races
+	if mgr.Pool == nil {
+		pool, err := faktory.NewPool(2)
+		if err != nil {
+			return fmt.Errorf("couldn't create Faktory connection pool in synthetic push: %w", err)
+		}
+		mgr.Pool = pool
 	}
 
-	return fmt.Errorf("inline job execution failed, unregistered job type %s", job.Type)
+	err := mgr.Dispatch(job)
+	if err != nil {
+		return errors.Wrap(err, fmt.Sprintf("job was immediately executed but failed. Job type %s, with args %+v", job.Type, job.Args))
+	}
+
+	return nil
 }
 
 func realPush(job *faktory.Job) error {