You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celix.apache.org by "Johan (JIRA)" <ji...@apache.org> on 2019/02/28 15:01:00 UTC

[jira] [Commented] (CELIX-447) Refactor CppUTest tests to gtest

    [ https://issues.apache.org/jira/browse/CELIX-447?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16780604#comment-16780604 ] 

Johan commented on CELIX-447:
-----------------------------

Potential issues would be:

 
 # CppUTest is easy to build for an end-user and is typically distributed with the package manager. For gtest+gmock this is not the case and the end-user needs to build and install from source by themselves. Alternatively Celix could bundle the binaries and headers, which would add work on the project side instead of the user side.
 # gmock does not support mocking C functions exactly, there are ways of working around this[0][1] which either introduces extra boilerplate or an extra dependency.

 

 [0] [https://github.com/hjagodzinski/C-Mock]

[1] https://stackoverflow.com/questions/31989040/can-gmock-be-used-for-stubbing-c-functions

> Refactor CppUTest tests to gtest
> --------------------------------
>
>                 Key: CELIX-447
>                 URL: https://issues.apache.org/jira/browse/CELIX-447
>             Project: Celix
>          Issue Type: Improvement
>          Components: Testing
>            Reporter: Pepijn Noltes
>            Priority: Major
>             Fix For: 2.3.0
>
>
> Currently CppUTest is used for testing and mocking and although this framework works fine, I would be handier to move to gtest( google test) for a few reasons:
> 1) more easier to integrate as a external project
> 2) much more heavily used than CppUTest
> 3) integrated support in the CLion IDE. This means that the IDE supports  directly running/debugging individually testcases/testsuites.
> This would also mean that the cpputest mocks should be removed. This can be done as long as we keep the code coverage as good as intact.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)