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.