Skip to content

Commit

Permalink
Merge pull request #309 from melexis/fix-linking-order
Browse files Browse the repository at this point in the history
Fix broken (re)linking from version 9.3.0
  • Loading branch information
Letme authored Oct 19, 2022
2 parents ab784f9 + d2e73fe commit ca35735
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 4 deletions.
15 changes: 14 additions & 1 deletion doc/unit_test.rst
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ Unit Tests
Unit Tests for mlx.traceability
-------------------------------

.. test item-link defined before item-relink and item definitions: item-link shall always be processed first
.. item-relink::
:remap: RQT-ATTRIBUTES_FAKE
:target: RQT-ATTRIBUTES
Expand Down Expand Up @@ -57,4 +59,15 @@ Unit Tests for mlx.traceability
:validates: RQT-MATRIX

.. item:: UTEST_ITEM_DIRECTIVE-MAKE_INTERNAL_ITEM_REF_SHOW_CAPTION
:validates: RQT-CAPTION

.. test item-relink defined after item-link and item definitions: item-link shall always be processed first
.. item-link::
:sources: nonexistent_item
:targets: RQT-CAPTION
:type: validates

.. item-relink::
:remap: nonexistent_item
:target: UTEST_ITEM_DIRECTIVE-MAKE_INTERNAL_ITEM_REF_SHOW_CAPTION
:type: validated_by
3 changes: 2 additions & 1 deletion mlx/traceable_item.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,8 @@ def all_relations(self):
targets.update(self.explicit_relations[relation])
if relation in self.implicit_relations:
targets.update(self.implicit_relations[relation])
yield relation, natsorted(targets)
if targets:
yield relation, natsorted(targets)

@staticmethod
def _add_relations(relations_of_self, relations_of_other):
Expand Down
4 changes: 2 additions & 2 deletions tools/doc-warnings.json
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
{
"sphinx": {
"enabled": true,
"min": 23,
"max": 23,
"min": 24,
"max": 24,
"exclude": [
"WARNING: the mlx.traceability extension is not safe for parallel reading",
"WARNING: doing serial read"
Expand Down

0 comments on commit ca35735

Please sign in to comment.