[FOP-1705] added regression to font-shorthand-test.fo that shows bug

• Type: Bug
• Status: Closed
• Resolution: Duplicate
• Affects Version/s: 0.95
• Component/s: fo/unqualified
• Environment: Operating System: Windows XP; Platform: PC
• External issue ID: 47859
• Assignee: fop-dev
• Reporter: Jonathan Levinson
• Created: 17/Sep/09 06:09
• Updated: 17/Sep/09 08:59
• Resolved: 01/Apr/12 13:55

Description

I added a regression to font-shorthand-test.fo that shows a bug in font shorthand capability.

We apparently have trouble handling the case where a quoted font follows an unquoted font in the comma separated list of fonts.

Here is the case that fails:

<fo:block font="xx-large/1.4 Arial, 'Times New Roman', sans-serif">
<test:assert property="font-family" expected="[Arial, Times New Roman, sans-serif]"/>
<test:assert property="font-size" expected="20736mpt"/>
<test:assert property="font-weight" expected="400" />
<test:assert property="font-style" expected="NORMAL" />
<test:assert property="line-height.optimum" expected="29030mpt" />
<test:assert property="font-variant" expected="NORMAL" />
Test font shorthand
</fo:block>

The error message is the following:

file:/C:/perforce/Users/levinson/fop-trunk/test/fotree/testcases/font-shorthand-test.fo Property 'font-family' expected to evaluate to '[Arial, Times New Roman, sans-serif]' but got '[sans-serif, Symbol, ZapfDingbats]' (test:assert in fo:block at line #72, column #94)

I add a patch file.

Issue Links

• duplicates: FOP-1694 Issue with parsing the 'font' shorthand