> However we shouldn't need to do both of the following: > > step.setContext(context) > step.doExecute(context) > > Getting rid of one or the other would be good. set/getContext is used for testing purposes and for the MacroStepBuilder. I'm not yet fully sure about possible implication of moving step.doExecute(context) to step.doExecute(). cheers Mittie