Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(278)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: recipe_engine/step_runner.py

Issue 2387763003: Add initial postprocess unit test thingy. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« recipe_engine/simulation_test.py ('K') | « recipe_engine/simulation_test.py ('k') | recipe_engine/third_party/astunparse/LICENSE » ('j') | recipe_engine/third_party/astunparse/README.google » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: recipe_engine/step_runner.py
diff --git a/recipe_engine/step_runner.py b/recipe_engine/step_runner.py
index dc3589dbe3249f0084bf46161a683616094774f1..c4d11bdc969deb2c77a17e60613fec43e3704c1b 100644
--- a/recipe_engine/step_runner.py
+++ b/recipe_engine/step_runner.py
@@ -471,8 +471,9 @@ class SimulationStepRunner(StepRunner):
@property
def steps_ran(self):
- return [self._rendered_step_to_dict(rs)
- for rs in self._step_history.itervalues()]
+ return collections.OrderedDict(
+ (name, self._rendered_step_to_dict(rs))
dnj 2016/10/04 16:42:35 Can we make this a namedtuple?
iannucci 2016/10/06 22:47:11 are you serious?
+ for name, rs in self._step_history.iteritems())
# Placeholders associated with a rendered step.
« recipe_engine/simulation_test.py ('K') | « recipe_engine/simulation_test.py ('k') | recipe_engine/third_party/astunparse/LICENSE » ('j') | recipe_engine/third_party/astunparse/README.google » ('J')

Powered by Google App Engine
This is Rietveld 408576698