diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2021-03-12 10:24:41 +0100 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2021-03-17 18:35:11 +0100 |
| commit | e7d6e8d7826769503253033e6e2fe6d3693515b4 (patch) | |
| tree | ad329b48c14a490a5856ca55edba8c524f3c737b | |
| parent | validation: Add default for transition map entries (diff) | |
| download | rtems-central-e7d6e8d7826769503253033e6e2fe6d3693515b4.tar.bz2 | |
validation: Shorten variable name
| -rw-r--r-- | rtemsspec/validation.py | 27 |
1 files changed, 11 insertions, 16 deletions
diff --git a/rtemsspec/validation.py b/rtemsspec/validation.py index c38cfadf..683656fb 100644 --- a/rtemsspec/validation.py +++ b/rtemsspec/validation.py @@ -569,8 +569,7 @@ class _ActionRequirementTestItem(_TestItem): def _add_transitions(self, trans_index: int, condition_index: int, map_index: int, transition: Dict[str, Any], - transition_map: _TransitionMap, - pre_cond_not_applicables: List[str], + transition_map: _TransitionMap, info: List[str], post_cond: Tuple[int, ...]) -> None: # pylint: disable=too-many-arguments # pylint: disable=too-many-locals @@ -582,11 +581,11 @@ class _ActionRequirementTestItem(_TestItem): if isinstance(states, str): assert states in ["all", "N/A"] for index in range(state_count): - self._add_transitions( - trans_index, condition_index + 1, map_index + index, - transition, transition_map, - pre_cond_not_applicables + [str(int(states == "N/A"))], - post_cond) + self._add_transitions(trans_index, condition_index + 1, + map_index + index, transition, + transition_map, + info + [str(int(states == "N/A"))], + post_cond) else: for state in states: try: @@ -600,8 +599,7 @@ class _ActionRequirementTestItem(_TestItem): raise ValueError(msg) from err self._add_transitions(trans_index, condition_index + 1, map_index + index, transition, - transition_map, - pre_cond_not_applicables + ["0"], + transition_map, info + ["0"], post_cond) else: enabled_by = enabled_by_to_exp(transition["enabled-by"], @@ -614,20 +612,17 @@ class _ActionRequirementTestItem(_TestItem): "defined by transition map entry " f"{transition_map[map_index][0].map_entry_index}") transition_map[map_index].append( - _Transition(enabled_by, post_cond, - " " + ", ".join(pre_cond_not_applicables), + _Transition(enabled_by, post_cond, " " + ", ".join(info), trans_index)) def _add_default(self, trans_index: int, transition_map: _TransitionMap, - pre_cond_not_applicables: List[str], - post_cond: Tuple[int, ...]) -> None: + info: List[str], post_cond: Tuple[int, ...]) -> None: for transition in transition_map: if not transition: transition.append( _Transition( - "1", post_cond, - " " + ", ".join(pre_cond_not_applicables + - ["0"] * self._pre_condition_count), + "1", post_cond, " " + + ", ".join(info + ["0"] * self._pre_condition_count), trans_index)) def _get_transition_map(self) -> _TransitionMap: |