summaryrefslogtreecommitdiff
path: root/tests/pyreverse
diff options
context:
space:
mode:
authorAlvaro Frias <alvaro.frias@eclypsium.com>2022-11-27 12:18:43 -0300
committerGitHub <noreply@github.com>2022-11-27 16:18:43 +0100
commitb950567b872c7177f976066e9b8982f08e62eb6c (patch)
treeee0a1c30ded4eb4ce2267adcc52b9dcfb9865039 /tests/pyreverse
parent978d1ab95603fa337e686aac8956366556c23080 (diff)
downloadpylint-git-b950567b872c7177f976066e9b8982f08e62eb6c.tar.gz
Feature: distinct Composition and Aggregation arrows (#7836)
* Update Linker to add aggregations_type and associations_type to nodes Update logic of nodes to check what kind of relationship does nodes have (association, aggregation) Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update ClassDiagram's extrac_relationship method to add aggregations links Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update DiagramWriter to generate AGGREGATION edges Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update printers to show aggregations Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update tests with changes on aggregations Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update Linker to add aggregations_type and associations_type to nodes Update logic of nodes to check what kind of relationship does nodes have (association, aggregation) Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update ClassDiagram's extrac_relationship method to add aggregations links Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update DiagramWriter to generate AGGREGATION edges Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update tests with changes on aggregations Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply pylint pre-commit correction Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Apply mypy corrections Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Add towrncrier fragment Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update towncrier fragment Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update towncrier fragment Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update doc/whatsnew/fragments/6543.feature Co-authored-by: Andreas Finkler <3929834+DudeNr33@users.noreply.github.com> * Add documentation Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * fix typo Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Add type hints Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * Update pylint/pyreverse/diagrams.py Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update type hints Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update fragment * Update fragment Signed-off-by: Alvaro Frias Garay <alvaro.frias@eclypsium.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Andreas Finkler <3929834+DudeNr33@users.noreply.github.com> Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
Diffstat (limited to 'tests/pyreverse')
-rw-r--r--tests/pyreverse/data/classes_No_Name.dot2
-rw-r--r--tests/pyreverse/data/classes_No_Name.html2
-rw-r--r--tests/pyreverse/data/classes_No_Name.mmd2
-rw-r--r--tests/pyreverse/data/classes_No_Name.puml2
-rw-r--r--tests/pyreverse/data/classes_colorized.dot2
-rw-r--r--tests/pyreverse/data/classes_colorized.puml2
-rw-r--r--tests/pyreverse/test_diadefs.py2
7 files changed, 7 insertions, 7 deletions
diff --git a/tests/pyreverse/data/classes_No_Name.dot b/tests/pyreverse/data/classes_No_Name.dot
index 01e3619d6..a598ab6d9 100644
--- a/tests/pyreverse/data/classes_No_Name.dot
+++ b/tests/pyreverse/data/classes_No_Name.dot
@@ -13,5 +13,5 @@ charset="utf-8"
"data.clientmodule_test.Ancestor" -> "data.suppliermodule_test.Interface" [arrowhead="empty", arrowtail="node", style="dashed"];
"data.suppliermodule_test.DoNothing" -> "data.clientmodule_test.Ancestor" [arrowhead="diamond", arrowtail="none", fontcolor="green", label="cls_member", style="solid"];
"data.suppliermodule_test.DoNothing" -> "data.clientmodule_test.Specialization" [arrowhead="diamond", arrowtail="none", fontcolor="green", label="relation", style="solid"];
-"data.suppliermodule_test.DoNothing2" -> "data.clientmodule_test.Specialization" [arrowhead="diamond", arrowtail="none", fontcolor="green", label="relation2", style="solid"];
+"data.suppliermodule_test.DoNothing2" -> "data.clientmodule_test.Specialization" [arrowhead="odiamond", arrowtail="none", fontcolor="green", label="relation2", style="solid"];
}
diff --git a/tests/pyreverse/data/classes_No_Name.html b/tests/pyreverse/data/classes_No_Name.html
index 8ec309d33..602f2e3b7 100644
--- a/tests/pyreverse/data/classes_No_Name.html
+++ b/tests/pyreverse/data/classes_No_Name.html
@@ -43,7 +43,7 @@
Ancestor ..|> Interface
DoNothing --* Ancestor : cls_member
DoNothing --* Specialization : relation
- DoNothing2 --* Specialization : relation2
+ DoNothing2 --o Specialization : relation2
</div>
</body>
diff --git a/tests/pyreverse/data/classes_No_Name.mmd b/tests/pyreverse/data/classes_No_Name.mmd
index 9b8397206..1db88b2ae 100644
--- a/tests/pyreverse/data/classes_No_Name.mmd
+++ b/tests/pyreverse/data/classes_No_Name.mmd
@@ -38,4 +38,4 @@ classDiagram
Ancestor ..|> Interface
DoNothing --* Ancestor : cls_member
DoNothing --* Specialization : relation
- DoNothing2 --* Specialization : relation2
+ DoNothing2 --o Specialization : relation2
diff --git a/tests/pyreverse/data/classes_No_Name.puml b/tests/pyreverse/data/classes_No_Name.puml
index ee3fa2bd4..837e6865c 100644
--- a/tests/pyreverse/data/classes_No_Name.puml
+++ b/tests/pyreverse/data/classes_No_Name.puml
@@ -39,5 +39,5 @@ data.clientmodule_test.Specialization --|> data.clientmodule_test.Ancestor
data.clientmodule_test.Ancestor ..|> data.suppliermodule_test.Interface
data.suppliermodule_test.DoNothing --* data.clientmodule_test.Ancestor : cls_member
data.suppliermodule_test.DoNothing --* data.clientmodule_test.Specialization : relation
-data.suppliermodule_test.DoNothing2 --* data.clientmodule_test.Specialization : relation2
+data.suppliermodule_test.DoNothing2 --o data.clientmodule_test.Specialization : relation2
@enduml
diff --git a/tests/pyreverse/data/classes_colorized.dot b/tests/pyreverse/data/classes_colorized.dot
index 09f72b782..4ff12a819 100644
--- a/tests/pyreverse/data/classes_colorized.dot
+++ b/tests/pyreverse/data/classes_colorized.dot
@@ -13,5 +13,5 @@ charset="utf-8"
"data.clientmodule_test.Ancestor" -> "data.suppliermodule_test.Interface" [arrowhead="empty", arrowtail="node", style="dashed"];
"data.suppliermodule_test.DoNothing" -> "data.clientmodule_test.Ancestor" [arrowhead="diamond", arrowtail="none", fontcolor="green", label="cls_member", style="solid"];
"data.suppliermodule_test.DoNothing" -> "data.clientmodule_test.Specialization" [arrowhead="diamond", arrowtail="none", fontcolor="green", label="relation", style="solid"];
-"data.suppliermodule_test.DoNothing2" -> "data.clientmodule_test.Specialization" [arrowhead="diamond", arrowtail="none", fontcolor="green", label="relation2", style="solid"];
+"data.suppliermodule_test.DoNothing2" -> "data.clientmodule_test.Specialization" [arrowhead="odiamond", arrowtail="none", fontcolor="green", label="relation2", style="solid"];
}
diff --git a/tests/pyreverse/data/classes_colorized.puml b/tests/pyreverse/data/classes_colorized.puml
index 2b450207a..7398ee60f 100644
--- a/tests/pyreverse/data/classes_colorized.puml
+++ b/tests/pyreverse/data/classes_colorized.puml
@@ -39,5 +39,5 @@ data.clientmodule_test.Specialization --|> data.clientmodule_test.Ancestor
data.clientmodule_test.Ancestor ..|> data.suppliermodule_test.Interface
data.suppliermodule_test.DoNothing --* data.clientmodule_test.Ancestor : cls_member
data.suppliermodule_test.DoNothing --* data.clientmodule_test.Specialization : relation
-data.suppliermodule_test.DoNothing2 --* data.clientmodule_test.Specialization : relation2
+data.suppliermodule_test.DoNothing2 --o data.clientmodule_test.Specialization : relation2
@enduml
diff --git a/tests/pyreverse/test_diadefs.py b/tests/pyreverse/test_diadefs.py
index 9a550c5db..da16eea33 100644
--- a/tests/pyreverse/test_diadefs.py
+++ b/tests/pyreverse/test_diadefs.py
@@ -99,9 +99,9 @@ def test_default_values() -> None:
class TestDefaultDiadefGenerator:
_should_rels = [
+ ("aggregation", "DoNothing2", "Specialization"),
("association", "DoNothing", "Ancestor"),
("association", "DoNothing", "Specialization"),
- ("association", "DoNothing2", "Specialization"),
("implements", "Ancestor", "Interface"),
("specialization", "Specialization", "Ancestor"),
]