Re: CppunitTest_chart2_dump Unit Test Failure

[Mike Kaganski <mike.kaganski@xxxxxxxxxxxxx>]

Hi Luke, Tamás,

On 20.03.2019 9:39, Tamás Zolnai wrote:
> Hi guys,
>
> Don't waste to much time on this issue. You can just increase the 
> delta (INT_EPS) to a bigger value for 32 bit windows systems and 
> that's all. I think it's enough if the precise values are checked on 
> 64 bit systems. This is a known issue that checking the positions of 
> chart components is unstable, so it might be just small platform 
> differences. As you can see in the code INT_EPS already set to a 
> bigger value for 32bit Linux, you can add a check for 32bit windows 
> too and set this delta to a different value which makes the test pass.

Thanks for suggestions.
Actually, I've already prepared https://gerrit.libreoffice.org/69458 - 
does it look reasonable?

> Best Regards,
> Tamás
>
> Luke Benes <lukebenes@xxxxxxxxxxx <mailto:lukebenes@xxxxxxxxxxx>> ezt 
> írta (időpont: 2019. márc. 20., Sze, 3:04):
>
>     Hey Mike,
>     After
>     https://cgit.freedesktop.org/libreoffice/core/commit/?id=166a4989a0d1e5a6271c66bceb73a27970afc882
>
>     tdf#123504: 0 and 360 are different angles in charts
>
>     My 32-bit Windows  box is failing with the following error:
>
>     ChartDataTest::verify finished in: 2796ms
>     PivotChartDataButtonTest::verify finished in: 962ms
>     PointLineChartTest::verify finished in: 15087ms
>     GridTest::verify finished in: 5035ms
>     AxisLabelTest::verify finished in: 3948ms
>     AxisGeometryTest::verify finished in: 8078ms
>     D:/core/chart2/qa/extras/chart2dump/chart2dump.cxx:902:PieChartTest::verify
>     double equality assertion failed
>     - Expected: 3210
>     - Actual  : 3207
>     - Delta   : 0.1
>     - Failing test file is: pie_chart_100_and_0.ods
>
>     PieChartTest::verify finished in: 8407ms
>     ColumnBarChartTest::verify finished in: 10112ms
>     D:/core/chart2/qa/extras/chart2dump/chart2dump.cxx(902) : error :
>     Assertion
>     Test name: PieChartTest::verify
>     double equality assertion failed
>     - Expected: 3210
>     - Actual  : 3207
>     - Delta   : 0.1
>     - Failing test file is: pie_chart_100_and_0.ods
>
>     Failures !!!
>     Run: 11   Failure total: 1   Failures: 1   Errors: 0
>
>     Error: a unit test failed, please do one of:
>     make CppunitTest_chart2_dump CPPUNITTRACE=TRUE # which is a
>     shortcut for the following line
>     make CppunitTest_chart2_dump CPPUNITTRACE="'C:/Program Files
>     (x86)/Microsoft Visual
>     Studio/2017/Community/Common7/IDE/devenv.exe' /debugexe" # for
>     interactive debugging in Visual Studio
>     make CppunitTest_chart2_dump CPPUNITTRACE="drmemory
>     -free_max_frames 20" # for memory checking (install Dr.Memory
>     first, and put it to your PATH)
>
>     You can limit the execution to just one particular test by:
>
>     make CppunitTest_chart2_dump CPPUNIT_TEST_NAME="testXYZ" ...above
>     mentioned params...
>
>     make[1]: *** [D:/core/solenv/gbuild/CppunitTest.mk:114:
>     D:/core/workdir/CppunitTest/chart2_dump.test] Error 1
>     make[1]: *** Waiting for unfinished jobs....
>     make: *** [Makefile:282: build] Error 2
>
>     Would you like any more debug info or info about my system?
>
>     Thanks
>
>     -Luke
>
>
>     _______________________________________________
>     LibreOffice mailing list
>     LibreOffice@xxxxxxxxxxxxxxxxxxxxx
>     <mailto:LibreOffice@xxxxxxxxxxxxxxxxxxxxx>
>     https://lists.freedesktop.org/mailman/listinfo/libreoffice

--
Best regards,
Mike Kaganski
_______________________________________________
LibreOffice mailing list
LibreOffice@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/libreoffice