From 290d702b24441a3455ff83df9bc47ce2a5be2ff0 Mon Sep 17 00:00:00 2001 From: Magnus Jacobsson Date: Thu, 6 Oct 2022 11:01:37 +0200 Subject: [PATCH] tests: test_edge_node_overlap_utilities: adjust max_edge_stem_arrow_overlap for 'diamond' arrow --- tests/test_edge_node_overlap_utilities.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/test_edge_node_overlap_utilities.cpp b/tests/test_edge_node_overlap_utilities.cpp index 847699f6f..f0302670c 100644 --- a/tests/test_edge_node_overlap_utilities.cpp +++ b/tests/test_edge_node_overlap_utilities.cpp @@ -446,8 +446,11 @@ void test_edge_node_overlap(const graph_options &graph_options, // FIXME: adjust this when `curve` is fixed for penwidth graph_options.edge_penwidth / 2 + graphviz_bezier_clip_margin}, {"diamond", - // FIXME: adjust this when `diamond` is fixed for penwidth - graph_options.edge_penwidth / 2 + graphviz_bezier_clip_margin}, + [&]() { // FIXME: calculate this accurately for diamond arrow + const auto tip_scale = 1.8027756377319939; // empirical value + return graph_options.edge_penwidth / 2 * tip_scale + + graphviz_bezier_clip_margin; + }()}, {"dot", graph_options.edge_penwidth / 2 + graphviz_bezier_clip_margin}, {"icurve", -- 2.40.0