Remove 'SaveOrderTask' and test state in class variables

Instead of saving task state in a class variable that is later
introspected by further test code just remove that concept (which
doesn't work in multiprocessing or worker engines which can not
have access those types of shared/globally available concepts due
to how they run) and use a specialized listener that can gather the same
information in a more decoupled manner (and it will work in multiprocessing
and worker engines correctly).

This allows our engine test cases to work in those engine types which
increases those engines test coverage (and future coverage and engine tests
that are added).

Fixes a bunch of occurrences of bug 1357117 as well that were removed
during this cleanup and adjustment process...

Change-Id: Ic9901de2902ac28ec255bef146be5846d18f9bfb

1 files changed, 1 insertions, 0 deletions

diff --git a/taskflow/listeners/base.py b/taskflow/listeners/base.py
index 42dc87e..1600dd3 100644
--- a/taskflow/listeners/base.py
+++ b/taskflow/listeners/base.py
@@ -153,6 +153,7 @@ class Listener(object):
 
     def __enter__(self):
         self.register()
+        return self
 
     def __exit__(self, type, value, tb):
         try: