eive/eive-obsw
eive
/
eive-obsw
12
3
Fork 0
Code Issues 3 Pull Requests 2 Releases 117 Wiki Activity

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
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
f1249392cf skeleton controller now unittested including fsfw. Still dirty WIP
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
0f42790aa1 more mocking an such in unittests
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
e95647c572 working on thermal controller unit test.
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
3400c538b3 added EventManagerMock, to be extended and moved into fsfw some day
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
bcac56a65c reworked test setup
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
62531a710f Merge branch 'mohr/automation' into mohr/catch2
EIVE/eive-obsw/pipeline/head This commit looks good Details
4704e6deb8 Merge branch 'mohr/automation' into mohr/catch2
52e420b0be tweaked build of catch2 
-only built when needed
-no more warnings as catch2 is compiled with default compiler flags
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
5bd613fb57 Merge branch 'mohr/automation' into mohr/catch2
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
15bab6ae33 c'mon...
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
c37e7022d1 argh
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
EIVE/eive-obsw/pipeline/pr-develop There was a failure building this commit Details
182b90ee6d small fixes
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
2f51f58c7e Merge branch 'develop' into mohr/catch2
EIVE/eive-obsw/pipeline/pr-develop There was a failure building this commit Details
05bd1cc8c6 forgot one
EIVE/eive-obsw/pipeline/head There was a failure building this commit Details
EIVE/eive-obsw/pipeline/pr-develop There was a failure building this commit Details
221d7b408a using new docker container name
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
EIVE/eive-obsw/pipeline/pr-develop This commit looks good Details
EIVE/eive-obsw/pipeline/head This commit looks good Details
458b33e5fd
submodule updates
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
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
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.
Delete Branch "mohr/catch2"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?