test failures with new fonts-crosextra-carlito

[Rene Engelhard]

Hi,

Am 26.06.23 um 09:06 schrieb Mike Kaganski:
> On 26.06.2023 8:15, Rene Engelhard wrote:
>> Hi,
>>
>> Am 25.06.23 um 13:29 schrieb Rene Engelhard:
>>> my beta1 (didn't try 7.5 yet) builds fail with a new verson of the 
>>> fonts-crosextra-carlito fonts (used for Calibri). LibreOffice itself 
>>> ships
>>
>> My 7.5.4 builds needs the following in addition to  the already 
>> posted (where applicable):
>>
>> rene at frodo:~/Debian/Pakete/LibreOffice/libreoffice/libreoffice-7.5.4.2$ 
>> diff -u sw/qa/extras/ooxmlexport/ooxmlexport14.cxx-old 
>> sw/qa/extras/ooxmlexport/ooxmlexport14.cxx
>> --- sw/qa/extras/ooxmlexport/ooxmlexport14.cxx-old    2023-06-26 
>> 06:30:09.119195022 +0200
>> +++ sw/qa/extras/ooxmlexport/ooxmlexport14.cxx    2023-06-26 
>> 06:30:18.891263759 +0200
>> @@ -1346,8 +1346,7 @@
>>   #if !defined(MACOSX)
>>   DECLARE_OOXMLEXPORT_TEST(testTdf146346, "tdf146346.docx")
>>   {
>> -    // This was 2 (by bad docDefault vertical margins around tables 
>> in footnotes)
>> -    CPPUNIT_ASSERT_EQUAL(1, getPages());
>> +    CPPUNIT_ASSERT_EQUAL(2, getPages());
>>   }
>>   #endif
>
I take that one back, it seems to be flaky. I get 2, the apply the 
patch, then somehow get 1, revert the patch, get 2 again etc. I'll 
disable it for now.[...]

 > 2. A problem in unit test, claiming one page in master, when actually 
it's two.

And the test seems to be flaky...


Regards,


Rene