Pyreverse: Use dashed lines for type-checking imports

pylint/pyreverse/diagrams.py

@@ -12,7 +12,7 @@
 import astroid
 from astroid import nodes, util
 
-from pylint.checkers.utils import decorated_with_property
+from pylint.checkers.utils import decorated_with_property, in_type_checking_block
 from pylint.pyreverse.utils import FilterMixIn
 
 
@@ -281,9 +281,17 @@
     def add_from_depend(self, node: nodes.ImportFrom, from_module: str) -> None:
         """Add dependencies created by from-imports."""
         mod_name = node.root().name
-        obj = self.module(mod_name)
-        if from_module not in obj.node.depends:
-            obj.node.depends.append(from_module)
+        package = self.module(mod_name).node
+
+        if not in_type_checking_block(node):
+            if from_module not in package.depends:
+                package.depends.append(from_module)
+        else:
+            if (
+                from_module not in package.type_depends
+                and from_module not in package.depends
+            ):
+                package.type_depends.append(from_module)
 
     def extract_relationships(self) -> None:
         """Extract relationships between nodes in the diagram."""
@@ -304,3 +312,10 @@
                 except KeyError:
                     continue
                 self.add_relationship(package_obj, dep, "depends")
+
+            for dep_name in package_obj.node.type_depends:
+                try:
+                    dep = self.get_module(dep_name, package_obj.node)
+                except KeyError:
+                    continue
+                self.add_relationship(package_obj, dep, "type_depends")

pylint/pyreverse/dot_printer.py

@@ -43,6 +43,7 @@ class HTMLLabels(Enum):
         "style": "solid",
     },
     EdgeType.USES: {"arrowtail": "none", "arrowhead": "open"},
+    EdgeType.TYPE_USES: {"arrowtail": "none", "arrowhead": "empty"},
 }
 
 

pylint/pyreverse/inspector.py

@@ -139,6 +139,7 @@ def visit_module(self, node: nodes.Module) -> None:
             return
         node.locals_type = collections.defaultdict(list)
         node.depends = []
+        node.type_depends = []
         if self.tag:
             node.uid = self.generate_id()
 

pylint/pyreverse/printer.py

@@ -25,6 +25,7 @@ class EdgeType(Enum):
     ASSOCIATION = "association"
     AGGREGATION = "aggregation"
     USES = "uses"
+    TYPE_USES = "type_uses"
 
 
 class Layout(Enum):

pylint/pyreverse/writer.py

@@ -76,6 +76,13 @@
                 type_=EdgeType.USES,
             )
 
+        for rel in diagram.get_relationships("type_depends"):
+            self.printer.emit_edge(
+                rel.from_object.fig_id,
+                rel.to_object.fig_id,
+                type_=EdgeType.TYPE_USES,
+            )
+
     def write_classes(self, diagram: ClassDiagram) -> None:
         """Write a class diagram."""
         # sorted to get predictable (hence testable) results