Skip to content
Snippets Groups Projects

[C/C++] Unittests with googletest

Merged Jonas Jucker requested to merge c_c++_tests into master

This MR adds C unit tests using googletest.

Edited by Yen-Chen Chen

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jonas Jucker mentioned in merge request !5 (closed)

    mentioned in merge request !5 (closed)

  • Author Maintainer

    @b382190 Could you make this set of tests more complete once you are back from holiday?

  • just my 2cts: IMO the C-part of libfortran-support is minimal compared to the rest. using googletest for that part is reasonable, but for the fortran code it seems like an overkill.

    This approach is more like a feasibility study, but I would be regard it for a production solution.

    Isn't there a simpler pure-fortran solution for this?

  • Author Maintainer

    For a more complete set of Fortran unittest please have a look at !13 (merged) .

    The idea is that C/C++ test are in googletest. Also we might want to test some of the fortran-wrappers of C-function and compare directly to results from the C-function -> there we need googletest.

    The test present in this MR were part of our investigation about unittest possibilities. It needs a consolidation and cleanup, but also more tests for the C/C++ parts.

    This should be done by @b382190 once he is back from holiday.

  • Some small comments

  • Jonas Jucker deleted the cmake_for_tests branch. This merge request now targets the master branch

    deleted the cmake_for_tests branch. This merge request now targets the master branch

  • Yen-Chen Chen added 1 commit

    added 1 commit

    Compare with previous version

  • Yen-Chen Chen added 1 commit

    added 1 commit

    • c7b21da9 - Set macro for all google tests

    Compare with previous version

  • Yen-Chen Chen added 3 commits

    added 3 commits

    • c0b2cb7c - Avoid unresolved symbols on macOS
    • 13c32569 - Resolve unused signal code for MAC OS
    • aa2336d8 - Merge remote-tracking branch 'origin/fix-unresolved-symbols-macos' into c_c++_tests

    Compare with previous version

  • Yen-Chen Chen added 1 commit

    added 1 commit

    Compare with previous version

  • Yen-Chen Chen added 1 commit

    added 1 commit

    Compare with previous version

  • Yen-Chen Chen added 1 commit

    added 1 commit

    Compare with previous version

  • Yen-Chen Chen added 5 commits

    added 5 commits

    Compare with previous version

  • Yen-Chen Chen changed target branch from master to testing

    changed target branch from master to testing

  • Yen-Chen Chen changed target branch from testing to master

    changed target branch from testing to master

  • Yen-Chen Chen added 1 commit

    added 1 commit

    • f1c44c46 - Change FetchContent for googletest

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading