From c81faf3537ff1a49ae392865a89577e56eb324f9 Mon Sep 17 00:00:00 2001
From: Roman Telezhynskyi <kroluku@gmail.com>
Date: Sat, 11 Jan 2020 14:51:25 +0200
Subject: [PATCH] Fix regressions. ref #989 (grafted from
 1bbf6a73a6e3bf5fd9dc2b962bd1428f391da68b)

--HG--
branch : develop
---
 src/libs/vgeometry/vellipticalarc.cpp         |  5 ++++
 src/test/ValentinaTest/tst_vellipticalarc.cpp | 26 ++++++++++++++-----
 2 files changed, 24 insertions(+), 7 deletions(-)

diff --git a/src/libs/vgeometry/vellipticalarc.cpp b/src/libs/vgeometry/vellipticalarc.cpp
index 6accc9b30..bc10ea2a8 100644
--- a/src/libs/vgeometry/vellipticalarc.cpp
+++ b/src/libs/vgeometry/vellipticalarc.cpp
@@ -295,6 +295,11 @@ QVector<QPointF> VEllipticalArc::GetPoints() const
     if (not sub.isEmpty())
     {
         polygon = ConstFirst (path.toSubpathPolygons());
+
+        if (not polygon.isEmpty() && not VFuzzyComparePoints(GetP1(), ConstFirst (polygon)))
+        {
+            polygon.removeFirst(); // remove point (0;0)
+        }
     }
 
     return static_cast<QVector<QPointF>>(polygon);
diff --git a/src/test/ValentinaTest/tst_vellipticalarc.cpp b/src/test/ValentinaTest/tst_vellipticalarc.cpp
index d638fbf44..0e03cfcb0 100644
--- a/src/test/ValentinaTest/tst_vellipticalarc.cpp
+++ b/src/test/ValentinaTest/tst_vellipticalarc.cpp
@@ -83,8 +83,9 @@ void TST_VEllipticalArc::CompareTwoWays()
 
     VEllipticalArc arc2(length, center, radius1, radius2, f1, rotationAngle);
 
-    const qreal lengthEps = ToPixel(0.4, Unit::Mm); // computing error
-    const QString errorLengthMsg =
+    const qreal lengthEps = ToPixel(0.45, Unit::Mm); // computing error
+
+    QString errorLengthMsg =
             QString("Difference between real and computing lengthes bigger than eps = %1. l1 = %2; l2 = %3");
     QVERIFY2(qAbs(arc2.GetLength() - length) <= lengthEps,
              qUtf8Printable(errorLengthMsg.arg(lengthEps).arg(arc2.GetLength()).arg(length)));
@@ -116,12 +117,16 @@ void TST_VEllipticalArc::NegativeArc()
     const qreal length = M_PI*(radius1+radius2)*(1+3*h/(10+qSqrt(4-3*h)))/2;
     VEllipticalArc arc(-length, center, radius1, radius2, f1, rotationAngle);
 
-    const qreal eps = 1; // computing error
+    const qreal eps = ToPixel(0.45, Unit::Mm); // computing error
     const QString errorMsg =
-            QString("Difference between real and computing lengthes bigger than eps = %1.").number(eps);
+            QString("Difference between real and computing lengthes bigger than eps = %1.  v1 = %2; v2 = %3");
 
-    QVERIFY2(qAbs(arc.GetLength() + length) <= eps, qUtf8Printable(errorMsg));
-    QVERIFY2(arc.GetEndAngle() - f2 <= eps, qUtf8Printable(errorMsg));
+    QVERIFY2(qAbs(arc.GetLength() + length) <= eps,
+             qUtf8Printable(errorMsg.arg(eps).arg(arc.GetLength()).arg(length)));
+
+    const qreal angleEps = 0.4;
+    QVERIFY2(arc.GetEndAngle() - f2 <= angleEps,
+             qUtf8Printable(errorMsg.arg(eps).arg(arc.GetEndAngle()).arg(f2)));
 }
 
 // cppcheck-suppress unusedFunction
@@ -501,7 +506,14 @@ void TST_VEllipticalArc::TestRotation()
 
     QVERIFY2(qAbs(arcOrigin.AngleArc() - rotatedArc.AngleArc()) <= 1.6,
             qUtf8Printable(QString("a1 = %1, a2 - %2").arg(arcOrigin.AngleArc()).arg(rotatedArc.AngleArc())));
-    QVERIFY(qAbs(arcOrigin.GetLength() - rotatedArc.GetLength()) <= ToPixel(1, Unit::Mm));
+
+    QString errorLengthMsg =
+            QString("Difference between real and computing lengthes bigger than eps = %1. l1 = %2; l2 = %3");
+    QVERIFY2(qAbs(arcOrigin.GetLength() - rotatedArc.GetLength()) <= ToPixel(1, Unit::Mm),
+             qUtf8Printable(errorLengthMsg.arg(ToPixel(1, Unit::Mm))
+                            .arg(arcOrigin.GetLength())
+                            .arg(rotatedArc.GetLength())));
+
     QCOMPARE(arcOrigin.GetRadius1(), rotatedArc.GetRadius1());
     QCOMPARE(arcOrigin.GetRadius2(), rotatedArc.GetRadius2());
     QCOMPARE(arcOrigin.GetRotationAngle(), rotatedArc.GetRotationAngle());