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