Adding Catch2 for unittests, reworked CI #165

Merged

muellerr merged 28 commits from mohr/catch2 into develop 2022-03-03 13:34:24 +01:00

Conversation 1 Commits 28 Files Changed 29 +484 -132

mohr commented 2022-03-02 18:07:14 +01:00

Owner

Copy Link

This adds Catch2 as a cmake dependency.

It also introduces a first POC unittest, for the almost-empty thermal controller.

Unittests are new optional target eive-unittest.

CI was changed to use a named docker build, which is built from a tagged commit docker_d* (to be automated).

CI was build was cleared up, inspired by the fsfw CI.

To be able to run the fsfw on the docker container, a docker parameter for the message queues was added.

This adds Catch2 as a cmake dependency. It also introduces a first POC unittest, for the almost-empty thermal controller. Unittests are new optional target `eive-unittest`. CI was changed to use a named docker build, which is built from a tagged commit `docker_d*` (to be automated). CI was build was cleared up, inspired by the fsfw CI. To be able to run the fsfw on the docker container, a docker parameter for the message queues was added.

mohr added 27 commits 2022-03-02 18:07:16 +01:00

catch2 running d9badee18f

unittesting a skeleton controller, still withouth fsfw f9721eb1ae

skeleton controller now unittested including fsfw. Still dirty WIP f1249392cf

some more fsfw objects and stubs for unittesting 587e9451ea

more mocking an such in unittests 0f42790aa1

working on thermal controller unit test. e95647c572

added EventManagerMock, to be extended and moved into fsfw some day 3400c538b3

continued work on event manager mock, slightly usable now 334ba6d4bf

I like my debug output really pretty 090979e5a9

cleanup 4ea8ff9876

reworked test setup bcac56a65c

Updated Dockerfile and Jenkinsfile 57280167a2

Merge branch 'mohr/automation' into mohr/catch2 62531a710f

what kind of language is this? 29efde6cbb

Merge branch 'mohr/automation' into mohr/catch2 4704e6deb8

tweaked build of catch2 ... 52e420b0be

-only built when needed
-no more warnings as catch2 is compiled with default compiler flags

enable unittests in Jenkinsfile 54e3b99708

added libgps to docker to be able to build for linux 6a8c9e14e9

Merge branch 'mohr/automation' into mohr/catch2 5bd613fb57

c'mon... 15bab6ae33

argh c37e7022d1

changing fs.mqueue.msg_max for docker to be able to run unit tests 606da48861

small fixes 182b90ee6d

Merge branch 'develop' into mohr/catch2 2f51f58c7e

forgot one 05bd1cc8c6

Merge branch 'mohr/buildfix' into mohr/catch2 f413ed6f7d

using new docker container name 221d7b408a

mohr commented 2022-03-02 18:08:27 +01:00

Author

Owner

Copy Link

As soon as eive/fsfw#36 is in use and #164 is merged, CI builds should work again.

As soon as https://egit.irs.uni-stuttgart.de/eive/fsfw/pulls/36 is in use and #164 is merged, CI builds should work again.

mohr requested review from muellerr 2022-03-02 18:08:44 +01:00

muellerr added 1 commit 2022-03-03 10:34:57 +01:00

submodule updates 458b33e5fd

muellerr approved these changes 2022-03-03 13:34:18 +01:00

muellerr merged commit e07713a6f7 into develop 2022-03-03 13:34:24 +01:00

muellerr referenced this issue from a commit 2022-03-03 13:34:25 +01:00

Merge pull request 'Adding Catch2 for unittests, reworked CI' (#165) from mohr/catch2 into develop

muellerr referenced this pull request 2022-03-03 13:37:17 +01:00

Develop is currently broken #164

mohr deleted branch mohr/catch2 2022-03-03 15:19:59 +01:00

muellerr added this to the v1.9.0 milestone 2022-03-08 10:19:32 +01:00

Sign in to join this conversation.

Reviewers

No reviewers

muellerr

Labels

Clear labels   

OPS TODO

Issues identified by OPS

  

api change

API change

  

breaking api change

API change which can break existing code

  

bug

Something is not working

  

documentation

Documentation

  

duplicate

This issue or pull request already exists

  

feature

New feature

  

help wanted

Need some help

  

invalid

Something is wrong

  

question

More information is needed

  

wontfix

This won't be fixed

No Label

OPS TODO

api change

breaking api change

bug

documentation

duplicate

feature

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No Milestone

v1.9.0

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: eive/eive-obsw#165

No description provided.