Fix tests

README.md

@@ -58,7 +58,7 @@ namespace Rubjerg.Graphviz.Test;
 public class Tutorial
 {
     public const string PointPattern = @"{X=[\d.]+, Y=[\d.]+}";
-    public const string RectPattern = @"{X=[\d.]+,Y=[\d.]+,Width=[\d.]+,Height=[\d.]+}";
+    public const string RectPattern = @"{X=[\d.]+, Y=[\d.]+, Width=[\d.]+, Height=[\d.]+}";
     public const string SplinePattern =
         @"{X=[\d.]+, Y=[\d.]+}, {X=[\d.]+, Y=[\d.]+}, {X=[\d.]+, Y=[\d.]+}, {X=[\d.]+, Y=[\d.]+}";
 
@@ -130,8 +130,10 @@ public class Tutorial
 
         // Or we can ask Graphviz to compute the layout and programatically read out the layout attributes
         // This will create a copy of our original graph with layout information attached to it in the form
-        // of attributes.
-        RootGraph layout = root.CreateLayout();
+        // of attributes. Graphviz outputs coordinates in a bottom-left originated coordinate system.
+        // But since many applications require rendering in a top-left originated coordinate system,
+        // we provide a way to translate the coordinates.
+        RootGraph layout = root.CreateLayout(coordinateSystem: CoordinateSystem.TopLeft);
 
         // There are convenience methods available that parse these attributes for us and give
         // back the layout information in an accessible form.

Rubjerg.Graphviz.Test/OldTutorial.cs

@@ -73,7 +73,7 @@ public void Layouting()
 
         // Like a bounding box of an object
         RectangleD nodeboundingbox = nodeA.GetBoundingBox();
-        Utils.AssertPattern(@"{X=[\d.]+,Y=[\d.]+,Width=[\d.]+,Height=[\d.]+}", nodeboundingbox.ToString());
+        Utils.AssertPattern(@"{X=[\d.]+, Y=[\d.]+, Width=[\d.]+, Height=[\d.]+}", nodeboundingbox.ToString());
 
         // Or splines between nodes
         Node nodeB = root.GetNode("B");
@@ -133,8 +133,8 @@ public void Clusters()
         SubGraph cluster = root.GetSubgraph("cluster_1");
         RectangleD clusterbox = cluster.GetBoundingBox();
         RectangleD rootgraphbox = root.GetBoundingBox();
-        Utils.AssertPattern(@"{X=[\d.]+,Y=[\d.]+,Width=[\d.]+,Height=[\d.]+}", clusterbox.ToString());
-        Utils.AssertPattern(@"{X=[\d.]+,Y=[\d.]+,Width=[\d.]+,Height=[\d.]+}", rootgraphbox.ToString());
+        Utils.AssertPattern(@"{X=[\d.]+, Y=[\d.]+, Width=[\d.]+, Height=[\d.]+}", clusterbox.ToString());
+        Utils.AssertPattern(@"{X=[\d.]+, Y=[\d.]+, Width=[\d.]+, Height=[\d.]+}", rootgraphbox.ToString());
     }
 
     [Test, Order(4)]

Rubjerg.Graphviz.Test/Reproductions.cs

@@ -40,7 +40,8 @@ public void TestRecordShapeAlignment(string fontname, double fontsize, double ma
         root.ComputeLayout();
         //TestContext.Write(root.ToDotString());
 
-        var rects = nodeA.GetRecordRectangles().ToList();
+        // This test is fixed by passing snapOntoDrawingCoordinates: true
+        var rects = nodeA.GetRecordRectangles(snapOntoDrawingCoordinates: true).ToList();
         Assert.That(rects[0].FarPoint().X, Is.EqualTo(rects[2].FarPoint().X));
     }
 

Rubjerg.Graphviz.Test/TestDotLayout.cs

@@ -34,13 +34,13 @@ public void TestLayoutMethodsWithoutLayout()
     {
         CreateSimpleTestGraph(out RootGraph root, out Node nodeA, out Edge edge);
 
-        Assert.AreEqual(root.GetBoundingBox(), default(RectangleF));
+        Assert.AreEqual(root.GetBoundingBox(), default(RectangleD));
         Assert.AreEqual(root.GetDrawing().Count, 0);
         Assert.AreEqual(root.GetLabelDrawing().Count, 0);
 
-        Assert.AreEqual(nodeA.GetPosition(), default(PointF));
-        Assert.AreEqual(nodeA.GetBoundingBox(), default(RectangleF));
-        Assert.AreEqual(nodeA.GetSize(), default(SizeF));
+        Assert.AreEqual(nodeA.GetPosition(), default(PointD));
+        Assert.AreEqual(nodeA.GetBoundingBox(), default(RectangleD));
+        Assert.AreEqual(nodeA.GetSize(), default(SizeD));
         Assert.AreEqual(nodeA.GetRecordRectangles().Count(), 0);
         Assert.AreEqual(nodeA.GetDrawing().Count, 0);
         Assert.AreEqual(nodeA.GetLabelDrawing().Count, 0);
@@ -181,7 +181,7 @@ public void TestRecordShapeOrder()
 
         var rects = nodeA.GetRecordRectangles().ToList();
 
-        Utils.AssertOrder(rects, r => (r.Origin.X, r.Origin.Y));
+        Utils.AssertOrder(rects, r => (r.Origin.X, -r.Origin.Y));
         Assert.That(rects.Count, Is.EqualTo(9));
     }
 

Rubjerg.Graphviz.Test/TestXDotLayout.cs

@@ -82,7 +82,7 @@ public void TestRecordShapeOrder()
         nodeA.SetAttribute("label", "1|2|3|{4|5}|6|{7|8|9}");
 
 
-        var xdotGraph = root.CreateLayout();
+        var xdotGraph = root.CreateLayout(coordinateSystem: CoordinateSystem.TopLeft);
 
         var xNodeA = xdotGraph.GetNode("A");
         var rects = xNodeA.GetRecordRectangles().ToList();
@@ -108,4 +108,15 @@ public void TestEmptyRecordShapes()
         var rects = xNodeA.GetRecordRectangles().ToList();
         Assert.That(rects.Count, Is.EqualTo(5));
     }
+
+    [Test()]
+    public void TestCoordinateTransformation()
+    {
+        RootGraph root = Utils.CreateUniqueTestGraph();
+        Node nodeA = root.GetOrAddNode("A");
+        var xdotGraph = root.CreateLayout(coordinateSystem: CoordinateSystem.TopLeft);
+        // Check that translating back gets us the old bounding box
+        var translatedBack = xdotGraph.GetBoundingBox().ForCoordSystem(CoordinateSystem.BottomLeft, xdotGraph.RawMaxY());
+        Assert.AreEqual(translatedBack, xdotGraph.RawBoundingBox());
+    }
 }

Rubjerg.Graphviz.Test/Tutorial.cs

@@ -7,7 +7,7 @@ namespace Rubjerg.Graphviz.Test;
 public class Tutorial
 {
     public const string PointPattern = @"{X=[\d.]+, Y=[\d.]+}";
-    public const string RectPattern = @"{X=[\d.]+,Y=[\d.]+,Width=[\d.]+,Height=[\d.]+}";
+    public const string RectPattern = @"{X=[\d.]+, Y=[\d.]+, Width=[\d.]+, Height=[\d.]+}";
     public const string SplinePattern =
         @"{X=[\d.]+, Y=[\d.]+}, {X=[\d.]+, Y=[\d.]+}, {X=[\d.]+, Y=[\d.]+}, {X=[\d.]+, Y=[\d.]+}";
 
@@ -79,8 +79,10 @@ public void Layouting()
 
         // Or we can ask Graphviz to compute the layout and programatically read out the layout attributes
         // This will create a copy of our original graph with layout information attached to it in the form
-        // of attributes.
-        RootGraph layout = root.CreateLayout();
+        // of attributes. Graphviz outputs coordinates in a bottom-left originated coordinate system.
+        // But since many applications require rendering in a top-left originated coordinate system,
+        // we provide a way to translate the coordinates.
+        RootGraph layout = root.CreateLayout(coordinateSystem: CoordinateSystem.TopLeft);
 
         // There are convenience methods available that parse these attributes for us and give
         // back the layout information in an accessible form.

Rubjerg.Graphviz/XDot.cs

@@ -72,6 +72,8 @@ internal PointD ForCoordSystem(CoordinateSystem coordSystem, double maxY)
             return this;
         return new PointD(X, maxY - Y);
     }
+
+    public override string ToString() => $"{{X={X}, Y={Y}}}";
 }
 
 /// <param name="Origin">The origin of the rectangle, which is the point closest to the origin of the coordinate system.</param>
@@ -104,6 +106,8 @@ internal RectangleD ForCoordSystem(CoordinateSystem coordSystem, double maxY)
             Origin = new PointD(translated.X, translated.Y - Height),
         };
     }
+
+    public override string ToString() => $"{{X={X}, Y={Y}, Width={Width}, Height={Height}}}";
 }
 
 public record struct ColorStop(float Frac, string HtmlColor);