[lnst] Changing sequences to list of dictionaries

Jiří Pírko jirka at fedoraproject.org
Thu Aug 16 11:43:10 UTC 2012


commit 891dc15236146bcf3d506d435e2a5c1b67d3fc71
Author: Jan Tluka <jtluka at redhat.com>
Date:   Wed Aug 15 14:58:30 2012 +0200

    Changing sequences to list of dictionaries
    
    Signed-off-by: Jan Tluka <jtluka at redhat.com>

 NetTest/NetTestController.py |    2 +-
 NetTest/NetTestParse.py      |   13 +++++++------
 2 files changed, 8 insertions(+), 7 deletions(-)
---
diff --git a/NetTest/NetTestController.py b/NetTest/NetTestController.py
index 72233c5..05d7261 100644
--- a/NetTest/NetTestController.py
+++ b/NetTest/NetTestController.py
@@ -329,7 +329,7 @@ class NetTestController:
 
     def _run_command_sequence(self, sequence):
         seq_passed = True
-        for command in sequence:
+        for command in sequence["commands"]:
             logging.info("Executing command: [%s]", str_command(command))
             cmd_res = self._run_command(command)
             if self._res_serializer:
diff --git a/NetTest/NetTestParse.py b/NetTest/NetTestParse.py
index 4c46b0c..93e7782 100644
--- a/NetTest/NetTestParse.py
+++ b/NetTest/NetTestParse.py
@@ -323,8 +323,9 @@ class NetConfigParse(RecipeParser):
 class CommandSequenceParse(RecipeParser):
     def parse(self, node):
         sequences = self._recipe["sequences"]
-        sequences.append([])
+        sequences.append({})
         seq_num = len(sequences) - 1
+        sequences[seq_num]["commands"] = []
 
         self._seq_num = seq_num
         self._seq_node = node
@@ -342,7 +343,7 @@ class CommandSequenceParse(RecipeParser):
     def _check_sequence(self, sequence):
         err = False
         bg_ids = {}
-        for i, command in enumerate(sequence):
+        for i, command in enumerate(sequence["commands"]):
             machine_id = command["machine_id"]
             if not machine_id in bg_ids:
                 bg_ids[machine_id] = set()
@@ -388,8 +389,8 @@ class CommandParse(RecipeParser):
     def parse(self, node):
         recipe = self._recipe
         command = {}
-        recipe["sequences"][self._seq_num].append(command)
-        self._cmd_num = len(recipe["sequences"][self._seq_num]) - 1
+        recipe["sequences"][self._seq_num]["commands"].append(command)
+        self._cmd_num = len(recipe["sequences"][self._seq_num]["commands"]) - 1
 
         if self._has_attribute(node, "machine_id"):
             machine_id = self._get_attribute(node, "machine_id", int)
@@ -427,7 +428,7 @@ class CommandParse(RecipeParser):
     def _options(self, node, params):
         seq = self._seq_num
         cmd = self._cmd_num
-        self._recipe["sequences"][seq][cmd]["options"] = {}
+        self._recipe["sequences"][seq]["commands"][cmd]["options"] = {}
 
         scheme = {"option": self._option}
         self._process_child_nodes(node, scheme)
@@ -435,7 +436,7 @@ class CommandParse(RecipeParser):
     def _option(self, node, params):
         seq = self._seq_num
         cmd = self._cmd_num
-        options = self._recipe["sequences"][seq][cmd]["options"]
+        options = self._recipe["sequences"][seq]["commands"][cmd]["options"]
 
         name = self._get_attribute(node, "name")
         if not name in options:


More information about the LNST-developers mailing list