https://bugs.documentfoundation.org/show_bug.cgi?id=97283
Bug ID: 97283
Summary: merge unit tests in sal module
Product: LibreOffice
Version: unspecified
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: medium
Component: LibreOffice
Assignee: libreoffice-bugs@lists.freedesktop.org
Reporter: dtar...@redhat.com
There are 33 separate test libraries in sal module. These should be merged into
3: one for each of osl, rtl and sal.
Most of the tests have the right category already in the name, e.g.,
CppunitTest_sal_osl_file should be merged into a (new) CppunitTest_sal_osl. For
these few that do not follow that naming scheme, check the source files for
what class they are testing (sal::Foo, rtl_Foo,...) and categorize them by
that. It might be a good idea to move the sources into the right subdir of
sal/qa as well, if they are not already there.
The easiest way is to rename one of the makefiles of the selected group (do not
forget to also change the name used inside the makefile and in Module_sal.mk)
and copy source files (and other extra stuff: used libraries, extra
dependencies, etc.) from the other makefiles to it. Then remove
CPPUNIT_PLUGIN_IMPLEMENT() from all source files from the group but one (you
will get linker error if you do not do this). Then remove the now unnecessary
makefiles.
It would be best to do this in 3 separate patches, one for each group.
--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
