Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


Hello Kendy, Stephan,

thanks for your help and pushing this (and the original patch).


The approach looks rather sane to me (and apparently to Kendy also, who
already pushed).  In general, the qadevOOo based tests are rather a mess,
and nobody loves them.  However, they proved useful in finding regressions
over and over again, so we should see to keep them working until newer,
better tests replace their code coverage (and even go beyond).

Ah, ok.

I just extended your approach of explicitly inserting an additional sheet to
qadevOOo/tests/java/mod/_sc/ScModelObj.java (see
<http://cgit.freedesktop.org/libreoffice/core/commit/?id=0210ec32f779bb2098657eb4ad69cca7a60835dc>
"Fix one more test that implicitly assumes more than one Calc sheet") and it
appears to work well.

Oh. Good.

Regards,
Astron.

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.