sal/qa/rtl/uuid/makefile.mk | 26 +++---- sal/qa/rtl/uuid/rtl_Uuid.cxx | 140 ++++++++++++++++++++----------------------- 2 files changed, 78 insertions(+), 88 deletions(-)
New commits: commit 933db9c80d5510abdd3569a025f654102d83aaf5 Author: Damjan Jovanovic <dam...@apache.org> Date: Sun Aug 30 19:10:42 2015 +0000 #i125003# migrate main/sal/qa/rtl/uuid from cppunit to Google Test. diff --git a/sal/qa/rtl/uuid/makefile.mk b/sal/qa/rtl/uuid/makefile.mk index 80f6b9a..df42f80 100644 --- a/sal/qa/rtl/uuid/makefile.mk +++ b/sal/qa/rtl/uuid/makefile.mk @@ -33,32 +33,30 @@ ENABLE_EXCEPTIONS=TRUE .INCLUDE : settings.mk +.IF "$(ENABLE_UNIT_TESTS)" != "YES" +all: + @echo unit tests are disabled. Nothing to do. + +.ELSE + CFLAGS+= $(LFS_CFLAGS) CXXFLAGS+= $(LFS_CFLAGS) -CFLAGSCXX += $(CPPUNIT_CFLAGS) # BEGIN ---------------------------------------------------------------- # auto generated Target:joblist by codegen.pl -SHL1OBJS= \ +APP1OBJS= \ $(SLO)$/rtl_Uuid.obj -SHL1TARGET= rtl_Uuid -SHL1STDLIBS= $(SALLIB) $(CPPUNITLIB) $(TESTSHL2LIB) - -SHL1IMPLIB= i$(SHL1TARGET) -# SHL1DEF= $(MISC)$/$(SHL1TARGET).def +APP1TARGET= rtl_Uuid +APP1STDLIBS= $(SALLIB) $(GTESTLIB) $(TESTSHL2LIB) +APP1RPATH = NONE +APP1TEST = enabled -DEF1NAME =$(SHL1TARGET) -# DEF1EXPORTFILE= export.exp -SHL1VERSIONMAP= $(PRJ)$/qa$/export.map # END ------------------------------------------------------------------ -#------------------------------- All object files ------------------------------- -# do this here, so we get right dependencies -SLOFILES=$(SHL1OBJS) # --- Targets ------------------------------------------------------ .INCLUDE : target.mk -.INCLUDE : _cppunit.mk +.ENDIF # "$(ENABLE_UNIT_TESTS)" != "YES" diff --git a/sal/qa/rtl/uuid/rtl_Uuid.cxx b/sal/qa/rtl/uuid/rtl_Uuid.cxx index 1016af1..42a6eed 100644 --- a/sal/qa/rtl/uuid/rtl_Uuid.cxx +++ b/sal/qa/rtl/uuid/rtl_Uuid.cxx @@ -27,7 +27,6 @@ #include <math.h> #include <stdio.h> -#include <testshl/simpleheader.hxx> #include <rtl/uuid.h> #include <rtl/ustring.h> #include <rtl/ustring.hxx> @@ -37,16 +36,18 @@ #include <time.h> #endif +#include "gtest/gtest.h" + using namespace rtl; /** print a UNI_CODE String. And also print some comments of the string. */ inline void printUString( const ::rtl::OUString & str, const sal_Char * msg = "" ) { - t_print("#%s #printUString_u# ", msg ); + printf("#%s #printUString_u# ", msg ); rtl::OString aString; aString = ::rtl::OUStringToOString( str, RTL_TEXTENCODING_ASCII_US ); - t_print("%s\n", (char *)aString.getStr( ) ); + printf("%s\n", (char *)aString.getStr( ) ); } /************************************************************************ @@ -71,21 +72,23 @@ void printUuid( sal_uInt8 *pNode ) namespace rtl_Uuid { -class createUuid : public CppUnit::TestFixture +class createUuid : public ::testing::Test { public: // initialise your test code values here. - void setUp() + void SetUp() { } - void tearDown() + void TearDown() { } +}; // class createUuid + #define TEST_UUID 20 - void createUuid_001() - { +TEST_F(createUuid, createUuid_001) +{ sal_uInt8 aNode[TEST_UUID][16]; sal_Int32 i,i2; for( i = 0 ; i < TEST_UUID ; i ++ ) @@ -106,11 +109,12 @@ public: if ( bRes == sal_False ) break; } - CPPUNIT_ASSERT_MESSAGE("createUuid: every uuid must be different.", bRes == sal_True ); - } - /* - void createUuid_002() - { + ASSERT_TRUE(bRes == sal_True) << "createUuid: every uuid must be different."; +} + +/* +TEST_F(createUuid, createUuid_002) +{ sal_uInt8 pNode[16]; sal_uInt8 aNode[TEST_UUID][16]; sal_Int32 i,i2; @@ -133,14 +137,8 @@ public: if ( bRes == sal_False ) break; } - CPPUNIT_ASSERT_MESSAGE("createUuid: every uuid must be different.", bRes == sal_True ); - }*/ - - CPPUNIT_TEST_SUITE(createUuid); - CPPUNIT_TEST(createUuid_001); - //CPPUNIT_TEST(createUuid_002); - CPPUNIT_TEST_SUITE_END(); -}; // class createUuid + ASSERT_TRUE(bRes == sal_True) << "createUuid: every uuid must be different."; +}*/ namespace ThreadHelper { @@ -155,70 +153,64 @@ namespace ThreadHelper } } -class createNamedUuid : public CppUnit::TestFixture +class createNamedUuid : public ::testing::Test { public: // initialise your test code values here. - void setUp() + void SetUp() { } - void tearDown() + void TearDown() { } +}; // class createNamedUuid - void createNamedUuid_001() - { - sal_uInt8 NameSpace_DNS[16] = RTL_UUID_NAMESPACE_DNS; - sal_uInt8 NameSpace_URL[16] = RTL_UUID_NAMESPACE_URL; - sal_uInt8 pPriorCalculatedUUID[16] = { - 0x52,0xc9,0x30,0xa5, - 0xd1,0x61,0x3b,0x16, - 0x98,0xc5,0xf8,0xd1, - 0x10,0x10,0x6d,0x4d }; - - sal_uInt8 pNamedUUID[16], pNamedUUID2[16]; - - // Same name does generate the same uuid - rtl_String *pName = 0; - rtl_string_newFromStr( &pName , "this is a bla.blubs.DNS-Name" ); - rtl_createNamedUuid( pNamedUUID , NameSpace_DNS , pName ); - rtl_createNamedUuid( pNamedUUID2 , NameSpace_DNS , pName ); - CPPUNIT_ASSERT_MESSAGE( "Same name should generate the same uuid", ! memcmp( pNamedUUID , pNamedUUID2 , 16 ) && rtl_compareUuid( pNamedUUID , pNamedUUID2 ) == 0 ); - CPPUNIT_ASSERT_MESSAGE( "Same name should generate the same uuid", ! memcmp( pNamedUUID , pPriorCalculatedUUID , 16 ) ); - - // Different names does not generate the same uuid - rtl_string_newFromStr( &pName , "this is a bla.blubs.DNS-Namf" ); - rtl_createNamedUuid( pNamedUUID2 , NameSpace_DNS , pName ); - CPPUNIT_ASSERT_MESSAGE("Different names does not generate the same uuid.", memcmp( pNamedUUID , pNamedUUID2 , 16 ) ); - - // the same name with different namespace uuid produces different uuids - rtl_createNamedUuid( pNamedUUID , NameSpace_URL , pName ); - CPPUNIT_ASSERT_MESSAGE( " same name with different namespace uuid produces different uuids", memcmp( pNamedUUID , pNamedUUID2 , 16 ) && rtl_compareUuid( pNamedUUID , pNamedUUID2 ) != 0); - - //test compareUuid - if ( rtl_compareUuid( pNamedUUID , pNamedUUID2 ) > 0 ) - { CPPUNIT_ASSERT_MESSAGE( " compare uuids", rtl_compareUuid( pNamedUUID2 , pNamedUUID ) < 0); - } - else - CPPUNIT_ASSERT_MESSAGE( " compare uuids", rtl_compareUuid( pNamedUUID2 , pNamedUUID ) > 0); - - rtl_string_release( pName ); +TEST_F(createNamedUuid, createNamedUuid_001) +{ + sal_uInt8 NameSpace_DNS[16] = RTL_UUID_NAMESPACE_DNS; + sal_uInt8 NameSpace_URL[16] = RTL_UUID_NAMESPACE_URL; + sal_uInt8 pPriorCalculatedUUID[16] = { + 0x52,0xc9,0x30,0xa5, + 0xd1,0x61,0x3b,0x16, + 0x98,0xc5,0xf8,0xd1, + 0x10,0x10,0x6d,0x4d }; + + sal_uInt8 pNamedUUID[16], pNamedUUID2[16]; + + // Same name does generate the same uuid + rtl_String *pName = 0; + rtl_string_newFromStr( &pName , "this is a bla.blubs.DNS-Name" ); + rtl_createNamedUuid( pNamedUUID , NameSpace_DNS , pName ); + rtl_createNamedUuid( pNamedUUID2 , NameSpace_DNS , pName ); + ASSERT_TRUE(! memcmp( pNamedUUID , pNamedUUID2 , 16 ) && rtl_compareUuid( pNamedUUID , pNamedUUID2 ) == 0) + << "Same name should generate the same uuid"; + ASSERT_TRUE(! memcmp( pNamedUUID , pPriorCalculatedUUID , 16 )) << "Same name should generate the same uuid"; + + // Different names does not generate the same uuid + rtl_string_newFromStr( &pName , "this is a bla.blubs.DNS-Namf" ); + rtl_createNamedUuid( pNamedUUID2 , NameSpace_DNS , pName ); + ASSERT_TRUE(memcmp( pNamedUUID , pNamedUUID2 , 16 )) << "Different names does not generate the same uuid."; + + // the same name with different namespace uuid produces different uuids + rtl_createNamedUuid( pNamedUUID , NameSpace_URL , pName ); + ASSERT_TRUE(memcmp( pNamedUUID , pNamedUUID2 , 16 ) && rtl_compareUuid( pNamedUUID , pNamedUUID2 ) != 0) + << " same name with different namespace uuid produces different uuids"; + + //test compareUuid + if ( rtl_compareUuid( pNamedUUID , pNamedUUID2 ) > 0 ) + { ASSERT_TRUE(rtl_compareUuid( pNamedUUID2 , pNamedUUID ) < 0) << " compare uuids"; } + else + ASSERT_TRUE(rtl_compareUuid( pNamedUUID2 , pNamedUUID ) > 0) << " compare uuids"; - CPPUNIT_TEST_SUITE(createNamedUuid); - CPPUNIT_TEST(createNamedUuid_001); - CPPUNIT_TEST_SUITE_END(); -}; // class createNamedUuid + rtl_string_release( pName ); +} -CPPUNIT_TEST_SUITE_NAMED_REGISTRATION(rtl_Uuid::createUuid, "rtl_Uuid"); -CPPUNIT_TEST_SUITE_NAMED_REGISTRATION(rtl_Uuid::createNamedUuid, "rtl_Uuid"); } // namespace rtl_Uuid -// ----------------------------------------------------------------------------- - -// this macro creates an empty function, which will called by the RegisterAllFunctions() -// to let the user the possibility to also register some functions by hand. -NOADDITIONAL; - - +int main(int argc, char **argv) +{ + ::testing::InitGoogleTest(&argc, argv); + return RUN_ALL_TESTS(); +} _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits