[step] Remove config indirection.

unittests/repo_test_util.py

Index: unittests/repo_test_util.py
diff --git a/unittests/repo_test_util.py b/unittests/repo_test_util.py
index bd51de0fd03b16fbd164e2eba638859095044faa..e441f1a0ba1facb3dffce165351617b68194ae01 100644
--- a/unittests/repo_test_util.py
+++ b/unittests/repo_test_util.py
@@ -307,7 +307,8 @@ class RepoTest(unittest.TestCase):
           '\n'.join([
             '',
             '  def %s(self):' % m_name,
-            '    return self.step_client.run_step(%r)' % {
+            '    return self.step_client.run_step('
+            '      self.step_client.StepConfig(**%r))' % {

[iannucci, 2017/06/12 19:37:57]

it's is this strictly necessary? This seems like a pretty unfortunate
abstraction violation... Can't this module just depend on recipe_engine/step?

[dnj, 2017/06/13 18:28:38]

Yeah I dunno why it's this way. Changed to call "step".

                 'name': m_name,
                 'cmd': m_cmd,
                 'ok_ret': [0],