shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
fonttools/Tests/svgLib/path/parser_test.py

446 lines
13 KiB
Python
Raw Normal View History

[Tests/misc/svgPath] add tests for SVGPath and parse_path
2017-09-11 17:38:33 +01:00
from fontTools.pens.recordingPen import RecordingPen
move fontTools.misc.svgPath to fontTools.svgLib
2017-09-12 08:46:04 -04:00
from fontTools.svgLib import parse_path
[Tests/misc/svgPath] add tests for SVGPath and parse_path
2017-09-11 17:38:33 +01:00
import pytest
@pytest.mark.parametrize(
"pathdef, expected",
[
# Examples from the SVG spec
(
"M 100 100 L 300 100 L 200 300 z",
[
("moveTo", ((100.0, 100.0),)),
("lineTo", ((300.0, 100.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
# for Z command behavior when there is multiple subpaths
(
"M 0 0 L 50 20 M 100 100 L 300 100 L 200 300 z",
[
("moveTo", ((0.0, 0.0),)),
("lineTo", ((50.0, 20.0),)),
("endPath", ()),
("moveTo", ((100.0, 100.0),)),
("lineTo", ((300.0, 100.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
(
"M100,200 C100,100 250,100 250,200 S400,300 400,200",
[
("moveTo", ((100.0, 200.0),)),
("curveTo", ((100.0, 100.0),
(250.0, 100.0),
(250.0, 200.0))),
("curveTo", ((250.0, 300.0),
(400.0, 300.0),
(400.0, 200.0))),
("endPath", ()),
]
),
(
"M100,200 C100,100 400,100 400,200",
[
("moveTo", ((100.0, 200.0),)),
("curveTo", ((100.0, 100.0),
(400.0, 100.0),
(400.0, 200.0))),
("endPath", ()),
]
),
(
"M100,500 C25,400 475,400 400,500",
[
("moveTo", ((100.0, 500.0),)),
("curveTo", ((25.0, 400.0),
(475.0, 400.0),
(400.0, 500.0))),
("endPath", ()),
]
),
(
"M100,800 C175,700 325,700 400,800",
[
("moveTo", ((100.0, 800.0),)),
("curveTo", ((175.0, 700.0),
(325.0, 700.0),
(400.0, 800.0))),
("endPath", ()),
]
),
(
"M600,200 C675,100 975,100 900,200",
[
("moveTo", ((600.0, 200.0),)),
("curveTo", ((675.0, 100.0),
(975.0, 100.0),
(900.0, 200.0))),
("endPath", ()),
]
),
(
"M600,500 C600,350 900,650 900,500",
[
("moveTo", ((600.0, 500.0),)),
("curveTo", ((600.0, 350.0),
(900.0, 650.0),
(900.0, 500.0))),
("endPath", ()),
]
),
(
"M600,800 C625,700 725,700 750,800 S875,900 900,800",
[
("moveTo", ((600.0, 800.0),)),
("curveTo", ((625.0, 700.0),
(725.0, 700.0),
(750.0, 800.0))),
("curveTo", ((775.0, 900.0),
(875.0, 900.0),
(900.0, 800.0))),
("endPath", ()),
]
),
(
"M200,300 Q400,50 600,300 T1000,300",
[
("moveTo", ((200.0, 300.0),)),
("qCurveTo", ((400.0, 50.0),
(600.0, 300.0))),
("qCurveTo", ((800.0, 550.0),
(1000.0, 300.0))),
("endPath", ()),
]
),
# End examples from SVG spec
# Relative moveto
(
"M 0 0 L 50 20 m 50 80 L 300 100 L 200 300 z",
[
("moveTo", ((0.0, 0.0),)),
("lineTo", ((50.0, 20.0),)),
("endPath", ()),
("moveTo", ((100.0, 100.0),)),
("lineTo", ((300.0, 100.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
# Initial smooth and relative curveTo
(
"M100,200 s 150,-100 150,0",
[
("moveTo", ((100.0, 200.0),)),
("curveTo", ((100.0, 200.0),
(250.0, 100.0),
(250.0, 200.0))),
("endPath", ()),
]
),
# Initial smooth and relative qCurveTo
(
"M100,200 t 150,0",
[
("moveTo", ((100.0, 200.0),)),
("qCurveTo", ((100.0, 200.0),
(250.0, 200.0))),
("endPath", ()),
]
),
# relative l command
(
"M 100 100 L 300 100 l -100 200 z",
[
("moveTo", ((100.0, 100.0),)),
("lineTo", ((300.0, 100.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
# relative q command
(
"M200,300 q200,-250 400,0",
[
("moveTo", ((200.0, 300.0),)),
("qCurveTo", ((400.0, 50.0),
(600.0, 300.0))),
("endPath", ()),
]
),
# absolute H command
(
"M 100 100 H 300 L 200 300 z",
[
("moveTo", ((100.0, 100.0),)),
("lineTo", ((300.0, 100.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
# relative h command
(
"M 100 100 h 200 L 200 300 z",
[
("moveTo", ((100.0, 100.0),)),
("lineTo", ((300.0, 100.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
# absolute V command
(
"M 100 100 V 300 L 200 300 z",
[
("moveTo", ((100.0, 100.0),)),
("lineTo", ((100.0, 300.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
# relative v command
(
"M 100 100 v 200 L 200 300 z",
[
("moveTo", ((100.0, 100.0),)),
("lineTo", ((100.0, 300.0),)),
("lineTo", ((200.0, 300.0),)),
("lineTo", ((100.0, 100.0),)),
("closePath", ()),
]
),
]
)
def test_parse_path(pathdef, expected):
pen = RecordingPen()
parse_path(pathdef, pen)
Revert "Merge pull request #1136 from santhoshtr/svg-arc-support" This reverts commit 5c392bc86542fbbc0c63335cfedae0d1406b0794, reversing changes made to 4b69d77ae57776480901e2af82c7d9c1c29de8d6.
2018-01-05 13:07:57 +00:00
assert pen.value == expected
[Tests/misc/svgPath] add tests for SVGPath and parse_path
2017-09-11 17:38:33 +01:00
@pytest.mark.parametrize(
"pathdef1, pathdef2",
[
# don't need spaces between numbers and commands
(
"M 100 100 L 200 200",
"M100 100L200 200",
),
# repeated implicit command
(
"M 100 200 L 200 100 L -100 -200",
"M 100 200 L 200 100 -100 -200"
),
# don't need spaces before a minus-sign
(
"M100,200c10-5,20-10,30-20",
"M 100 200 c 10 -5 20 -10 30 -20"
),
# closed paths have an implicit lineTo if they don't
# end on the same point as the initial moveTo
(
"M 100 100 L 300 100 L 200 300 z",
"M 100 100 L 300 100 L 200 300 L 100 100 z"
)
]
)
def test_equivalent_paths(pathdef1, pathdef2):
pen1 = RecordingPen()
parse_path(pathdef1, pen1)
pen2 = RecordingPen()
parse_path(pathdef2, pen2)
assert pen1.value == pen2.value
def test_exponents():
# It can be e or E, the plus is optional, and a minimum of +/-3.4e38 must be supported.
pen = RecordingPen()
parse_path("M-3.4e38 3.4E+38L-3.4E-38,3.4e-38", pen)
expected = [
("moveTo", ((-3.4e+38, 3.4e+38),)),
("lineTo", ((-3.4e-38, 3.4e-38),)),
("endPath", ()),
]
assert pen.value == expected
Handle one more valid real number format in svgLib parse_path() Using svg.draw(pen) and parse_path() from svgLib threw exception ValueError: could not convert string to float: 'a' on a SVG <path> string generated by Inkscape. Altering the path string might object to other string bits like 'Z' or 'v', or give even stranger exceptions. Finally tracked it down to the path containing particular valid numbers like "-4e-5" or "1e-4". Changing these to "-4.0e-5" or "1.0e-4" would stop the exceptions. The parse_path() was not accepting valid SVG real numbers. The specification for real number formats is a bit of a mess in CSS land right now, but the reassuringly concrete spec is: https://www.w3.org/TR/css-syntax-3/#number-token-diagram which allows a real number having an exponent but without having a fractional part, such as the number "1e3". This change updates an RE to make fractional parts optional, and adds a test for this valid SVG number format.
2022-05-03 17:48:14 -05:00
pen = RecordingPen()
parse_path("M-3e38 3E+38L-3E-38,3e-38", pen)
expected = [
("moveTo", ((-3e+38, 3e+38),)),
("lineTo", ((-3e-38, 3e-38),)),
("endPath", ()),
]
assert pen.value == expected
[Tests/misc/svgPath] add tests for SVGPath and parse_path
2017-09-11 17:38:33 +01:00
def test_invalid_implicit_command():
with pytest.raises(ValueError) as exc_info:
parse_path("M 100 100 L 200 200 Z 100 200", RecordingPen())
assert exc_info.match("Unallowed implicit command")
Revert "Merge pull request #1136 from santhoshtr/svg-arc-support" This reverts commit 5c392bc86542fbbc0c63335cfedae0d1406b0794, reversing changes made to 4b69d77ae57776480901e2af82c7d9c1c29de8d6.
2018-01-05 13:07:57 +00:00
add tests for parsing SVG arcs
2019-01-24 10:55:38 +00:00
def test_arc_to_cubic_bezier():
pen = RecordingPen()
parse_path("M300,200 h-150 a150,150 0 1,0 150,-150 z", pen)
expected = [
('moveTo', ((300.0, 200.0),)),
('lineTo', ((150.0, 200.0),)),
(
'curveTo',
(
(150.0, 282.842),
(217.157, 350.0),
(300.0, 350.0)
)
),
(
'curveTo',
(
(382.842, 350.0),
(450.0, 282.842),
(450.0, 200.0)
)
),
(
'curveTo',
(
(450.0, 117.157),
(382.842, 50.0),
(300.0, 50.0)
)
),
('lineTo', ((300.0, 200.0),)),
('closePath', ())
]
result = list(pen.value)
assert len(result) == len(expected)
for (cmd1, points1), (cmd2, points2) in zip(result, expected):
assert cmd1 == cmd2
assert len(points1) == len(points2)
for pt1, pt2 in zip(points1, points2):
assert pt1 == pytest.approx(pt2, rel=1e-5)
class ArcRecordingPen(RecordingPen):
def arcTo(self, rx, ry, rotation, arc_large, arc_sweep, end_point):
self.value.append(
("arcTo", (rx, ry, rotation, arc_large, arc_sweep, end_point))
)
def test_arc_pen_with_arcTo():
pen = ArcRecordingPen()
parse_path("M300,200 h-150 a150,150 0 1,0 150,-150 z", pen)
expected = [
('moveTo', ((300.0, 200.0),)),
('lineTo', ((150.0, 200.0),)),
('arcTo', (150.0, 150.0, 0.0, True, False, (300.0, 50.0))),
('lineTo', ((300.0, 200.0),)),
('closePath', ())
]
assert pen.value == expected
svgLib: handle arc commands with bool flags not separated by space/comma Some SVG authoring tool write arc commands without any space or comma around the boolean 'large-arc' and 'sweep' flags, leading our svg path parser to choke. This makes the path parser smarter by special-casing arc command parsing so that it only consumes one character ('0' or '1') for these special boolean flags.
2020-10-29 15:00:10 +00:00
@pytest.mark.parametrize(
"path, expected",
[
(
"M1-2A3-4-1.0 01.5.7",
[
("moveTo", ((1.0, -2.0),)),
Update svgLib test for now non-negative arc radius values Missed this test in previous commit. Test is testing that parsing extraction of individual parameters works as specified when spaces are omitted. Such as signaled when a new number is specified with a leading sign +/- Test previously used "M1-2A3-4-1.0 01.5.7", where the first arc radius value was '3' and the second was '-4'. Now that we are forcing radius values to be non-negative using abs() the value returned in the test is not ("arcTo", (3.0, -4.0, -1.0, False, True, (0.5, 0.7))), but rather ("arcTo", (3.0, 4.0, -1.0, False, True, (0.5, 0.7))), Changed to expect the positive value, nicely the test continues to test omitting spaces, but now also tests that negative radius values are forced non-negative.
2022-05-03 17:21:48 -05:00
("arcTo", (3.0, 4.0, -1.0, False, True, (0.5, 0.7))),
svgLib: handle arc commands with bool flags not separated by space/comma Some SVG authoring tool write arc commands without any space or comma around the boolean 'large-arc' and 'sweep' flags, leading our svg path parser to choke. This makes the path parser smarter by special-casing arc command parsing so that it only consumes one character ('0' or '1') for these special boolean flags.
2020-10-29 15:00:10 +00:00
("endPath", ()),
],
),
(
"M21.58 7.19a2.51 2.51 0 10-1.77-1.77",
[
("moveTo", ((21.58, 7.19),)),
("arcTo", (2.51, 2.51, 0.0, True, False, (19.81, 5.42))),
("endPath", ()),
],
),
(
"M22 12a25.87 25.87 0 00-.42-4.81",
[
("moveTo", ((22.0, 12.0),)),
("arcTo", (25.87, 25.87, 0.0, False, False, (21.58, 7.19))),
("endPath", ()),
],
),
(
"M0,0 A1.2 1.2 0 012 15.8",
[
("moveTo", ((0.0, 0.0),)),
("arcTo", (1.2, 1.2, 0.0, False, True, (2.0, 15.8))),
("endPath", ()),
],
),
(
"M12 7a5 5 0 105 5 5 5 0 00-5-5",
[
("moveTo", ((12.0, 7.0),)),
("arcTo", (5.0, 5.0, 0.0, True, False, (17.0, 12.0))),
("arcTo", (5.0, 5.0, 0.0, False, False, (12.0, 7.0))),
("endPath", ()),
],
)
],
)
def test_arc_flags_without_spaces(path, expected):
pen = ArcRecordingPen()
parse_path(path, pen)
assert pen.value == expected
@pytest.mark.parametrize(
"path", ["A", "A0,0,0,0,0,0", "A 0 0 0 0 0 0 0 0 0 0 0 0 0"]
)
def test_invalid_arc_not_enough_args(path):
pen = ArcRecordingPen()
with pytest.raises(ValueError, match="Invalid arc command") as e:
parse_path(path, pen)
assert isinstance(e.value.__cause__, ValueError)
assert "Not enough arguments" in str(e.value.__cause__)
def test_invalid_arc_argument_value():
pen = ArcRecordingPen()
with pytest.raises(ValueError, match="Invalid arc command") as e:
parse_path("M0,0 A0,0,0,2,0,0,0", pen)
cause = e.value.__cause__
assert isinstance(cause, ValueError)
assert "Invalid argument for 'large-arc-flag' parameter: '2'" in str(cause)
pen = ArcRecordingPen()
with pytest.raises(ValueError, match="Invalid arc command") as e:
parse_path("M0,0 A0,0,0,0,-2.0,0,0", pen)
cause = e.value.__cause__
assert isinstance(cause, ValueError)
assert "Invalid argument for 'sweep-flag' parameter: '-2.0'" in str(cause)
Reference in New Issue Copy Permalink